Почему нужно писать так?

Question

[JTester]

Здравствуйте! Я вот заинтересовался таким вопросом, в чём разница данного кода? Первый вариант просто более правильный, верно?

1-й вариант:

if (($test = $this->getTest()) !== null) {
//
}

2-й вариант

if ($test = $this->getTest() !== null) {
//
}

Вот в чем разница?

Comments

[nokimaro]

А можно ещё и так

if(null !== $test = $this->getTest()) {
//
}

Answer 1

[Rsa97]

С точки зрения стиля и то и то плохо.
С точки зрения синтаксиса присвоение имеет меньший приоритет, чем сравнение. Следовательно, вторая запись будет распознана как

if ($test = ($this->getTest() !== null)) {
//
}

Comments

[JTester]

А вот ещё вопрос, он конечно может быть не в тему
Скажем у меня в базе есть столбец "test" и он может быть пустым, также у меня есть функция getTest()
Мне нужно проверять, есть ли у меня данный столбец? Или же можно сразу вернуть значение данного столбца?

[Rsa97]

у меня в базе есть столбец - есть ли у меня данный столбец?

Если у вас в базе есть столбец, то он есть. Если у него значение null, то он всё равно есть, просто со значением null. Что должна в этом случае возвращать функция - это вопрос либо к вам, если вы единственный разработчик, либо к тому, кто выдал задание на написание этой функции.

Answer 2

[Илья]

Первый вариант просто более правильный, верно?

Неверно.

https://www.php.net/manual/ru/language.operators.p...

Comments

[JTester]

За это спасибо, не находил такое :)

[rPman]

JTester, дополню
в данном случае возможно это не проблема, так как использован оператор !==
но все же, рекомендую поглядывать сюда