Есть ли разница?

Question

[aopil]

if ($test == "Text")
{
     // code
}
else
{
     if ($test == "ABC")
     {
     // code
     }
}

и

if ($test == "Text")
{
     // code
}
elseif ($test == "ABC")
{
     // code
}

Подскажите, есть ли разница? Встретил в коде первый вариант, но мне второй вроде больше нравится.

Comments

[FanatPHP]

за первый бьют
не сильно, но линейкой по пальцам тоже неприятно получать

[aopil]

FanatPHP, т.е. первый вариант правильный?

[FanatPHP]

перепутал. второй правильный

[Сергей Соколов]

фиг. скобки — на той же строке. См. PSR-12: Extended Coding Style:

if ($expr1) {
    // if body
} elseif ($expr2) {
    // elseif body
} else {
    // else body;
}

Answer 1

[DevMan]

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

вам бы почитать про метрики кода в целом, и цикломатическую сложность в частности.

Comments

[aopil]

DevMan

if ($i == 0)
{
}
elseif ($i == 1)
{
}
elseif ($i == 2)
{
}
elseif ($i == 3)
{
}
elseif ($i == 4)
{
}
elseif ($i == 5)
{
}

Получается в таком варианте, когда проверок идет много, лучше использовать первый вариант?

Мне казалось наоборот)

[DevMan]

aopil, в этом варианте вообще проще использовать switch/match.
и внимательно прочтите мой ответ.

[FanatPHP]

Мне казалось наоборот)

ага и сдвинуть весь код вправо чтобы он вообще не читался

[aopil]

FanatPHP, ну кто-то написал код который я сейчас изучаю, и да, весь код сдвинут вправо, и читать так не удобно, ну лично мне, поэтому и решил спросить как правильно, и есть ли может какая-то разница.

[DevMan]

aopil, я скажу крамолу: если код работает как надо - это правильно.
красота/оптимальность - понятия другого рода.

[aopil]

DevMan, часть вопроса была как раз в том, что возможно есть какая-то разница между двумя приведенными кода, в плане отработки, видимо ее нет, поэтому буду использовать второй вариант - более читаемо получается.

[DevMan]

aopil, я пытался донести простую мысль: если два варианта кода делают одно и то же - используйте тот, который вам понятней, а не который "правильней".
возможно, со временем, вы измените своё мнение. возможно - нет.

[edward_freedom]

aopil, можно вообще без условий с помощью словаря

Answer 2

[Илья]

Есть, код проще для понимания

Comments

[aopil]

Илья в каком из случаев для вас проще?

[Илья]

aopil, второй, многоуровневые условия потом трудно разбирать