Почему базовый класс ловит исключение?

Question

[mafia8]

Почему базовый класс ловит исключение? Ведь он не знает все особенности всех своих производных классов.

<?php ## Наследование исключений.
  // Исключение - ошибка файловых операций.
  class FilesystemException extends Exception
  {
    private $name;
    public function __construct($name)
    { 
      parent::__construct($name);
      $this->name = $name; 
    }
    public function getName() { return $this->name; }
  }
  // Исключение - файл не найден.
  class FileNotFoundException extends FilesystemException {}
  // Исключение - Ошибка записи в файл.
  class FileWriteException extends FilesystemException {}

  try {
    // Генерируем исключение типа FileNotFoundException.
    if (!file_exists("spoon")) 
      throw new FileNotFoundException("spoon");
  } catch (FilesystemException $e) {
    // Ловим ЛЮБОЕ файловое исключение!
    echo "Ошибка при работе с файлом '{$e->getName()}'.<br />";
  } catch (Exception $e) {
    // Ловим все остальные исключения, которые еще не поймали.
    echo "Другое исключение: {$e->getDirName()}.<br />";
  }
?>

Answer 1

[galaxy]

А почему catch (Exception $e) ловит все остальные исключения, вы не задумывались? Ведь он не знает все особенности всех своих производных классов.

Блок catch обрабатывает исключение подходящего типа в порядке объявления этих блоков. Объект класса FileNotFoundException является также и объектом класса FilesystemException (как и объектом класса Exception).

Хотите обрабатывать отдельно FileNotFoundException, вставьте соответствующий catch-блок до catch (FilesystemException $e).

Answer 2

[FanatPHP]

Это очень плохой подход.
Все делается ради красивости, а не для реальной пользы.
FileWriteException может случиться примерно по 100500 разных причин.
НИ ОДНА ИЗ НИХ не будет сообщена несчастному разработчику, который будет пытаться понять, ПОЧЕМУ не получилось файл записать. Ему, как в том анекдоте, будут говорить "слушайте свою песню Валенки!", вместо реального сообщения об ошибке.

Кидать красивости можно на уровне файлового враппера, но при этом обязательно сохранять исходное сообщение об ошибке!

public function write($filename, $data) {
    try {
        file_put_contents($filename, $data);
    } catch (Throwable $e) {
        $e = new FileWriteException($e->getMessage());
        $e->setName($filename);
        throw $e;
    }
}

Comments

[mafia8]

PHP сообщает, почему не получилось записать файл?

[FanatPHP]

в смысле? ты никогда не видел сообщение об ошибке при попытке записать файл? даже такое распространённое как permission denied?

[mafia8]

Сейчас посмотрел и увидел. Через set_error_handler. Через исключения не получилось.

[FanatPHP]

mafia8, ну это понятно. через set_error_handler ошибки можно превратить в исключения и дальше уже ловить.

[mafia8]

FanatPHP, ловить ошибки file_put_contents через ошибки, через исключения не получилось.
И непонятно, зачем создавать новое исключение, если $e - уже исключение. И зачем передавать (бросать) дальше.

[FanatPHP]

mafia8, в каком смысле не получилось? Код по ссылке не работает? Что происходит в случае ошибки?
Новое создавать потому, что $e это инстанс ErrorException, а ты же вроде хотел FilesystemException ?
и снова бросать именно поэтому

Но если тебя устроит ErrorException, то разумеется, ни ловить, ни бросать не надо.
Тогда сделать set_error_handler и просто писать file_put_contents($filename, $data);, без всякой лишней обвязки