Как вернуть код ошибки из метода?

Question

[Дмитрий Б]

Добрый вечер. Подскажите пожалуйста как лучше возвращать код ошибки из метода.
К примеру есть функция:

/**
     * @return User|bool
     */
    public function register()
    {
      //  ...
        if (empty($userData)) {
            $errorCode = $this::USER_DATA_EMPTY_ERROR;
        } else if ($this->authenticate() === true) {
            $errorCode = $this::USER_REGISTERED_ERROR;
        } else {
            return $registeredUser;
        }
    }

Варианты:

1. Возвращать true или код ошибки, но в будущем может вызвать серьёзные проблемы.Допустим если мы забыли написать строгое сравнение ===
   
2. Выкидывать исключение. Это скорее относится к фатальным ошибкам, поэтому не подходит
   
3. Запись кода ошибки в свойство объекта, в котором находится метод. Хороший вариант, но не хочется засорять класс
   

Как делаете вы?

Answer 1

[FanatPHP]

Если исключение ловить, то оно не относится к фатальным ошибкам.

Comments

[Дмитрий Б]

Спасибо! Этот вариант мне всё таки больше нравится, тем более, что он уже встроен в PHP

Answer 2

[Сергей Семенко]

Третий вариант самый оптимальный (для меня), таким образом можно получить все ошибки сразу или по отдельности. А ещё лучше записывать не коды ошибок, а сами ошибки.

Comments

[Дмитрий Б]

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

[Сергей Семенко]

Дмитрий Байбухтин: ну разве что так :)

[Igor Kalashnikov]

@abler Вы имели ввиду строку, типа "object not exists" ?
Или ошибку как объект? Если как объект то выглядеть это будет типа :

var error = {
    errorMessage :  'User not logged in',
    errorCode :  'UE001', 
}

Такое конечно лучше отдавать, чем просто коды, но выше правильно сказали про эксепшены - самое правильное решение.

[Дмитрий Б]

Ihor Kalashnikov: почитал об исключениях, много где пишут, что их не рекомендуется использовать для обработки обычных ошибок.

[Igor Kalashnikov]

Дмитрий Байбухтин: давайте ссылки, пообсуждаем.

[Дмитрий Б]

Ihor Kalashnikov: например комментарии к этой статье habrahabr.ru/post/100137

[Igor Kalashnikov]

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

Метод isUserAvailable() может возвращать true/false и заменять false на исключение - глупо, но можно пробросить "DB is not available" и вы должны такое поймать и обработать, или пробросить еще выше, пока не будет логического места, где обработчик будет уместен.

[Дмитрий Б]

Ihor Kalashnikov: как тогда вы бы поступили если из метода регистрации нужно было бы вернуть код ошибки, что пользователь уже существует? Ведь из false я не могу понять что именно пошло не так.

[Igor Kalashnikov]

Дмитрий Байбухтин: эксепшеном, потому что регистрировать существующие ники запрещено, и сначала нужно дергать валидаторы. Потому что правил может быть много и лучше заранее позаботится о точке расширения. В эксепшен бы передавал бы объект ошибки, с кодом для компьютера, с текстом для разработчика, и с текстом для юзера (это например).

П.С. скорее всего метод был бы REST-вызовом, поэтому завернул бы его в какой-нибудь JSON на самом верху, по коду ошибки, но это уже ErrorHandling, чуть другая тема.

[Igor Kalashnikov]

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

[Дмитрий Б]

Ihor Kalashnikov: я Yii2 использую, поэтому там всё готово, была проблема только в получении ошибки. Спасибо )

Answer 3

[Армянское Радио]

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

Comments

[Mrrl]

Если в программе исключения используются для отработки штатных ситуаций, то практически невозможной становится отладка в режиме "остановиться, когда будет выброшено исключение". Особенно, если это какой-нибудь null reference.

[Армянское Радио]

Mrrl: Зависит от безмозглости отладчика. Можно и break в нужный обработчик поставить.

[Mrrl]

Армянское Радио: Имеется в виду обработчик исключения? Но к этому моменту мы уже потеряем весь контекст, в котором исключение произошло.

[Армянское Радио]

Mrrl: Тут разные трюки можно применять - например, для GDB предлагают "садиться" отладчиком на внутренний обработчик рантайма C++, в который происходит "прилетание" из throw, там подсматривать тип и действовать по обстоятельствам. Я от отладки отладчиком давно отказался (распределенную многопоточку особо не поотлаживаешь) и использую вдумчивое протоколирование..

Answer 4

[Андрей Буров]

Возвращать true или код ошибки, но в будущем может вызвать серьёзные проблемы.Допустим если мы забыли написать строгое сравнение ===

Если кто-то что-то забыл или не знает, это его проблемы.

Comments

[Дмитрий Б]

Спасибо за предложение побыдлокодить )