Rails：向after_save回調中產生的用戶顯示錯誤

Question

如何解決after_save回調中生成的錯誤，然後最終將其顯示給用戶？在我的模型代碼看起來是這樣的：

Class MyModel 
    after_save :call_other_class_responsible_for_parsing 

    def call_other_class_responsible_for_parsing 
    # this method is used by multiple models 
    ModelTwo.parse_css 
    end 
end 

在我的控制，我現在將用戶重定向的其他地方，如果升級成功，但是，我認爲更新是成功的，如果它通過了所有現有的驗證的和回調中沒有錯誤（來自Less :: Parser）。

編輯：

我混淆了我的想法在我原來的問題。 MyModel從相應的控制器中獲取，然後從模型中運行after_save回調。在call_other_class_responsible_for_parsing裏面，有一個對另一個模型的調用，比如ModelTwo，它可以進行Less解析。我一直在使用這樣的代碼嘗試：

def self.parse_css 
    @my_model = MyModel.find(1) 
    css_to_compile = Less::Parser.new.parse(css).to_css 
    rescue Less::Error => error 
     @my_model.errors[:base] << "Error message"  
     false 
    end 
    end 

但false並不妨礙交易得手，因此重定向發生。

Answer 1

你可以使用事務：

def create 
    MyModel.transaction do 
    @my_model.save 
    @my_model.my_method 
    end 
    rescue ActiveRecord::RecordInvalid => exception 
    # rescue active record exception here 
    rescue Less::Error => exception 
    # rescue less error here 
end 

這消除對回調的需要。

我希望這有助於！

Answer 2

回調鏈隱式包裝在一個事務中。當回調返回false或引發異常時，整個事務回滾並保存失敗。

就你而言，你解析了一些CSS，所以我不確定after_save是否適合這個。 我建議你試一試。考慮以下幾點：

class MyModel < ActiveRecord::Base 
    validate :valid_css 

    private 

    def parsed_css 
    @parsed_css ||= Less::Parser.new.parse(css).to_css 
    end 

    def valid_css 
    parsed_css 
    rescue Less::Error => error 
    errors.add(:css, "Cannot parse CSS: #{error}") 
    end 
end 

這將解析CSS 之前保存對象，如果它是無效的添加錯誤。此外，#to_css的結果將被存儲，以便您不需要重新計算它。 這是我推薦的方法。

如果你想堅持after_save那麼你應該提出異常中止交易。根據你的情況，它只是關於不救Less::Error：

class MyModel < ActiveRecord::Base 
    after_save :compile_css 

    private 

    def compile_css 
    css_to_compile = Less::Parser.new.parse(css).to_css 
    end 
end 

如果您無法決定要使用的方法，然後在下方留下一個問題，我會幫助你。