Как обрабатывать findOrFail в laravel?

Question

[igor1112]

Есть код выборки новости из базы по id:

public function getNews($id) {
       return  App\News::findOrFail($id);
}

Зачем здесь нужно генерировать исключение? Не лучше ли просто проверить на пустоту объект и в этом случае вернуть пустой объект, а иначе вернуть объект с найденной новостью?

И если всё-таки правильнее генерить исключение, то как эту штуку обрабатывать и где?

Comments

[Arman]

а где этот метод? в контроллере? понимаете для чего Исключения?

[Дмитрий Кузнецов]

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

[igor1112]

Arik, Не в контроллере. Контролллер обращается к сервисному слою, а сервисный слой обращается к репозиторию, и этот код как раз из репозитория. Контроллер ждёт объект, и без исключений он к нему придёт, пустой или с новостью. И зачем исключение то?

Answer 1

[Дмитрий]

try
{
    $news= App\News::findOrFail($id);
}
catch(ModelNotFoundException $e)
{
    dd($e);
}

Comments

[igor1112]

Но дело в том, что возващаемый пустой объект это не ошибка, это часть логики.

[Дмитрий]

Но дело в том, что если где то в коде будет вызов свойства объекта($news->title), то это приведет к ошибке, т.к. объект пустой.

Answer 2

[UksusoFF]

Чтобы, например, поймать ModelNotFound в app/Http/Handler.php и вернуть красивую 404. Хочешь проверять на null - никто не запрещает - используй просто find.