Rails 4：使用表单对象时验证关联的 ActiveRecord 模型

Question

我正在使用基于 ActiveModel 的表单对象来处理应用程序的注册（注册）。 signup 类抽象出account 和user（帐户的主要用户）的信息。

但是，我发现我在 signup 类中复制了 account 和 user 的验证逻辑。在编写规范时（使用 rspec），我意识到这种重复可能表明我处理此问题的方式存在问题。

有没有办法将signup 类中的验证传递给account 和user 模型而不复制它？这样，验证将保留在这些模型中，我可以在 signup 类中引用/调用它。

下面是 signup 类，我有它的作品，但似乎是重复代码...

class Signup
  include ActiveModel::Model

  # Scopes
  #----------------------------------------------------------------------

  # NOOP

  # Macros
  #----------------------------------------------------------------------

  attr_accessor :slug, :email, :password, :password_confirmation

  # Associations
  #----------------------------------------------------------------------

  # NOOP

  # Validations
  #----------------------------------------------------------------------

  validate :verify_unique_email
  validate :verify_unique_slug
  validates :email, presence: true, format: { with: /@/, message: "is invalid" }
  validates :password, presence: true, length: { minimum: 8 }, confirmation: true
  validates :password_confirmation, presence: true
  validates :slug,
    presence: true,
    format: { with: /\A[\w-]+\z/, message: "is invalid" },
    exclusion: { in: %w[signup signups login] }


  # Methods
  #----------------------------------------------------------------------

  def account
    @account ||= Account.new
  end

  def user
    @user ||= account.build_primary_user
  end

  def save
    account.active = true
    account.slug = slug

    user.email = email
    user.password = password
    user.password_confirmation = password_confirmation

    if valid?
      ActiveRecord::Base.transaction do
        account.save!
        user.save!
      end

      true
    else
      false
    end
  end

  def save!
    save
  end

  private

  def verify_unique_email
    if User.exists?(email: email)
      errors.add :email, "is invalid"
    end
  end

  def verify_unique_slug
    if Account.exists?(slug: slug)
      errors.add :slug, "has already been taken"
    end
  end
end

这是account 模型，注意重复验证：

class Account < ActiveRecord::Base
  has_one :primary_user, -> { where(primary: true) }, class_name: User
  has_many :users, dependent: :destroy

  validates :slug,
    uniqueness: true,
    presence: true,
    format: { with: /\A[\w-]+\z/, message: "is invalid" },
    exclusion: { in: %w[signup signups login] }
end

Answer 1

我喜欢你使用form object 所做的事情。 validates_associated :user, :account 可能会有所帮助，但错误消息可能有点奇怪。相反，我可能会使用mixins for common validations：

class Account < ActiveRecord::Base
  module Validations
    extend ActiveSupport::Concern
    included do
      validates :slug, presence: true
    end
  end
  include Validations
end

class Signup
  include ActiveModel::Model
  include Account::Validations
  extend Forwardable
  def_delegators :account, :slug
end

Answer 2

最简单的方法是：

def valid?
  user.valid? && account.valid? && super
end

然后您可以摆脱重复的验证。

仅当您需要在User 和Account 之外的其他验证时，才需要super。这也会将错误添加到User 和Account 对象上，这可能比Signup 类更有用。