Более правильный способ обработки не найденной сущности?

Question

[Антон Иванов]

Всем привет!

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

Способ 1:

entity = Entity.find(params[:id]) rescue redirect_to('/')

Способ 2:

entity = Entity.find_by_id(params[:id])
redirect_to('/') unless entity

Answer 1

[Запорожченко Олег]

Всегда должен быть find. А ошибка отлавливается в ApplicationController

unless Rails.application.config.consider_all_requests_local
    rescue_from Exception, with: :respond_with_500

    rescue_from ActiveRecord::RecordNotFound,
                ActionController::RoutingError,
                ActiveRecord::RecordInvalid,
                with: :respond_with_404

    rescue_from ActionController::BadRequest, with: :respond_with_400
  end

def respond_with_404(exception=nil)
    redirect_to root_path
  end

Comments

[Антон Иванов]

А если для каждого контроллера нужна своя обработка исключения? Редирект на index этого контроллера

[Запорожченко Олег]

тогда более правильным будет искать и проверять на результат в фильтре

before_filter :load_resource, only: [:show, :edit, :update, :destroy]

def load_resource
@entity = Entity.find_by_id(params[:id])
unless @entity.present?
redirect_to action: :index

false
end
end

[Антон Иванов]

Олег: Ну а тут использован find_by_id, а не find :) В основном вопрос был про разницу find_by_id/unless и find/rescue :)

[Запорожченко Олег]

Все зависит от задачи. Стандартом является мой первый ответ, потому что приложение не должно выводить ссылки на несуществующие объекты, а значит и переходов по ним быть не может. Если кто-то намеренно пытается подставлять несуществующий id или администратор удалил запись (т.е. довольно редкий случай) - простой редирект на главную является достаточно нормальным поведением, либо отображение 404 страницы. Т.е. переход по такой ссылки не является вашим юзкейсом и Вам всего линь нужно обработать исключительную ситуацию, то используем перехват исключений

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

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

[Антон Иванов]

Олег: Спасибо, примерно так и думал