Есть ли способ улучшить данный код?

Question

[romazhan]

Если просто написать $result = $request[$key];, то будет Warning, который означает отсутствие $key в $request. Неужели нужно обязательно писать именно так?:

if(array_key_exists($key, $request)) {
    $result = $request[$key];
}

Answer 1

[FanatPHP]

можно, но для этого надо знать, откуда берется $request

для всех внутренних переменных, которые задаются в коде, ответом будет "сделать так, чтобы переменные всегда содержали все нужные значения".
для переменных, которые приходят извне, есть варианты

плюс, конкретно этот код совершенно бессмысленный, он переносит проблему на ступеньку ниже
и теперь уже при обращении к $result будет та же самая ошибка.

Comments

[romazhan]

В более старых версиях php можно было писать $request[$key] и не имело значение есть $key в $request или нет :c

[FanatPHP]

нет, нельзя было
можно было только заткнув РНР рот, чтобы он не сообщал об ошибках

[romazhan]

FanatPHP, ой, тогда вопрос не о чем. Как удалить?!

[FanatPHP]

удалять не надо, надо ответить на вопрос откуда берется $request
и откуда берется $key заодно
после этого можно будет узнать как сделать проще/правильнее

[romazhan]

FanatPHP, $request приходит из вне, но может прийти и без $key

[FanatPHP]

если приходит извне, то можно написать как в соседнем ответе.

[romazhan]

FanatPHP, спасибо, чёт я тупанул, думал если так написать, то тот же варнинг будет, спасибо.

[FanatPHP]

$var ?? значение - это сокращенная запись isset($var) ? $var : значения

Answer 2

[Dark_Dante]

Я бы написал это так
$result = $request[$key] ?? null;

Comments

[FanatPHP]

вот поэтому у РНР всегда будет его говённая репутация
потому что все ламеры и говнокодеры стараются сделать пошустрее, лишь бы ошибок не писало