Это плохое программирование, чтобы просто бросать исключения, а не в блок catch try?

Question

Я сейчас работаю в PHP. Я работаю над системой ошибок для моей CMS, которую я создаю (для удовольствия). Для фатальных ошибок в моей системе (не в компиляторе php) я создал класс FatalException, который расширяет встроенный класс Exception. Так как эти типы ошибок, во всяком случае, останавливают систему, я выбрал выход в __construct.

class FatalException extends Exception{ 
     public function __construct($message) { 
      exit("<h1 style='color:red;' >FATAL ERROR: $message </h1>"); 
     } 
    } 

Так что в моем коде, я кое-что проверить, как подключение к базе данных, и если он не может, то я буду просто бросить FatalException («Не удается подключиться к базе данных: $ database_error_message»). Он не будет в блоке try/catch.

Когда я запускаю код и не могу подключиться к базе данных, например, все, что я вижу на экране, - это предложение с большими красными буквами. Так что все работает отлично, но это плохая практика/кодирование?

EDIT:

На самом деле на самом деле это не начать этот путь. Сначала я регистрировал ошибку, а затем выходил из области catch, но потом я подумал, что если все фатальные ошибки выйдут в любом случае, тогда просто добавьте конструктор in. Тогда я заметил, что на самом деле он не попал в зону улова, которую он покидал. Так положить заявление в Try/поймать блок был видом спорного вопроса. Это приводит к вопросу.

Answer 1

Даже вы обертываете exit() в функцию-член исключения, вы здесь не используете исключение для обработки ошибок, но только exit() - исключение никогда не бросается, PHP останавливается до этого.

Таким образом, это работает отлично, но это плохая практика/кодирование?

Да, это плохая практика. Он также работает просто отлично, если бы вы создали себе функцию:

function error_exit($message) { 
    exit("<h1 style='color:red;' >FATAL ERROR: $message </h1>"); 
} 

Вместо думать о том, хотите ли вы использовать Excpetions или вы хотите exit().

Answer 2

Это плохая идея imho. Для правильной работы необходимо убрать исключение. Однако вы можете создать такой метод, как -> error ($ index); который прикрепляется к каждому объекту, который вы делаете. Оттуда вы можете перенаправить ошибку в конкретный класс с блоком try/catch, чтобы правильно обрабатывать ошибки.

class TestClass 
{ 
    public function error($index) 
    { 
     try 
     { 
     // Convert index to exception and throw it. 
     } 
     catch (Exception $e) 
     { 
     // Handle the error 
     } 
    } 
} 
$a = new TestClass(); 
$a->error(1000); 

Обратите внимание, что это не работы за исключением, что PHP броски, вы должны поймать их по отдельности или работать с исполнительными концентраторов.

Answer 3

Если вы идете в exit() безоговорочно в конструкторе, на самом деле нет смысла указывать конструктор, не говоря уже о создании класса Exception. Вы могли бы проще (и честно) иметь статическую функцию, называемую Fatal::Die($message).

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

Что делать, если конкретная страница вашего сайта может эффективно справиться без подключения к базе данных (просто отсутствует «последние новости» или что-то в этом роде)? Затем он мог бы catch(Database_Exception $e) и продолжить, пока остальная часть вашего сайта просто попадает прямо в сообщение «oh no something wrong wrong».

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