Как осуществлять подавление ошибок в PHP?

Question

[tushev]

Я часто вижу, что пишут примерно такой код.

if(isset($arr['a']) && isset($arr['a']['b']) && isset($arr['a']['b']['c'])) {
    $v = $arr['a']['b']['c'];
}
else {
    $v = null;
}

Но ведь тоже самое, можно написать компактнее, воспользовавшись подавлением ошибки доступа к элементам массива. В случае ошибки будет возвращен null.
$v = @$arr['a']['b']['c'];
Почему многие предпочитают первый способ? Религия не позволяет использовать подавление ошибок даже для минимального выражения? Или есть еще какие-то причины?

Comments

[tushev]

Мне привели упрощенные способы типа:
$v = isset($arr['a']['b']['c']) ? $arr['a']['b']['c'] : null;
Но, все равно, здесь нужно будет повторить выражение $arr['a']['b']['c'] дважды. Сначала для проверки на наличие элемента, а потом для доступа. И в приведенном примере выражение очень простое. А может быть что то мудреное, типа:
$arr[$tables[$t]['name']]['rows'][$r]['cols'][$c]['format']['borders']['top']

Answer 1

[andymitrich]

Не знаю насчет религии, но точно можно было бы обойтись одним isset($arr['a']['b']['c']) в условии.

Comments

[tushev]

Даже в вашем примере, нужно будет повторить выражение $arr['a']['b']['c'] дважды. Сначала для проверки на наличие элемента, а потом для доступа. И в приведенном примере выражение очень простое. А может быть что то мудреное, типа:
$arr[$tables[$t]['name']]['rows'][$r]['cols'][$c]['format']['borders']['top']

[andymitrich]

Ну воспользуйтесь тернарным оператором - не надо будет повторять выражение дважды.

[tushev]

Как это? С тернарным как раз и придется повторить дважды. В первом и во втором аргументе.

[andymitrich]

Начиная с верии 5.3, тернарный оператор в PHP допускает сокращенную запись, то есть позволяет опустить среднюю часть, если условие истинно.

[tushev]

Спасибо за подсказку о сокращенном тернарном операторе. Я это нововведение упустил. Удобная штука.

[Назар Мокринский]

@andymitrich Ваш синтаксис не корректен, при отсутствии $arr['a'] либо $arr['a']['b'] получите ошибку

[andymitrich]

@nazarpc Позвольте не согласиться - мой синтаксис корректен. В приведенном отрезке кода я не использую никаких запрещенных в PHP конструкций - вероятно, вы выражаетесь не корректно.
Далее, перед тем как опубликовать здесь этот код, я опробовал его у себя на локальной машине и проверил - советую вам впредь делать то же самое, тогда, возможно, надобность в таких комментариях отпадет.

[Назар Мокринский]

@andymitrich Действительно, прошу прощения. Почему-то всегда был уверен, что вложенные нужно проверять поэлементно.

Answer 2

[Александр Зеленин]

если есть ошибка, ее нужно устранить, а не подавить. Подавление ошибок - один из признаков говнокода.

Comments

[tushev]

Не устранить, а гарантированно поймать. Ведь не считается грехом запихивать участок кода в try-catch.

В приведенном примере, хоть формально и выполняется подавление, но на практике ошибка обрабатывается путем возврата значения null. Последующий код, разумеется должен быть расcчитан на значение null.

Answer 3

[sajgak]

$v = isset($arr['a']['b']['c']) ? $arr['a']['b']['c'] : null;

Answer 4

[catanfa]

вот здесь автор предлагает несколько решений, объясняет подводные камни nikic.github.io/2014/01/10/The-case-against-the-if...

а если начать использовать @, то можно потерять действительно нужные ошибки и отладка станет очень тяжелой. Представьте, ничего плохого не произойдёт, если вы одну-единственную переменную в проекте назовёте $a;Но если таких переменных будет много, код станет невыносимым. Здесь похоже такая же логика.

Answer 5

[Push Pull]

$arr[$tables[$t]['name']]['rows'][$r]['cols'][$c]['format']['borders']['top'] - это конечно 3.14здец товарищи, но такая конструкция мне никогда не встречалась.

Есть несколько причин:

1) Наглядно видно что происходит в скрипте (есть ключ? нет - ставим default)
2) Упрощает отладку в дальнейшем. "@" скрывает ВСЕ ошибки.
3) Потому, что оператор "@" замедляет выполнение конструкции вида iffset(key) ? key : default в два раза. Даже если ошибки не возникает.

И если уж исходить из редких ситуаций. Представьте насколько замедлится скрипт при прохождении 100000 элементов.

Comments

[tushev]

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

Answer 6

[Назар Мокринский]

if(isset($arr['a']['b']['c'])) {
    $v = $arr['a']['b']['c'];
}
else {
    $v = null;
}

$v = isset($arr['a']['b']['c']) ? $arr['a']['b']['c'] : null;

Потому что:
1) сразу понятно что имелось ввиду
2) быстрее
3) ошибки нужно не допускать, а допускать и подавлять сознательно - очень плохой тон, не сразу понятно что вы ожидали получить в результате, это быдлокод, не поддающийся отладке
4) становится похоже на то, что вы вообще не понимаете, что происходит и решили просто убрать отображение ошибки, то что вы там ожидаете null не очевидно чуть менее чем полностью

$arr[$tables[$t]['name']]['rows'][$r]['cols'][$c]['format']['borders']['top']

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