Каковы лучшие практики для ловли и повторного исключения?

Должны ли перехваченные исключения перебрасываться напрямую, или они должны быть обернуты вокруг нового исключения?

То есть я должен сделать это:

try {
  $connect = new CONNECT($db, $user, $password, $driver, $host);
} catch (Exception $e) {
  throw $e;
}

или это:

try {
  $connect = new CONNECT($db, $user, $password, $driver, $host);
} catch (Exception $e) {
  throw new Exception("Exception Message", 1, $e);
}

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

Вы не должны ловить исключение, если не собираетесь делать что-то значимое .

«Что-то значимое» может быть одним из следующих:

Обработка исключения

Наиболее очевидным значимым действием является обработка исключения, например, путем отображения сообщения об ошибке и отмены операции:

try {
    $connect = new CONNECT($db, $user, $password, $driver, $host);
}
catch (Exception $e) {
    echo "Error while connecting to database!";
    die;
}

Ведение журнала или частичная очистка

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

try {
    $connect = new CONNECT($db, $user, $password, $driver, $host);
}
catch (Exception $e) {
    logException($e); // does something
    throw $e;
}

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

$connect = new CONNECT($db, $user, $password, $driver, $host);
try {
    $connect->insertSomeRecord();
}
catch (Exception $e) {
    $connect->disconnect(); // we don't want to keep the connection open anymore
    throw $e; // but we also don't know how to respond to the failure
}

В PHP 5.5 введено finallyключевое слово, поэтому для сценариев очистки теперь есть другой способ приблизиться к этому. Если код очистки должен выполняться независимо от того, что произошло (т. Е. Как при ошибке, так и при успехе), теперь это можно сделать, прозрачно разрешая распространяться любым сгенерированным исключениям:

$connect = new CONNECT($db, $user, $password, $driver, $host);
try {
    $connect->insertSomeRecord();
}
finally {
    $connect->disconnect(); // no matter what
}

Ошибка абстракции (за исключением цепочки)

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

class ComponentInitException extends Exception {
    // public constructors etc as in Exception
}

class Component {
    public function __construct() {
        try {
            $connect = new CONNECT($db, $user, $password, $driver, $host);
        }
        catch (Exception $e) {
            throw new ComponentInitException($e->getMessage(), $e->getCode(), $e);
        }
    }
}

В этом случае вы не хотите, чтобы пользователи Componentзнали, что это реализовано с использованием соединения с базой данных (возможно, вы хотите оставить свои параметры открытыми и использовать файловое хранилище в будущем). Таким образом, в вашей спецификации Componentбудет сказано, что «в случае сбоя инициализации ComponentInitExceptionбудет выброшено». Это позволяет потребителям Componentловить исключения ожидаемого типа, а также позволяет отладочному коду получать доступ ко всем (зависящим от реализации) деталям .

Обеспечение более богатого контекста (за исключением цепочки)

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

class FileOperation {
    public static function copyFiles() {
        try {
            $copier = new FileCopier(); // the constructor may throw

            // this may throw if the files do no not exist
            $copier->ensureSourceFilesExist();

            // this may throw if the directory cannot be created
            $copier->createTargetDirectory();

            // this may throw if copying a file fails
            $copier->performCopy();
        }
        catch (Exception $e) {
            throw new Exception("Could not perform copy operation.", 0, $e);
        }
    }
}

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

Значение этого показано на примере сценария, в котором, например, создание UserProfileобъекта приводит к копированию файлов, поскольку профиль пользователя хранится в файлах и поддерживает семантику транзакций: вы можете «отменить» изменения, потому что они выполняются только на Копия профиля, пока вы не совершите.

В этом случае, если вы сделали

try {
    $profile = UserProfile::getInstance();
}

и в результате возникнет ошибка исключения «Целевой каталог не может быть создан», вы будете иметь право на путаницу. Обтекание этого «основного» исключения слоями других исключений, которые предоставляют контекст, значительно упростит обработку ошибки («Ошибка создания копии профиля» -> «Ошибка операции копирования файла» -> «Не удалось создать целевой каталог»).

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

Допустим, вы строите модель. Предполагается, что модель абстрагирует всю устойчивость данных и проверку от остальной части приложения. Итак, что же происходит, когда вы получаете ошибку базы данных? Если вы отбрасываете, вы пропускаете DatabaseQueryExceptionабстракцию. Чтобы понять почему, подумайте об абстракции на секунду. Вам не важно, как модель хранит данные, только то, что она делает. Точно так же вас не волнует, что именно пошло не так в базовых системах модели, просто вы знаете, что что-то пошло не так, и примерно то, что пошло не так.

Таким образом, перебрасывая исключение DatabaseQueryException, вы пропускаете абстракцию и требуете от вызывающего кода понять семантику того, что происходит в модели. Вместо этого создайте универсальное ModelStorageExceptionи оберните пойманное DatabaseQueryExceptionвнутри этого. Таким образом, ваш вызывающий код все еще может попытаться обработать ошибку семантически, но это не имеет значения для базовой технологии Модели, поскольку вы только выявляете ошибки из этого уровня абстракции. Более того, поскольку вы обернули исключение, если оно вспыхнуло полностью и требует регистрации, вы можете проследить до брошенного корневого исключения (пройтись по цепочке), чтобы у вас все еще была вся необходимая отладочная информация!

Не просто поймайте и выбросьте одно и то же исключение, если вам не нужно выполнить некоторую постобработку. Но такой блок, как } catch (Exception $e) { throw $e; }бессмысленно. Но вы можете переписать исключения для некоторого значительного увеличения абстракции.

ИМХО, ловить исключение, чтобы просто отбросить его, бесполезно . В этом случае просто не перехватывайте его, и пусть ранее вызванные методы обрабатывают его (иначе говоря , «верхние» в стеке вызовов) .

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

Способ , чтобы добавить некоторую информацию, чтобы расширить Exceptionкласс, чтобы иметь исключения , как NullParameterException, DatabaseExceptionи т.д. Более того , это позволит developper только поймать некоторые исключения , которые он может обрабатывать. Например, можно только поймать DatabaseExceptionи попытаться выяснить, что вызвало Exception, например, переподключение к базе данных.

Вы должны взглянуть на Exception Best Practices в PHP 5.3.

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

http://ralphschindler.com/2010/09/15/exception-best-practices-in-php-5-3

Вы обычно думаете об этом таким образом.

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

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