Как лучше сравнивать переменную со значением в условии PHP?

Question

[Andrey]

Много раз видел, что кодеры в условии указывают

if (3 == $someVar) {
    ...
}

Чем оно лучше такого сравнения?

if ($someVar == 3) {
    ...
}

Answer 1

[Алексей Яхненко]

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

Например, ты можешь случайно написать
if ($user = 'admin') {

И кто бы ни был юзер, отныне он станет админом. Ну, в рамках текущего запроса.

А вот 'admin' = $user написать не получится.

По той же причине крайне не рекомендуется использовать присваивание в условии, многие любят так делать, типа
if ($user = loadUser()) {
поскольку не всегда можно точно определить, это опечатка или так и задумано. Условие должно быть условием.

Comments

[lnked]

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

if (!$this->mcache_enable || ($this->mcache_enable && !$value = $this->mcache->get('value'))) {
$value = '1';
}
echo $value;

[Алексей Яхненко]

Лично моё мнение - это какой-то ужас а не условие. Но это лично мое мнение :)Эдик:

[lnked]

Алексей Яхненко: Да смотрится страшно)) но я не пишу else с таким условием и сразу получаю данные

[65536]

а потом через пол года сам не понимаешь и приходится втыкать и парсить мозгом. лучше выносить. а совсем сложные условия лучше вообще разбивать на части по смыслу, типа такого gyazo.com/0ee38a7f69636df726033c4b1fc23485

[Алексей Яхненко]

65536: ага, только не стоит называть переменные a, b, c. Можно вообще все условие выделить в функцию с понятным названием.

[65536]

Алексей Яхненко: я для себя решил, что это единственный случай когда можно, потому-что вот они объявлены, а вот они использованы и на этом их участие окончено. это просто условная группа и не обязательно выдумывать осмысленное имя, тем более не всегда может получиться или получится очень длинное. а так да руки можно рубить за эти а б м х ср бб б1 итд

[Алексей Яхненко]

65536: как раз таки обязательно. Мне придётся распарсиь эти 3 длинные строчки чтобы понять, о чем условие. А написал бы
if ($cat_not_set && $model_not_set,...) или что оно там значит - сразу все понятно.
Ну и заодно от кучи отрицаний в условии можно избавиться, а то оно тоже не способствует быстрому пониманию.

[65536]

Алексей Яхненко: быть может, но тогда уж !$has_model_column_but_model_not_set && !$has_cat_column_but_cat_not_set && !$has_manufacturer_columns_but_manufacturer_not_set, а без отрицаний даже не знаю)

[65536]

Алексей Яхненко: ну да, вы правы, взглянул на них от третьего лица и понял, что материл бы автора

Answer 2

[sivabur]

ПО скорости одно и тоже. Пишется исключительно из того чтобы случайно не написать так:

if (3 =$someVar) {
    ...
}

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

Comments

[DevMan]

sivabur: мыслите верно, только пробел здесь не при делах.

[Алексей Яхненко]

Ты какую-то фигню по-моему написал. Перепроверь.

[sivabur]

Спешил ошибся да вместо = пробел написал))исправил)

[sivabur]

DevMan: машинально не то слово написал))

[sivabur]

Я же первый ответил ех((

Answer 3

[Олег Титаренко]

Без разницы. Тут дело в читаемости
в первом варианте ты указываешь что если 3 = Х
во втором варианте если X = 3

и все)

Answer 4

[index0h]

== используйте только в случае, если уверены в типах данных с обоих сторон, в противном случае ===