Я наткнулся на ситуацию, когда мое приложение ищет идентификатор, который не существует в базе данных. Вызывается исключение. Конечно, это довольно стандартная ситуация для любого веб-разработчика.
Я знаю, что использование спасения с ситуацией довольно аккуратно, например:
def show
@customer = Customer.find(params[:id])
rescue ActiveRecord::RecordNotFound #customer with that id cannot be found
redirect_to action: :index #redirect to index page takes place instead of crashing
end
Если клиент не может быть найден, пользователь будет перенаправлен на индексную страницу. Это работает абсолютно нормально.
Теперь это все хорошо, но мне нужно выполнить те же попытки восстановления в таких действиях, как show, edit, destroy и т.д., т.е. каждый метод контроллера, которому нужен определенный идентификатор.
Сказав, что здесь мой вопрос: Разве нет способа вообще сообщить моему контроллеру, что если он не может найти идентификатор в любом из своих методов, он должен перенаправить на индексную страницу (или, как правило, выполнить определенную задачу)?
Для этой задачи вы должны использовать rescue_from. См. Пример в Обзор контроллера действий.
class ApplicationController < ActionController::Base
rescue_from ActiveRecord::RecordNotFound, :with => :record_not_found
private
def record_not_found
redirect_to action: :index
end
end
Rails имеет встроенный метод rescue_from:
class CustomersController < ApplicationController
rescue_from ActiveRecord::RecordNotFound, with: :index
...
end
Если вы говорите об этом в одном контроллере (в отличие от этого в глобальном масштабе в каждом контроллере), вот несколько вариантов:
Вы можете использовать файл before_filter для настройки вашего ресурса:
class CustomerController < ApplicationController
before_filter :get_customer, :only => [ :show, :update, :delete ]
def show
end
private
def get_customer
@customer = ActiveRecord.find(params[:id])
rescue ActiveRecord::RecordNotFound
redirect_to :action => :index
end
end
Или вы можете использовать метод вместо этого. Я двигался в этом направлении, а не использовал переменные экземпляра внутри представлений, и это также помогло бы вам решить вашу проблему:
class CustomerController < ApplicationController
def show
# Uses customer instead of @customer
end
private
def customer
@customer ||= Customer.find(params[:id])
rescue ActiveRecord::RecordNotFound
redirect_to :action => :index
end
helper_method :customer
end
В некоторых случаях я рекомендую использовать Model.find_by_id(id), а не Model.find(id). Вместо того, чтобы бросать исключение, .find_by_id возвращает nil. если запись не найдена.
Просто убедитесь, что nils избегают NoMethodError!
P.S. Для того, что стоит, Model.find_by_id(id) функционально эквивалентен Model.where(id: id), что позволит вам построить дополнительные отношения, если вы хотите.