Почему и как Вы использовали бы Исключения в этом демонстрационном коде PHP?

Я не программист PHP, но это похоже на C #.

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

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

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

Я бы сказал, что это хорошая практика проектирования, чтобы избежать перехвата всех исключений «catch (Exception $ e)» и разработки вместо этого по контракту, как вы, кажется, делаете в предыдущем примере. Сначала я хотел бы более конкретно указать тип создаваемого исключения, а затем при необходимости работать оттуда. В противном случае держитесь подальше от try-> catch и исключений.

Ни в коем случае нельзя предполагать, что файл существует только потому, что вы вызвали file_exists () ! Функция file_exists не открывает файл, поэтому файл потенциально можно удалить, переименовать или переместить в любое время!

class FileOutputter
{
    public function outputFile($file)
    {
        if (!file_exists($file))
            return false;
        ///<--- file may be deleted right here without you knowing it
        return file_get_contents($file);//<-- then this will throw an error
            //if you don't catch it, you're program may halt.
    }
}

Я считаю, что так лучше:

class FileOutputter
{
    public function outputFile($file)
    {
        try{
            if (!file_exists($file))
                return false;
            return file_get_contents($file);
        }catch(Exception e){
            check_what_went_wrong_and_go_to_plan_B();
        }
    }
}

( Изменить ]: И, вероятно, даже лучше попытаться открыть файл с самого начала. Если вам это удастся, у вас есть «блокировка» файла, и он не исчезнет просто так. Если нет, то перехватите исключение и посмотрите, что произошло. неправильно.

С другой стороны, вы можете подумать, что такой уровень избыточности просто глуп. В этом случае я не думаю, что вам вообще нужно беспокоиться о try / catch :)

Просто примечание, ваш код для средства вывода файлов недействителен, поскольку file_get_contents ($ file) не вызывает исключения. Однако это вызовет предупреждение, если файл не существует или недоступен по какой-либо причине. Кроме того, вы возвращаете исключение из outputFile, когда вам, вероятно, следует просто позволить ошибке распространиться вверх по стеку вызовов.

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

function exception_error_handler($errno, $errstr, $errfile, $errline ) {
    throw new ErrorException($errstr, 0, $errno, $errfile, $errline);
}
set_error_handler("exception_error_handler");

That Таким образом, любые стандартные вызовы функций, вызывающие ошибку, вызовут исключение.

Итак, я бы изменил FileOutputter на это (с добавленным выше фрагментом):

class FileOutputter
{
 public function outputFile($file)
 {
  if (!file_exists($file))
   throw new Exception("File not found.",123);

   return file_get_contents($file);  
 }
}

Тогда вызывающий код в основном тот же.

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