Rails中after_validation清除验证报错的实现异常问题

问题描述 #

希望在模型字段验证失败时将其设为null，实现了如下MyModel模型：

class MyModel < ApplicationRecord
  attr_accessor :should_set_fields_to_null_if_invalid

  NULL_IF_INVALID_FIELDS = [:address, :phone]

  after_validation :set_fields_to_null_if_invalid, if: :should_set_fields_to_null_if_invalid
  ...
  # custom validations for address and phone
  ...

  def set_fields_to_null_if_invalid
    NULL_IF_INVALID_FIELDS.each do |attribute|
      self[attribute] = nil if errors.key?(attribute)
      errors.delete(attribute)
    end
  end
end

执行save!时，即使已经移除对应字段的报错并设为null，仍触发无报错信息的异常：

ActiveRecord::RecordInvalid:
  Validation failed: 
# /Users/albertjankowski/.rvm/gems/ruby-3.0.3/gems/activerecord-6.1.7.3/lib/active_record/validations.rb:80:in `raise_validation_error'
# /Users/albertjankowski/.rvm/gems/ruby-3.0.3/gems/activerecord-6.1.7.3/lib/active_record/validations.rb:53:in `save!'

问题原因 #

1. 回调时机错误：after_validation是在所有验证流程完成后执行的，此时ActiveRecord已经标记模型为invalid?状态。即使删除了errors中的字段错误，模型的验证状态不会自动重置，save!依然会判定模型无效并抛出异常。
2. 未处理全局错误：可能存在errors[:base]中的非字段错误（比如模型级别的验证错误），现有代码未处理这类错误，导致验证失败状态依然存在。

正确实现方案 #

方案一：验证前提前处理字段（推荐） #

将字段处理逻辑移到before_validation回调，在验证执行前就把无效字段设为null，从根源避免触发验证错误：

class MyModel < ApplicationRecord
  attr_accessor :should_set_fields_to_null_if_invalid

  NULL_IF_INVALID_FIELDS = [:address, :phone]

  # 验证前处理字段
  before_validation :set_fields_to_null_if_invalid, if: :should_set_fields_to_null_if_invalid

  # 自定义验证：仅当字段非null时执行
  validates :address, length: { minimum: 5 }, unless: -> { address.nil? }
  validates :phone, format: { with: /^\d{11}$/ }, unless: -> { phone.nil? }

  def set_fields_to_null_if_invalid
    NULL_IF_INVALID_FIELDS.each do |attribute|
      value = send(attribute)
      # 根据自定义验证规则判断字段是否无效
      if invalid_field?(attribute, value)
        self[attribute] = nil
      end
    end
  end

  private

  def invalid_field?(attribute, value)
    case attribute
    when :address
      value.present? && value.length < 5
    when :phone
      value.present? && !value.match?(/^\d{11}$/)
    end
  end
end

方案二：验证流程中处理错误 #

如果需要保留原有的验证逻辑，改用validate回调在验证过程中处理错误，确保模型状态被正确更新：

class MyModel < ApplicationRecord
  attr_accessor :should_set_fields_to_null_if_invalid

  NULL_IF_INVALID_FIELDS = [:address, :phone]

  # 自定义验证回调，在验证流程中处理错误
  validate :handle_invalid_fields, if: :should_set_fields_to_null_if_invalid

  # 原有自定义验证
  validates :address, length: { minimum: 5 }
  validates :phone, format: { with: /^\d{11}$/ }

  def handle_invalid_fields
    NULL_IF_INVALID_FIELDS.each do |attribute|
      # 触发该字段的验证（确保错误被记录）
      validate_attribute(attribute)
      if errors.key?(attribute)
        self[attribute] = nil
        errors.delete(attribute)
      end
    end
    # 清除可能存在的全局错误（不需要可删除）
    errors.delete(:base)
  end
end

方案三：手动重置验证状态（不推荐） #

如果必须使用after_validation，处理完错误后手动重置模型的验证状态，但这种方式可能绕过其他必要验证，需谨慎使用：

def set_fields_to_null_if_invalid
  NULL_IF_INVALID_FIELDS.each do |attribute|
    self[attribute] = nil if errors.key?(attribute)
    errors.delete(attribute)
  end
  # 清除全局错误
  errors.delete(:base)
  # 重置模型验证状态
  @errors = ActiveModel::Errors.new(self)
end

内容的提问来源于stack exchange，提问作者albertski