Как правильно обрабатывать ошибки в PHP PDO?

Question

[liltrill]

Всем привет!)

Я запутался в элементарных вещах, пытаюсь найти инфу, так больше путаю себя.

Например есть action с регистрацией, перед тем как продолжить регистрацию я проверяю занята ли почта

$emailFlag = User::checkEmailUser($options['email']);
if ($emailFlag > 0) {
     echo json_encode(array('flag' => false, 'text' => 'Ошибка: Почта занята!'));
     return true;
}

public static function checkEmailUser($email)
    {
        // Соединение с БД
        $db = Db::getConnection();

        // Текст запроса к БД
        $sql = "SELECT * FROM user WHERE email = :email";

        // Подготовленный запрос
        $result = $db->prepare($sql);
        $result->bindParam(':email', $email);
        $result->execute();

        // Возврат полученного результата
        return $result->rowCount();
    }

Если возьмем сразу то что SQL запрос верный, то может ли произойти просто сбой в MySql?, и как его обработать? Прочитал про исключения, но как красиво сделать? без try вообще можно?

Еще даже так сформулирую: Как правильно и красиво написать запрос к MySql, чтобы пользователь увидел что с MySql что то не так, типа такого: Ой, MySql не отвечает...

P.S: Использую MVC паттерн

Answer 1

[Антон Р.]

Почитай про try, throw, catch - https://www.php.net/manual/ru/pdo.error-handling.php
То что юзеру не нужно видеть код ошибки это очень правильно.

Comments

[liltrill]

Спасибо, прочитаю!) Еще вопрос - может ли произойти сбой в запросе? С учетом что все верно, и сбой по неведомой причине.

[Антон Р.]

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

[kafkiansky]

liltrill, неведомых причин не бывает. Или вы неправильно настроили соединение, или обратились не к той базе, где нет таких таблиц, или неправильно написали запрос.

[liltrill]

kafkiansky, так вот это я и хотел услышать) Благодарен за ответ!. Если не сложно то можно еще вот на это ответить: Если я уверен что все правильно, все данные обработаны, запросы проверенны имеет ли смысл использовать try/catch? Ведь это излишне, так как ошибок не будет.

[kafkiansky]

liltrill, зависит от вашей уверенности в собственных запросах. Хорошим правилом является логировать потенциально опасные участки кода и перехватывать ошибки, чтобы они не появились на клиенте. Быть уверенным на 100% нельзя, потому что вы можете применить миграцию, которая изменит схему таблицы, а поменять запрос забыть, а без логов вам будет трудно понять, что произошло. Если совсем простой сервис, то я оборачиваю в try..catch только транзакции.

[FanatPHP]

Антон Р., может

Answer 2

[kafkiansky]

Чем, собственно, вам try..catch не нравится?

try {
   $connection->beginTransaction();

   $this->questions->declineQuestion(Question::NEW, $uniqid);
   $this->mentorWorks->failWork($userId, $questionId, $uniqid);

   $connection->commit();
} catch (\PDOException $e) {
   $connection->rollBack();
   $this->logger->warning($e->getMessage);
}

Comments

[liltrill]

В транзакциях использовать try..catch мне нравится.

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

[kafkiansky]

liltrill, а если не получится подключиться к базе? Ну вот вы поменяли пароль или юзернейм базы, а в коде поменять забыли. Ошибки куда? На клиент. Обновили таблицу, а забыли поменять запрос. Ошибки куда? На клиент. Можно быть уверенным в моменте, но не постоянно, т.е. сегодня запрос или соединение работают, а завтра база обновилась, и все.

[liltrill]

kafkiansky, Понял, спасибо за ответы, вы ответили на все мои вопросы!

[Антон Р.]

А что делает rollBack() ? Если ошибка и запрос не выполнился всё равно

[kafkiansky]

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

[Антон Р.]

kafkiansky, понятно. А как реализовать "откат" записи в БД, хотя бы схематично?

[kafkiansky]

Антон Р., в транзакции или как? rollback() откатывает все запросы, если была ошибка.

[liltrill]

kafkiansky, Перегнул, не хотел до такого доходить, дело в не насмешке, человек просто не уподобился прочитать комментарии и понять что тот кусок кода мне был не интересен, и все ответы в комментариях, но тут я не прав, отрицать этого не буду.

[FanatPHP]

Тем, что вопрос был не про транзакции.

Answer 3

[FanatPHP]

Стоило на неделю лечь в больницу, как изо всех щелей поналезли гамадрилы

Первый, традиционно для тостера, рекомендует читать про то, в чем сам не смыслит ни бельмеса.

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

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

Как правильно и красиво написать запрос к MySql, чтобы пользователь увидел что с MySql что то не так, типа такого: Ой, MySql не отвечает

Никак.

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

Обработку ошибок надо делать унифицированно и осуществлять в одном месте. А не размазывать, как манную кашу по тарелке, по всему коду.

Берём функцию, set_exception_handler(), и пишем в нее один раз все что хотим сказать юзеру в случае ошибки. Любой ошибки, а не только соединения с базой данных.

Примечание: юзеру глубоко фиолетово, MySQL не отвечает, или космическая станция на Марсе. Он, скорее всего, даже такого слова не знает. И грузить его подробностями не нужно.

Comments

[liltrill]

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

Я во многих вопросах нахожу ваши ответы, вы меня уже не раз спасали).

P.S: Я не хотел использовать try..catch, так как запросов много и пришлось бы писать во многих моделях/контроллерах обработку исключения, что раздуло бы модели или контроллеры. Я конечно думал написать функцию которая проверяла было ли исключение в запросе, но не представляю как она будет работать, и будет ли вообще, ну в любом случае ее нужно было бы вызывать, что тоже не хотелось.

[FanatPHP]

В принципе, вместо этой функции можно написать один большой try catch, буквально перед первой строчкой кода и после последней. Это возможно, если на сайте используется единая точка входа.

Но решение с функцией кажется мне более красивым и гибким.

А писать для каждого запроса - это, конечно, не вариант совсем.

Есть только одно исключение, при соединении. Почитайте здесь: https://phpdelusions.net/pdo_examples/connect_to_mysql

Если будет непонятно, я объясню

Answer 4

[Nujabes37]

https://github.com/Umarbatalov/http-service/blob/m... Если интересно можешь глянуть как сделал я, хотя я не говорю, что это правильно) Только вместо кода ошибки, возвращай, что-нибудь другое, эт я для себя делал)

Comments

[Антон Р.]

Ссылка не работает, 404

[liltrill]

Доступ закрыт :(

[Nujabes37]

liltrill, теперь?

[liltrill]

Nujabes37, открылось :)