Php! или === null?

Question

[4sadly]

Надо проверить переменную $error на наличие ошибок, что лучше:
if($error === null){}
или
if(!$error){}
?

Answer 1

[Максим Федоров]

if($error === null){} предпочтительный (иногда он обязательный по стандартам), тк вариант if(!$error){} — это авто-приведение к bool:

• например 0 приведется к false, в то время как он может быть валидным интовым значением
  
• пустой массив — тоже может быть валидным значением, но с !$arr он приведется к false... конечно допускать в программе одновременно и [] и null стоит как можно реже, но такое может произойти
  

!$any можно использовать комфортно, когда вы (и программа) уверена, что работает либо с объектом, либо с bool

Answer 2

[FanatPHP]

если переменная была заранее объявлена, то
if(!$error){}
если не была, то
if(!empty($error)){}

Comments

[4sadly]

У меня есть функция, которая возвращает либо строку с ошибкой либо ничего, что в таком случае?

[FanatPHP]

Я так понимаю что ты присваиваешь переменной какое-то значение.
Это значит переменная объявлена перед проверкой.
То есть первый вариант

Answer 3

[rPman]

Если есть возможность, меняйте тип возвращаемого значения, т.е. зоведите специальный класс Error и после вызова проверяйте тип результата.

Еще есть практика выноса статуса ошибок из возвращаемого значения, так как ошибки бывают разные и требуют разной ответной реакции, с помощью исключений - классов потомков Exception

В этом случае каждый вариант ошибки должен быть отловлен в своей секции catch а вызов в try (саму ошибку инициализировать с помощью throw).

Comments

[FanatPHP]

Нет. Функция которая возвращает разный тип результата - это говнокод.
Исключение уже лучше, но тут надо понимать две вещи
1. Если исключение кидается только для того чтобы его тут же поймать - оно не нужно
2. Автор мне только что написал, что у него функция лабо возвращает ошибку, или ничего. То есть это функция проверки чего-либо, и ошибка - это её нормальное поведение. А исключение должно кидаться только если функция не может выполнить свою работу.

[rPman]

Окей, считаете очень штатные вещи говнокодом?, а как вам возвращать две переменные?
list($result,$error)=$obj->method()

или возвращать ответ в переменных?
$error=$obj->method(&$result)

или выверты в переменных или даже спец методы класса (и такое бывает даже в тему)
$result=$obj->method()
$error=$obj->getLastError()

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