Обращение к несуществующему элементу массива. Правильно VS Лаконично?

Question

[Данила Риджи]

Допустим есть массив-конфиг php приложения. Есть настройки debug:

'debug' => array(
        'enabled' => true,
        'level' =>  -1,
    ),

Обрабатывая опцию enabled, я могу написать так:

// Вариант 1 - Длинный и наверное правильный
if(array_key_exists('enabled',$debug)) {
    $isDisplayErrors = $debug['enabled'];
} else {
    $isDisplayErrors = false; // default
}
ini_set('display_errors', $isDisplayErrors);

//Вариант 2 - Короче и наверное правильный
ini_set('display_errors', ((array_key_exists('enabled',$debug)) ? $debug['enabled'] : false);

//Вариант 3 
ini_set('display_errors', $debug['enabled'] === true);

Вариант номер 3, который мне очень нравится лаконичностью, вызовет notice, если ключ будет отсутствовать. Концептуально, что еще плохого в данном подходе, помимо засорения вывода нотисами в дебаг режиме ?

Answer 1

[Сергей Протько]

Правильный вариант:

$isDisplayErrors = isset($debug['enabled']) && $debug['enabled'];
// или более понятно
$isDisplayErrors = isset($debug['enabled']) ? 
                                    $debug['enabled'] : false;

Comments

[Данила Риджи]

Я правильно понимаю, что плодить notice неважно по каким причинам это плохая практика ?!

[Данила Риджи]

Ваш первый вариант, кстати, весьма симпатичен, возьму на заметку )

[Алекс]

Относительно.

[Сергей Протько]

плодить notice - плохая практика, но опять же не критично.

Answer 2

[VeMax]

А вариант с isset($array['element']) ?

Comments

[Данила Риджи]

Можно поподробрней ?! Если вы просто про то, как проверить ключ на существование, то вопрос не в этом

[VeMax]

Ниже уже написали, про вариант с isset. Я о том же.

Answer 3

[_ _]

Я всегда пользовался @$array['key']
Если нужно в результате именно boolean, то !!@$array['key']

Answer 4

[Алекс]

В конфиге приложения $isDisplayErrors = array(); - то есть объявите переменную!

Answer 5

[Алексей Рехов]

Воспользуйтесь функцией empty():

ini_set('display_errors', !empty($debug['enabled']));

Comments

[Данила Риджи]

Отличный вариант, спасибо :)))

[Данила Риджи]

Я правильно понимаю что у него очень ограниченный контекст использования: только для установки булевых значений и с использованием массивов исключительно с одним уровнем вложенности ?!

[Сергей Протько]

нет. empty проверяет пустая переменная или нет. Пустой она считается, если там false, 0, пустая строка, пустой массив...

[Данила Риджи]

empty($debug['options']['enabled']) вернет false, если ключа enabled нет, но есть options

Answer 6

[zloyusr]

Вариант от Никиты Попова одного из core разработчиков php:
($_=& $debug['enable']) ?: false;