Использование тернарного оператора ?: с подавлением ошибки?

Question

Владимир @lazycommit

PHP

Использование тернарного оператора ?: с подавлением ошибки?

Нормально ли использовать $c = @$a ? : 'default';вместо$c = isset($a) ? $a : 'default';?

И вроде ошибка, всего лишь, уровня E_NOTICE, а @ смущает все равно. С другой стороны — удобно же!

Кто как считает?

UPD1: Так как empty() игнорирует неопределенную переменную, а «Таблица сравнения типов в PHP» говорит, что !empty() полностью соответствует нестрогому булеву, то корректнее была бы замена:$c = !empty($a) ? $a : 'default'; на $c = @$a ? : 'default';
UPD2: (так как никто не читает уточнения). Рассматриваются, как основные варианты, так же доступ к неопределенным ключам массивов и магическим свойствам объекта, которые могут быть заданы извне.

$e = !empty($array['doubtful']['element']) ? $array['doubtful']['element'] : 'default';
// and
$e = @$array['doubtful']['element'] ? : 'default';

$p = !empty($object->magicProperty) ? $object->magicProperty : 'default';
// and
$p = @$object->magicProperty ? : 'default';

Вопрос задан более трёх лет назад
5310 просмотров

2 комментария

Подписаться 5 Оценить 2 комментария

Владимир @lazycommit Автор вопроса

Примеры использования:

$v = !empty($viewVariable) ? $viewVariable : 'default';
// and
$v = @$viewVariable ? : 'default'; //one pity IDE definition warning

$e = !empty($array['doubtful']['element']) ? $array['doubtful']['element'] : 'default';
// and
$e = @$array['doubtful']['element'] ? : 'default';

$p = !empty($object->magicProperty) ? $object->magicProperty : 'default';
// and
$p = @$object->magicProperty ? : 'default';

Написано более трёх лет назад

Владимир @lazycommit Автор вопроса

Такие моменты вынужденной проверки предположительно не инициализированных переменных возникают, в основном, из устарелого кода, внешних (не задокументированных) источников данных, магии и прочего отсутствия профессионализма разработчиков.
Например:
Самопальный фреймворк, наследие php4, тяжелые попытки перехода на ООП и MVC парадигму. В контроллерах магические свойства получают данные запроса. Контроллеры передаются аргументом на обработку во всяческие менеджеры, где могут быть изменены их (магические опять же) свойства.
Это очень печальные примеры. И хотелось бы больше никогда не встречать такого на практике, но изначально ошибки уровня «Переменная не инициализирована» просто игнорировались на уровне error_reporting(E_ERROR) и никакая проверка не проводилась. Разработку вести с такой директивой совесть не позволяет. Поэтому пришлось расставлять проверки. Но это выявило лексическую тяжбу в PHP, которую хочется решить наиболее удобным методом.

Написано более трёх лет назад

Примеры использования:

$v = !empty($viewVariable) ? $viewVariable : 'default'; // and $v = @$viewVariable ? : 'default'; //one pity IDE definition warning

$e = !empty($array['doubtful']['element']) ? $array['doubtful']['element'] : 'default'; // and $e = @$array['doubtful']['element'] ? : 'default';

$p = !empty($object->magicProperty) ? $object->magicProperty : 'default'; // and $p = @$object->magicProperty ? : 'default';
Такие моменты вынужденной проверки предположительно не инициализированных переменных возникают, в основном, из устарелого кода, внешних (не задокументированных) источников данных, магии и прочего отсутствия профессионализма разработчиков.
Например:
Самопальный фреймворк, наследие php4, тяжелые попытки перехода на ООП и MVC парадигму. В контроллерах магические свойства получают данные запроса. Контроллеры передаются аргументом на обработку во всяческие менеджеры, где могут быть изменены их (магические опять же) свойства.
Это очень печальные примеры. И хотелось бы больше никогда не встречать такого на практике, но изначально ошибки уровня «Переменная не инициализирована» просто игнорировались на уровне error_reporting(E_ERROR) и никакая проверка не проводилась. Разработку вести с такой директивой совесть не позволяет. Поэтому пришлось расставлять проверки. Но это выявило лексическую тяжбу в PHP, которую хочется решить наиболее удобным методом.

Answer 1 · 2013-08-13 21:23:14

Неудивительно, что вас смущает использование «собаки», нутро верно подсказывает, что это — умалчивание ошибки и во многих случаях используется как костыль. А isset, empty и is_null — это очевидная логика, причем разная, и более чистый и понятный код. Я бы рекомендовал забыть что «собака» вообще существует, пока нужда остро не прижмет в каком-нибудь стареньком проекте.

Использование тернарного оператора ?: с подавлением ошибки?

Войдите на сайт