控制 Rails 验证的顺序

Question

我有一个 Rails 模型，其中有 7 个数字属性，由用户通过表单填写。

我需要验证每个属性的存在，这显然很容易使用

validates :attribute1, :presence => true
validates :attribute2, :presence => true
# and so on through the attributes

，但是我还需要运行一个自定义验证器，它接受许多属性并用它们进行一些计算。如果这些计算的结果不在一定范围内，则该模型应被宣布无效。

就其本身而言，这也很容易

validate :calculations_ok?

def calculations_ok?
  errors[:base] << "Not within required range" unless within_required_range?
end

def within_required_range?
  # check the calculations and return true or false here
end

，但问题是方法“验证”总是在方法“验证”之前运行。这意味着，如果用户将必填字段之一留空，rails 在尝试使用空白属性进行计算时会抛出错误。

那么我怎样才能首先检查所有必需的属性是否存在呢？

Answer 1

我不确定是否能保证这些验证按什么顺序运行，因为它可能取决于 attributes 哈希本身的最终排序方式。您可能最好让您的 validate 方法更具弹性，并且在缺少某些所需数据时根本不运行。例如：

def within_required_range?
  return if ([ a, b, c, d ].any?(&:blank?))

  # ...
end

如果变量 a 到 d 中的任何一个变量为空（包括 nil、空数组或字符串等），则会退出。

Answer 2

对于稍微复杂的情况，另一种方法是创建一个辅助方法，该方法首先对依赖属性运行验证。然后你就可以进行 :calculations_ok?有条件地运行验证。

validates :attribute1, :presence => true
validates :attribute2, :presence => true
...
validates :attribute7, :presence => true

validate :calculations_ok?, :unless => Proc.new { |a| a.dependent_attributes_valid? }

def dependent_attributes_valid?
  [:attribute1, ..., :attribute7].each do |field|
    self.class.validators_on(field).each { |v| v.validate(self) }
    return false if self.errors.messages[field].present?
  end
  return true
end

我必须为项目创建类似的东西，因为对依赖属性的验证非常复杂。我相当于：calculations_ok？如果依赖属性未正确验证，则会引发异常。

优点：

• 相对干燥，特别是当您的验证很复杂时，
• 确保您的错误数组报告正确的失败验证，而不是宏验证
• 自动包含对您稍后添加的依赖属性的任何其他验证

注意事项：

• 所有验证两次
• 可能会运行您可能不希望的 对依赖属性运行的所有验证

Answer 3

查看 http://railscasts.com/episodes/211-validations-in-rails -3

实现自定义验证器后，您只需

validates :attribute1, :calculations_ok => true

这样做即可解决您的问题。

Answer 4

我记得很久以前就遇到过这个问题，仍然不清楚是否可以设置验证顺序，并且如果验证返回错误，执行链是否会停止。

我认为 Rails 没有提供这个选项。这是有道理的；我们希望显示记录中的所有错误（包括由于输入无效、验证而失败后出现的错误）。

一种可能的方法是仅在要验证的输入存在时才进行验证：

def within_required_range?
  return unless [attribute1, attribute2, ..].all?(&:present?)
  
  # check the calculations and return true or false here
end

使其美观并使其美观。使用 Rails 惯用的验证选项进行更好的结构化（单一职责）：

validates :attribute1, :presence => true
validates :attribute2, :presence => true
# and so on through the attributes

validate :calculations_ok?, if: :attributes_present?

private
  def attributes_present?
    [attribute1, attribute2, ..].all?(&:present?)
  end

  def calculations_ok?
    errors[:base] << "Not within required range" unless within_required_range?
  end

  def within_required_range?
    # check the calculations and return true or false here
  end

Answer 5

James H 解决方案对我来说最有意义。然而，需要考虑的另一件事是，如果您对依赖验证有条件，则还需要检查它们才能获得 dependent_attributes_valid？打电话去上班。

IE。

    validates :attribute1, presence: true
    validates :attribute1, uniqueness: true, if: :attribute1?
    validates :attribute1, numericality: true, unless: Proc.new {|r| r.attribute1.index("@") }
    validates :attribute2, presence: true
    ...
    validates :attribute7, presence: true

    validate :calculations_ok?, unless: Proc.new { |a| a.dependent_attributes_valid? }

    def dependent_attributes_valid?
      [:attribute1, ..., :attribute7].each do |field|
        self.class.validators_on(field).each do |v|
          # Surely there is a better way with rails?
          existing_error = v.attributes.select{|a| self.errors[a].present? }.present?

          if_condition = v.options[:if]
          validation_if_condition_passes = if_condition.blank?
          validation_if_condition_passes ||= if_condition.class == Proc ? if_condition.call(self) : !!self.send(if_condition)

          unless_condition = v.options[:unless]
          validation_unless_condition_passes = unless_condition.blank?
          validation_unless_condition_passes ||= unless_condition.class == Proc ? unless_condition.call(self) : !!self.send(unless_condition)

          if !existing_error and validation_if_condition_passes and validation_unless_condition_passes
            v.validate(self)
          end
        end
        return false if self.errors.messages[field].present?
      end
      return true
    end