Rails Newbie: Khuyến nghị xử lý lỗi trong bộ điều khiển

Question

Xin lỗi nếu câu hỏi là hiển nhiên, tôi chỉ bắt đầu làm việc với Rails.
Tôi có một đoạn mã sau vào một số phương pháp điều khiển bây giờ:

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 

Vậy câu hỏi là, cách tốt nhất để làm điều tương tự cho các lỗi trong tất cả các phương pháp là gì?
Tôi có nên sử dụng save! và xử lý nó trong rescue_action?

Hoặc tôi có nên tự mình làm phương pháp respond và vượt qua save trong một khối không?

Answer 1

Thường thuận tiện hơn khi sử dụng biến thể lưu và cứu ngoại lệ mà sau này trong khối so với chi nhánh như vậy. Lợi thế của trường hợp ngoại lệ là họ sẽ phá vỡ các giao dịch.

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 

Bạn sẽ thấy rằng nó được thực sự khó khăn để wrangle theo cách của bạn ra khỏi một đống lồng nhau if báo cáo khi cố gắng tiết kiệm nhiều hơn một đối tượng tại một thời điểm, nhưng một đơn giản rescue cho trường hợp ngoại lệ sẽ xử lý nó gọn gàng .

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

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

Nếu một trong số đó tiết kiệm được, bạn sẽ nhận được ngoại lệ. Để đảm bảo rằng tất cả chúng đều hiển thị các lỗi xác thực, bạn có thể phải gọi .valid? trên mỗi số để đánh dấu chúng hoặc bạn sẽ có những lỗi sau khi lỗi không được kiểm tra.

Answer 2

Không phải là điều xấu khi sử dụng mẫu if @object.save. Tuy nhiên, nếu bạn đang làm chính xác như nhau cho tất cả các hành động của bạn trên bộ điều khiển của bạn, bạn có thể xác định một hành động rescue_from.

Something like

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