Правильно ли я понял централизованную обработку исключений в PHP?

Question

[Алексей]

Всем привет!

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

Код ниже помещаем в файл типа error-handler.php и инклудим на все необходимые страницы.

// Файл error-handler.php
// Централизованная обработка исключений

ini_set("log_errors", 1);
ini_set("error_log", __DIR__ . "/php-errors.log");

// Уровень ошибок display_errors = 1 используется во время разработки,
// на рабочем сервере установить на 0
ini_set('display_errors', 1);
ini_set('display_startup_errors', 1);
error_reporting(E_ALL & ~E_NOTICE);

// Обрабатываем все исключения, возникающие во время работы приложения
function customExceptionHandler($e)
{
    error_log($e);
    http_response_code(500);
    // Если в режиме отладки, то показываем исключения
    if (ini_get('display_errors')) {
        echo $e;
    } else {
        echo "<h1>500 Ошибка сервера.</h1>
              Произошла ошибка на стороне сервера.<br>
              Пожалуйста, попробуйте еще раз позже.";
    }

}

set_exception_handler('customExceptionHandler');

Т.е. в customExceptionHandler мы ловим все возможные исключения, и предпринимаем соответствующие действия.

Иногда возникают ошибки (Error), а не исключения, тогда предлагают добавить такой код, стоит ли?

set_error_handler(function ($level, $message, $file = '', $line = 0)
{
    throw new ErrorException($message, 0, $level, $file, $line);
});

Т.е. ошибку меняем на исключение, и уже обрабатываем опять же в нашем customExceptionHandler.

Буду благодарен за любые комменты )

Comments

[Дмитрий]

Добрый вечер.
https://github.com/codedokode/pasta/blob/master/ph...

[FanatPHP]

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

[Алексей]

Дмитрий, спасибо читаю

[Дмитрий]

Алексей, пожалуйста.

[FanatPHP]

Дмитрий, ближе к концу прямо неплохая статья, но в ней нет акцента имено на централизованную обработку ошибок.
Не написано большими красными буквами, что все те телодвижения, которые мы совершали на протяжении статьи, все эти "if (!file_exists($file)) {" или "die("Ошибочка:" становятся не нужны.

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

[HellWalk]

Когда разбирался с аналогичной темой - смотрел реализацию ErrorHandler в Симфони:
https://github.com/symfony/error-handler

[FanatPHP]

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

[HellWalk]

FanatPHP,
Чтобы разобраться с какой-то темой хорошо - нужно посмотреть несколько реализаций, сравнить их плюсы/минусы, а потом еще свой велосипед написать (идеальный для конкретного программиста).

Но для простого ознакомления, конечно, хватит и примера из данного вопроса. Там по сути одну функцию нужно знать и все (set_exception_handler). Все остальное - тонкости и детали.

Answer 1

[FanatPHP]

В целом правильно, неплохая проработка материала.
У меня есть только пара замечаний, не относящихся напрямую к централизованному обработчику.

• error_reporting(E_ALL & ~E_NOTICE); делать не стоит. Если только не приходится работать с адовым легаси, которое сыпет нотисами, лучше отлавливать все ошибки. Учитывая же что в 8-ке обращение к несуществующей переменной станет не нотисом а предупреждением, такая конструкция со временем станет бессмысленной. что означает - ошибки надо исправлять, а не замалчивать.
  
• ini_set("error_log", __DIR__ . "/php-errors.log"); будет не очень хорошей идеей, если файл error-handler.php выше корня веб-сервера. Ошибки надо прятать подальше.
  
• ini_set('display_startup_errors', 1); - это какая-то дичь, которая кочует из руководства в руководство. Никто никогда этих стартап еррорс не видел, но многие старательно пишут это заклинание у себя в коде. Это по-любому связано с настройкой сервера, и в отладке ошибок поможет примерно ничем.
  
• само по себе задание настроек через ini-set ненадежно. Ошибка может случиться до того, как РНР прочитает эту команду. Задавать надо в конфигурации веб-сервера.
  
• в теории можно добавить флаг или автоматическую проверку на джейсон запрос. и соответственно кодировать ответ в джейсон. Но это только для криовруких фронтендеров, которые не умеют читать НТТР статусы, а ждут что им все разжуют в джейсоне, и без error: true они не поймут, что была ошибка
  
• стек вызовов может быть довольно длинным, и раздувать логи. Можно подумать о более укороченном варианте.
  

Вообще всё зависит от задач. Например все современные фреймворки в режиме разработки выдают развесистую страницу с отчетом об ошибке, которая включает в себя и кусок кода вокруг строи, на которой произошла ошибка.
Но как именно базовый обработчик, необходимый минимум - вполне годно.

Я только не понял, почему вопрос про error_handler. Что именно смущает?

Comments

[Алексей]

Спасибо за емкий ответ!

Я только не понял, почему вопрос про error_handler. Что именно смущает?

Опыта мало, не совсем был уверен что это правильный подход инклудить так на всех страницах файл.. так нормально делать?

[Алексей]

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

[FanatPHP]

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

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

[Алексей]

FanatPHP, спасибо, вникаю потихоньку

[TostPOST]

Вообще иногда startup erorrs появляются, но чаще всего это ошибка того кто настраивал, так как в 99% случаев ошибки из-за расширений. Например вот недавно пытался поставить swoole на php8.0rc1. Из-за того, что pecl как то неправильно работал (некоторые части из php8.0-dev были взяты из 7.4 версии). Так вот, тогда вылетела та самая startup error. Но впринципе добавлять обработчик необязательно, так как можно заранее проверить тем же php -v.