PHP @ вместо isset для проверки значения $_GET

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

Основная проблема, связанная с использованием оператора @, заключается в том, что он, вероятно, станет соглашением в вашем коде, поэтому, хотя ваш пример может показаться безобидным, позже вы или ваша команда можете обнаружить, что вы или ваша команда используете:

if( @IsAvailable() ) {

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

Подумайте, насколько сильно вы можете замедлить работу вашего приложения, когда ваш веб-сайт/приложение начнет получать десятки/сотни тысяч (или более) запросов в день. Если вы стандартно подавляете ошибки, у вас, вероятно, есть десятки на каждый запрос - вдруг ваш сайт заметно медленнее, чем вам хотелось бы.

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

Если вы не принимаете проблему производительности в качестве аргумента, то это действительно нормально. Но это не обязательно должно быть во всех случаях, потому что @ подавит все возможные ошибки, даже те, о которых вы не подумали. Но в этом случае кажется, что другие ошибки, кроме той, которую вы хотите подавить, невозможны.

Я согласен с вами, что использовать isset() перед чтением значения очень некрасиво, и я тоже не люблю его писать. Но вставка @ перед оператором кажется мне уродливой. Это может снизить читаемость более длинного кода.

Хорошая новость заключается в том, что, начиная с PHP 7, мы можем использовать гораздо более приятный способ, оператор нулевого объединения ??, который работает следующим образом:

if($_GET['key'] ?? '' === '123' ) {}

Это в основном замена для этого:

$result = isset($value) ? $value : $anotherValue;

теперь вы можете использовать

$result = $value ?? $anotherValue;