Можно ли таким образом использовать try-catch?

Question

[Architecktor133]

К примеру у меня есть такой код:

function foo() {

    # Начало транзакции

    try {
        # Одно обращение к БД
    } catch(PDOException $e) {
        throw new PDOException("Уточняющее сообщение: " . $e->getMessage());
    }

    try {
        # Другое обращение к БД
    } catch(PDOException $e) {
        throw new PDOException("Другое уточняющее сообщение: " . $e->getMessage());
    }

    # Коммит транзакции

}

Насколько такая конструкция легальна, и используется ли такое вообще?

Comments

[Алексей Скляров]

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

Answer 1

[Алексей Уколов]

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

Answer 2

[FanatPHP]

Можно, используется. Но в 99% случаев не нужна.
В данном случае скорее всего тоже. Ожидать исключения в транзакции - плохая практика. Скорее всего, либо никакие уточнения не нужны, либо данные не были провалидированы до начала транзакции.

Comments

[Станислав Б]

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

[FanatPHP]

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