Rails 3复杂的验证设计

Question

我有一个Contribution和Reward模型。 贡献belongs_to奖励。 我希望贡献只有在其金额大于其奖励的minimum_contribution时才有效。

所以我有

Class Contribution < ActiveRecord::Base
  attr_accessible :reward_id, :reward
  belongs_to :reward

  validates_presence_of :amount
  validates_numericality_of :amount, greater_than: 0
  validates_presence_of :reward

  validates_each :amount, unless: SOMECONDITION do |contribution, attr, amount|
    reward = contribution.reward
    contribution.errors.add(attr,
                            "Contribution must be at least #{reward.minimum_contribution}") unless amount > reward.minimum_contribution
  end
end

我的问题是适合SOMECONDITION 。 在运行此验证之前，我需要确保

1. 贡献的有效值已定义（甚至是数字）。 否则进行比较， unless value > reward.minimum_contribution会出现错误，因为您无法与nil比较
2. 贡献的奖励已定义
3. 贡献的奖励的minimum_contribution定义为数值

我应该在lambda中为“ SOMECONDITION执行所有SOMECONDITION吗？ 拥有如此多的防护措施来进行单个验证只是感觉不对劲。 我是否设计正确？ 例如，是否有适当的设计在我的验证中抛出异常， unless amount > contribution.reward.minimum_contribution在amount实际上为nil时， unless amount > contribution.reward.minimum_contribution nil ？ （显然，我一直希望金额是数字，但我不确定我在验证中应该有多偏执/我可以完全信任什么）

Answer 1

您可以将validates_each :amount替换为：

...
validate :validate_mininum_amount

def validate_mininum_amount
  errors.add :amount, "some error message" if amount.to_i < reward.min_contribution
end
...

注意：添加.to_i保证金额将是0 ，如果它是nil 。 或者，您可以添加特定的nil检查。 在validate_mininum_amount方法中添加您需要的其他任何检查。