Как правильно обрабатывать ошибки и исключения?

Question

[Max Ba]

Парни, верно ли я сделал простой скрипт по выводу ошибки? Получается всегда теперь оборачивать код в try/catch если метод, помимо bool, может вернуть и текст?

class Test{
  
  public function create(array $data){
    
    if(empty($data)){
      return false;
    }
    
    if(empty($data['message'])){
      throw new Exception('Сообщение не может быть пустым');
    }
    
    //save...
	
    return true;
  }
  
}

$test = new Test();

try{
  $result = $test->create(array('message' => 'v'));
  var_dump($result);
}catch(Exception $e){
  echo $e->getMessage();
}

Comments

[FanatPHP]

а в чем смысл проверки if(empty($data))?
И еще вопрос, зачем методу возвращать что-либо? Зачем возвращать булево значение? Зачем текст?

[Max Ba]

FanatPHP, вопрос скорее про исключение, чем кусок кода, который я не удалил.

Касаемо проверки на текст, метод должен получить массив с парой обязательных полей. Если какое-то поле пустое, то метод должен вернуть FALSE и сообщение пользователю.

[FanatPHP]

ну так если проверка if(empty($data)) не нужна, то и возвращать FALSE нет смысла :)
Текст передается в искючении. Полчается, что возвращать вообще ничего не нужно

[Max Ba]

FanatPHP, согласен, не сразу вник) спасибо.

Answer 1

[FanatPHP]

Если я правильно понял функционал

class AppException extends Exception {}

class Test{
  public function create(array $data){
    if(empty($data['message'])){
      throw new AppException('Сообщение не может быть пустым');
    }
  }
}

$test = new Test();

try{
  $test->create(array('message' => 'v'));
  // redirect
}catch(AppException $e){
  echo $e->getMessage();
}

Идея в том, что при использовании исключений, никаких ручных проверок делать не надо. Если исключения не было - значит всё нормально. А если было, то код попросту не будет исполнен.

Comments

[Max Ba]

Да уж, непривычно как-то без true и false... Открытие прям сделал для себя с этим try/catch)
:)

Answer 2

[Кирилл Несмеянов]

Стоит разделять публичные и приватные исключения. Пользователям не стоит видеть внутреннюю кухню. Так что:

class HttpExcepton extends RuntimeException {}
class ExampleHttpException extends HttpExcepton {}

//

throw new ExampleHttpException('Сообщение не может быть пустым', 422);

Так что всё это дело можно свести к одному единственному try/catch, который будет выводить текст из всех HttpExcepton (и статус код, например, в данном примере 422), и выводить сообщения "Internal Server Error" от любых других.

Comments

[Max Ba]

А плохая практика не использовать bool, а например int 1 в случае успеха и текст в случае ошибки? Грубо говоря:

$result = $test->create();
if($result == 1){
  //redirect....
}else{
  return $result;
}

[Max Ba]

а например int 1

или также текст 'successful'

[Max Ba]

И кстати ошибка это не для разработчика, а для пользователя.

[Кирилл Несмеянов]

А плохая практика не использовать bool, а например int 1 в случае успеха и текст в случае ошибки? Грубо говоря:

А если использовать исключения, то весь код сведётся к:

$result = $test->create();

Без всяких if/else и прочего. Всё. Никаких проверок, больше не надо делать.

В случае ошибки программа прервётся и твой обработчик (один единственный try/catch в программе, прямо внутри самого первого файла, грубо говоря в index.php) в самом верху получит информацию о том, что что-то пошло не так. И даже по типу исключения поймёт, является ли это ошибкой внутренней или её можно смело показать пользователю.

Хотя для валидации обычно применяют не исключения, а отдельные контейнерные структуры, которые связывают ключ с набором ошибок (ну т.е. название поля, например, с набором ошибок "должно быть не пустым" и "должно быть длиннее 2х символов").

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

Так что логика пользовательской валидации может выглядеть так (только пример):

$validator = new Validator($_POST, [
  'message' => [Validator/NotEmpty::class, Validator/Exists::class]
]);

if (! $validator->valid()) {
  throw new ValidationException($validator->getErrorMessages());
}