Rails ActiveRecord回调搞乱

时间:2013-03-11 18:42:34

标签: ruby-on-rails activerecord

这是推动坚果。我有一个简单的死回调函数来初始化和验证类子类:

class A < ActiveRecord::Base
   has_many :bs
   after_initialize :add_t_instance
   validate :has_only_one_t

  protected

  def add_t_instance
      bs << B.new(:a => self, :type => "T") unless bs.map(&:type).count("T") > 0
  end

  def has_only_one_t
     unless bs.map(&:type).count("T") < 2
       errors.add(:bs, 'has too many Ts")
     end
  end

end

现在,运行时出现了魔力:

a = A.new
>>[#<A>]
a.bs
>> [#<T>]
a.save
>> true
a.id
>> 15

到目前为止,一切都很顺利,但是:

s = A.find(15)
s.bs
>>[#<T>,#<T>]
s.bs.count
>> 2
s.valid?
>> false
s.errors.full_messages
>> "Too many Ts"

我在这里错过了什么?!?!世界上有什么可以加入第二个#T?

1 个答案:

答案 0 :(得分:1)

每当实例化活动记录对象时,不仅在创建新实例之后,而且在从数据库加载现有实例之后,都会调用(至少对我来说)after_initialize。因此,当您运行A.find(15)时,您将创建第二个B.

您可以通过检查是否正在处理回调中的新记录来解决问题,例如

def add_t_instance
  if new_record?
    bs << B.new(:a => self, :type => "T") unless bs.map(&:type).count("T") > 0
  end
end

或者您可以在before_initialize声明本身上放置一个条件,或者尝试使用before_create回调。