Зачем нужны исключения?

Question

[dev400]

Есть такая ситуация

public function getNews($id) {

        $this->key = "id";
        if(!$rows = parent::findById($id)){
            throw new \Exception('not find news');
        }
        return $rows;

    }

Если новостей нет, выбрасывается исключение.

Так почему бы не сделать просто

public function getNews($id) {

        $this->key = "id";
        if(!$rows = parent::findById($id)){
            return "Нет новостей";
        }
        return $rows;

    }

Вместо каши с исключением, мы увидим красивое сообщение.

Answer 1

[crmMaster]

Вместо ожидаемого массива вы получите строку. Поведение функции не определено.

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

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

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

Вообще, вопрос очень странный, джуниорам за такие вопросы надо проводить профилактическую беседу про ООП и обработку ошибок, а за подобный код - бить по рукам палкой.

Comments

[Андрей Рязанцев]

Понтов много, а на вопрос так и не ответил. Зачем тебе бектрейс в вышеприведенном случае?

[crmMaster]

Андрей Рязанцев, Если у тебя не хватило ума понять что там написано, это не мои проблемы.
Бектрейс нужен для отладки.

[Андрей Рязанцев]

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

[crmMaster]

Да тебе я смотрю ничего не нужно, особенно мозги. Ты проект автора видел? Откуда твои утверждения что ему отладка не нужна?
Конечно, если ты всю жизнь пишешь такое дерьмо, как MALDA, https://github.com/Pum-purum/malda то тебе не то что отладка не нужна, тебе программирование не нужно. С таким интеллектом иди лучше на завод, гайки на конвеере крутить, хоть какая-то польза будет, чем от очередного пыхапе говнокодера из глубинки, который думает что он умеет программировать.

[crmMaster]

Андрей Рязанцев,
Топовый код
https://github.com/Pum-purum/malda/blob/master/ove...

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

[Андрей Рязанцев]

Это мой первый проект, который я делал в качестве изучения PHP. Без учителей и репетиторов, в свободное от работы время. Специально из всех моих проектов ты выбрал именно его. Лол. Забейся в угол и поплачь еще.

[crmMaster]

Андрей Рязанцев, то есть дурачок, который 2 года назад начал учить PHP пытается тут что-то доказать людям, которые 3 года назад знали PHP лучше него?

И тебе сколько лет? "Вытри сопельки, иди к маме, поплачь" - это тебя в школе так с оппонентами общаться научили? Ну так здесь не школа, тут взрослые люди, и таких умников-вайтишников, которые думают что что-то знают в серьез никто не воспринимает.

[Андрей Рязанцев]

Посмотри на своей первый коммент, там где ты советовал "по рукам бить палкой". Отношение к тебе сформировалось сразу же. Нормальный специалист так разговаривать не будет.

[crmMaster]

Андрей Рязанцев, Ну тебе то с высоты своего огромного двух летнего опыта куда виднее, кто нормальный разраб. И если по рукам не бить, вырастают такие мальтоделатели, которые вместо знакомства с нормальными практиками херачат код как в 2008 - вместе с хтмл

[Андрей Рязанцев]

конечно виднее. я рад, что тут наши мнения сходятся.

Answer 2

[Станислав Макаров]

Исключения - это принципиальный подход к обработке ошибок и структурирования кода обработки нештатных ситуаций. Исключения сегодня есть во многих современных ОО-языках, и в целом работа с ними совершенно не зависит от того, как, куда и какие выдаются сообщения.
То, что вам вываливается не очень удобный для чтения текст - это вопрос конкретно поведения PHP для непойманных исключений, а не исключений в принципе. Если почитаете не только про то, как исключения выбрасывать, а еще и как их обрабатывать, то возможно поймете, что при возникновении исключения можно совершить любые необходимые действия в скрипте, как то записать ошибку в текстовый лог, записать в БД с датой и временем, или даже послать SMS дежурному администратору.
Не знаю, что вы использовали в качестве источника (неужели книги по PHP так плохи?), но вы что-то явно не дочитали про исключения. Сейчас ваш вопрос похож на такой: "Зачем мне нужна фура, если я могу добраться на работу на 38-м автобусе?". Т.е. как если бы вы раньше не видели грузового транспорта, и не представляли бы, что с ним вообще делают.

Answer 3

[trevoga_su]

В данном примере все не правильно - уместнее всего вернуть пустой массив или объект, реализующий интерфейс Countable (при условии, что в случае нахождения новостей они все также складируются в этот объект).

Отсутствие новости - НЕ исключительная ситуация. Пример не правильный на все 100%.
Исключительная ситуация - мы не может к СУБД законнектиться, что бы новости получить.

А исключения нужны, Ты просто их не умеешь готовить и не понимаешь, что это такое. В интернете об исключениях масса информации. Нет смысла объяснять тут.

Comments

[Станислав Макаров]

Запрос по id, нелогично возвращать массив (пустой или непустой). Если конечно это id новости, а не автора или еще чего)

[trevoga_su]

Станислав Макаров: а, я думал там список возвращается.. ну тогда да, я не прав.

[Станислав Макаров]

trevoga_su "News" - слово такое, по нему не поймешь сразу, много или одна)