save/delete exceptions about dja_orm HOT 3 OPEN

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

Быстрее и проще выполнить

if (false === $model->save()) {
    throw new \Exception('Error while save');
}

Нежели использовать try / catch

from dja_orm.

@max107 ну зато в catch у тебя сразу есть причина и даже разные причины, разных уровней.
а при save: bool тебе надо куда-то записывать ошибки и потом доставать + там могут быть эксепшены уровнем ниже, которые все равно нужно обработать.

я ваще для себя вывел то что bool дожны возвращать методы отвечающие на вопрос (да\нет?).
hasSmth, isSmth, canSmth и тд.
save - это действие, а не вопрос.

from dja_orm.

И именно действие должно возвращать статус: да или нет. Если тебе нужно отловить ошибки, используй что то вроде Sentry и его аналоги.

Логика простая:

bool
ты проверяешь если запись не сохранена, то бросаешь исключение. Sentry отлавливает.

exception
ты проверяешь не бросилось ли исключение, чтобы узнать что за исключение. Передаешь в sentry / обрабатываешь сам.

Разницы никакой, но куча оберток в виде try catch обеспечена.

from dja_orm.