Нужна ли проверка после сохранения модели? Или это паранойя?

Question

[chelkaz]

Скажите, нужна ли проверка после сохранения модели? А то у меня непонятная фобия, я начинаю проверять все что можно, например после сохранения (при учете что все данные вошли) делаю так:

$save = $section->save();
        if($save)
        {
            // Сохранилось и нужные сообщения
        }
// тут на всякий сообщения об ошибки...

Comments

[Максим Тимофеев]

а в каком месте это yii2?

Answer 1

[Andrzej Wielski]

Это в корне неверно.
Перехватывать ошибки стоит через try...catch.

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

Comments

[chelkaz]

Почему? В данном случаи ведь пользователь как раз увидит сообщение из условия если $save вернет фолс.

[Tesla]

chelkaz: А если $section->save() бросит Exception, например, из-за проблемы с БД, у вас прервется выполнение кода, и никакого сообщения об ошибке не будет.

[chelkaz]

Ну тогда прихожу к выводу, что нужно слушать через события модели... Через метод уже updated после метода updating... Я кстате такими событиями и произвожу очистку после обновления или удаления модели и старых фото например. Например после удалении каталога, в котором много категорий и элементов и фото и прочего...

[Andrzej Wielski]

chelkaz: при чем тут события?
Если вы выполните save либо update (да что угодно) в момент, когда ляжет база данных, либо появится какая-то другая проблема - всплывет Exception, и никакого сообщения об ошибке не будет.

Та же функция save (update, delete, продолжить нужными) не вернет false / null.

[Дмитрий Евграфович]

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

Answer 2

[Дмитрий Евграфович]

насколько я помню, save в случае ошибки кидает исключение. Andrzej Wielski правильно написал, только единственное, что хочу добавить - почитайте документацию ларавела, там есть раздел про обработку исключений. если кратко - не стоит после каждого save писать обработку, достаточно сделать в одном файле и всего лишь один раз.