Rails Newbie: рекомендации по обработке ошибок в контроллере

Question

Извините, если вопрос очевиден, я только начинаю работать с Rails.
У меня есть следующий код в несколько методов контроллера в настоящее время:

respond_to do |format| 
    if @project.save 
     format.html { redirect_to(edit_project_url(@project), :notice => '#{user.name} added to #{role}.') } 
     format.js 
    else 
     format.html { render :action => "edit" } 
     format.js #... 
    end 
end 

Таким образом, вопрос, что это лучший способ сделать то же самое для ошибок во всех методах?
Рекомендуется ли использовать save! и обрабатывать его в rescue_action?

Или должен ли я сделать свой собственный метод respond и пройти save в блоке?

Answer 1

Часто бывает полезно использовать исключающий вариант сохранения и спасения, который позже в блоке, чем для такой ветви. Преимущество для исключений заключается в том, что они выйдут из транзакций.

def create 
    @project.save! 

    respond_to do |format| 
    format.html { redirect_to(edit_project_url(@project), :notice => '#{user.name} added to #{role}.') } 
    format.js 
    end 

rescue ActiveRecord::RecordInvalid 
    respond_to do |format| 
    format.html { render :action => "edit" } 
    format.js #... 
    end 
end 

Вы обнаружите, что это становится действительно сложно пререкаться свой путь из кучи вложенных if заявления при попытке сохранить более одного объекта в то время, но простой rescue исключения будет обрабатывать аккуратно ,

def create 
    Project.transaction do 
    @project.save! 
    @something_else.save! 
    @other_stuff.save! 
    end 

    # ... 
rescue ActiveRecord::RecordInvalid 
    # ... 
end 

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

Answer 2

Неплохо использовать шаблон if @object.save. Однако, если вы делаете то же самое для всех своих действий на своем контроллере, вы можете определить действие rescue_from.

Something like

class MyController < ActionController::Base 
    rescue_from ActiveRecord::RecordInvalid do 
    render :action => edit 
    end 
end