堆栈级别太深和before_save

时间:2014-11-27 22:45:01

标签: ruby-on-rails ruby-on-rails-3 infinite-loop

好的,这件事让我抓狂。我有一个看起来大致如此的小方法:

class PdfResult < ActiveRecord::Base
  attr_accessible :press_ready_url, :low_resolution_url, :error_code,
                  :document_id

  before_save :update_values

  def created?
    return true if press_ready_url.present? && low_resolution_url.present?
  end

  def error?
    error_code == 201 || error_code == 204 ? false : true
  end

  private

  def update_values
    return if error?
    self.updated_at = Time.now
    if created?
      self.error_code = 201
    else
      update_attributes(press_ready_url: nil, low_resolution_url: nil)
      self.error_code = 204
    end
    save!
  end
end

我的error方法只会导致stack level too deep错误。有人可以帮我理解为什么吗?根据我的逻辑,它应该工作得很好。谢谢。我是否需要防范error_code以某种方式改变?

1 个答案:

答案 0 :(得分:6)

update_attributes保存模型,触发回调。您不应该在save回调中触发before_save,或者是,您将耗尽堆栈。

而不是:

update_attributes(press_ready_url: nil, low_resolution_url: nil)

使用此:

self.press_ready_url = self.low_resolution_url = nil