Как правильно хэндлить Exception?

Question

[Twitt]

Вижу в коде проекта есть:

if (!empty($temporary)) {
                $this->moveToDatabase($temporary);
            }

И этот метод как раз подсвечивается, что unhandled exception. Внутри этого метода кидается эксепшн, если не получается вставить данные в бд. Как правильнее тут схэндлить эксепшн? Обернуть в try catch, где просто нужно логать эту ошибку? А если мне не надо, чтобы программа продолжалась дальше, каким будет правильным решением? Я пришел к примерно такому решению:

if (!empty($temporary)) {
                try {
                $this->moveToDatabase($temporary);
                } catch (\Exception $e) {
                     $this->logger->log("Y-m-d H:i:s | " . $e->getMessage());
                     throw new Exception();
                   }
            }

Т.е. залогировать и кинуть эксепшн по новой. На сколько я правильно делаю? Как обычно стоит хэндлить эксепшены? Внутри catch обычно происходит логирование просто?

Comments

[kafkiansky]

И кто увидит это исключение выше? Судя по всему, это ваша самая верхняя точка, поэтому кидать исключение дальше не нужно. Ну или хотя бы кидать другое исключение с человекопонятной ошибкой, если это все же не верхняя точка приложения. А вообще, ваш код превращается в лапшу из try...catch. Вы либо ловите и подавляйте, либо не ловите и оставляйте до момента, когда ответ отправится пользователю, где уже словите и залогируете.

Answer 1

[Maxim Colesnic]

if (!empty($temporary)) {
    try {
        $this->moveToDatabase($temporary);
    } catch (\Exception $e) {
        $this->logger->log("Y-m-d H:i:s | " . $e->getMessage());
        throw $e;
    }
}

Answer 2

[Антон Шаманов]

как вариант запись в логе сделать в базовом исключении.