Как лучше с исключением поступить?

Question

[dev400]

Допустим нет категорий и модель бросила исключение

public function allCategories() {
        while(!$rows = parent::findAll()){
            throw new \Exception('Нет категорий');
        }
        return $rows;
    }

Показать его пользователю, или сделать более информативное исключение и записать его в файл, а пользователю сформировать сообщение об ошибке?

Answer 1

[sts]

И себе всё запиши, и пользователю скажи что он что-то не то искать пытается

Answer 2

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

while(!$rows = parent::findAll()){
    throw new \Exception('Нет категорий');
}
return $rows;

может лучше

if (($rows = parent::findAll()) === false);
    throw new \Exception('Нет категорий');
}
return $rows;

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

Answer 3

[dmitriy]

вам точно здесь нужно исключение? может просто вернуть null ?

Comments

[Adamos]

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

[dmitriy]

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