Как писать условие if если очень много значений для проверки, а нужно чтобы они за экран не залезали, не превращались в очень длинную строку?

Question

[Nentra]

Как писать условие if если очень много значений для проверки, а нужно чтобы они за экран не залезали, не превращались в очень длинную строку?

Comments

[Saboteur]

Уменьшить шрифт

Answer 1

[Ипатьев]

Это очень хороший вопрос!

Для длинных условий можно и нужно использовать такую замечательную практику, как именованные условия!
В большинстве случаев, когда условия не зависят друг от друга, можно просто присвоить результат вычисления условия переменной с обязательно говорящим именем, и дальше уже использовать эти переменные:

$dateIsValid = $obj->getDate() >= getCurrentDate();
$balanceOk = $obj->getBalance() > 0;
// Любые другие условия
if ($dateIsValid && $balanceOK) ...

Это повышает читабельность не только сокращая само тело условного оператора, но в первую очередь превращая его в связный текст!

Если же условия зависят друг от друга, то можно оформлять их в виде функций (методов). И тогда они не будут выполнены, если до них не дойдет очередь.

Comments

[Adamos]

Эта замечательная практика будет впустую выполнять getBanance() в вашем коде, если дата не валидна. Возможно, дергая для этого базу и создавая прочий оверхед на ровном месте.

[v3shin]

Adamos, решается созданием функций/методов для проверок.

[Adamos]

v3shin, отдельная функция только для того, чтобы вызвать отдельную функцию - тоже не лучший код. Мы же понимаем, что компьютеру-то без разницы, какой длины строчка в программе. А вот вызов отдельных методов в ряде языков будет тратить лишние ресурсы, а то и мешать компилятору оптимизировать.

[v3shin]

Adamos, выглядит как экономия на спичках. Имхо, if ($obj->dateIsValid() && $obj->balanceOK()) {}
будет отрабатывать не намного медленнее расписанных условий.

[Ипатьев]

Adamos, компьютеру все равно, а человеку не все равно.
Мы все поначалу пишем нубский код вида if ($p>0&&$r) echo"..." - как будто нас штрафуют за каждый лишний символ :)

Но с опытом приходит понимание, что пишем мы всё-таки для программиста, и читабельность кода очень важна.

И да, прочитать информативное название метода куда проще, чем даже одну строчку ванильного кода. Это называется рефакторинг

[Adamos]

v3shin, в одноразовом коде (который в вебе довольно обычен) действительно экономия никакая. Пока не достигнуты масштабы, где эта экономия умножается на миллионы.
Но здесь речь о методах и коде вообще, и тут я сужу с позиции человека, которому доводилось профилировать код, выпоняющийся в цикле миллионы раз в секунду. Там "спичек" не бывает... и опытный программист не разбрасывается ресурсами, даже столь ничтожными, совершенно машинально. Даже в вебе.

[Adamos]

Роман Юрьевич Ипатьев, рефакторинг требуется, когда код стал плохо читаться или хуже работать.
Для описываемого случая читаемость без проблем восстанавливается разбиением на строчки \n\t&& и написанием над каждой из них комментария, что проверяет эта часть. И компьютеру не помешает, и читаться будет, как роман.

[Ипатьев]

Adamos, Разбиение условия на строчки - это ужас.
Но тут наверное уже вопрос субъективного восприятия.
Но про вызов функции в - это уже реально экономия на спичках :)

[Adamos]

Роман Юрьевич Ипатьев, когда все крутится в виртуальной машине - вызовы функций легкие и не создают больше проблем, чем все остальное. Когда же дело происходит ближе к машинному коду - вызов каждой функции означает оверхед, с которым компилятор старается бороться, избавляясь от вызова функции вообще. Часто ему это удается, но не всегда. Так что насчет "реально" - it depends.

[Сергей delphinpro]

Разбиение условия на строчки - это ужас.

Да с какого хрена-то?
Даже ваше

if (
     $dateIsValid
  && $balanceOK
) {
  // if body
}

Лучше читается чем в одну строку. Не говоря уже о конструкциях длиннее 60 символов

[Сергей delphinpro]

v3shin,

Имхо, if ($obj->dateIsValid() && $obj->balanceOK()) {}
будет отрабатывать не намного медленнее расписанных условий.

Не намного быстрее вы хотели сказать. Потому что вторая функция не будет выполнена, если первая вернула false

[v3shin]

Сергей delphinpro, сравниваю с кодом

if (
    $obj->getDate() >= getCurrentDate()
    && $balanceOk = $obj->getBalance() > 0
) {}

Answer 2

[Сергей delphinpro]

PSR-2: https://www.php-fig.org/psr/psr-2/
PSR-12: https://www.php-fig.org/psr/psr-12/

if (
    $expr1
    && $expr2
) {
    // if body
} elseif (
    $expr3
    && $expr4
) {
    // elseif body
}

Comments

[galliard]

Но выглядит ужасно

Answer 3

[Rsa97]

It depends.
Абстрактных решений тут не будет, всё зависит от того, что это именно за условия.
Из стандартных методов - упрощение логических выражений, вынесение условий в функции, замена проверок вида x == 1 || x == 2 || ... || x == n на [1, 2, ..., n].includes(x) и т.п.