Rails 以复杂形式验证多个子模型

Question

我以复杂的形式使用多个子模型（a la http://railsforum.com/viewtopic .php?id=28447）。该表单运行良好，但在将表单数据接受到数据库之前，我需要验证子模型集的属性。我想出了一种最有效但非常笨拙的方法来做到这一点。似乎必须有更好的方法，所以我寻求建议......

基本上一个人有多个分布。分布具有（除其他外）百分比属性。对于给定的人，他们的分配总数必须达到 100% 才有效。这对我来说是“交易”，但我认为我应该先给验证者一个机会。

我尝试将其编写为自定义验证器，但验证器仅适用于已保存到数据库的数据。它没有检查表单提交的参数。换句话说，我能够通过保存的表单输入无效的百分比，然后由于模型中已有错误数据，后续编辑全部失败。

接下来，我在 Person 模型中扩展了 update_attributes，添加了一个事务：

def update_attributes(params)
  retval = true
  self.transaction do
    retval = super

    unless distributions_exactly_100?
      retval = false
      errors.add_to_base("Distribution must add up to exactly 100%")
      raise ActiveRecord::Rollback
    end
  end
  retval
end

retval 业务很丑陋，但这或多或少有效（有时，当发现错误并重新渲染时，表单中会缺少一些待处理的分配）。还有一个额外的细微差别让我相信这是一个糟糕的方法：如果我的分布关联是使用辅助方法定义的，如下所示，我无法在我的 update_attributes() （或在distributions_exactly_100?)，因为它们会访问数据库，而不是在一组刚刚分配但尚未提交的分布上进行操作。

 has_many :distributions do
   def for_month_and_year(month, year)
     find :all, :conditions => ['month = ? and year = ?', month, year]
   end

   def total_for_month_and_year(month, year)
     sum :percentage, :conditions => ['month = ? and year = ?', month, year]
   end

   ...

   def years_and_months
     ds = find(:all, :order => 'year DESC, month DESC')
     (ds.collect {|d| [d.year, d.month]}).uniq
   end

 end

我唯一能想到的另一件事是在进入 update_attributes 的过程中将参数本身作为文本进行处理。但这是错误的。 :)

还有其他人对整个儿童集合进行验证吗？正确的做法是什么？

Answer 1

我不建议在 update_attributes 中设置错误。将验证保留在正常位置。

回到您的问题，您是否可以更改验证检查以处理内存中的分布，而不是在数据库上执行计算？

# in Person model
validate :ensure_distributions_equal_100

def ensure_distributions_equal_100
  percent = distributions.map(&:percent).sum
  if percent != 100
    errors.add_to_base("Distribution must add up to exactly 100%, they are #{percent}")
  end
end