ActiveModel::Validator调用模型私有方法的优雅实现方案咨询

方案一：用Concern共享验证逻辑（最符合Rails惯例） #

既然两个模型的验证需求相近，直接用ActiveSupport::Concern把验证逻辑封装起来，让模型包含这个模块即可。这样验证逻辑在模型内部执行，天然能访问私有方法，完全规避跨类调用私有方法的问题：

# app/models/concerns/game_settings_validations.rb
module GameSettingsValidations
  extend ActiveSupport::Concern

  included do
    # 直接在模型内部定义验证，指定触发时机
    validate :check_max_players, on: :create
  end

  private

  def check_max_players
    # 这里写原来的验证逻辑，比如：
    if max_players > allowed_max_players
      errors.add(:max_players, "人数不能超过上限")
    end
    # 其他验证规则...
  end
end

然后在两个模型中引入：

class Game < ApplicationRecord
  include GameSettingsValidations
end

class TournamentGame < ApplicationRecord
  include GameSettingsValidations
end

这种方式既保持了check_max_players的私有性，又实现了逻辑复用，完全符合Rails的设计思路。

方案二：重构验证器，直接封装验证逻辑 #

如果你坚持要保留独立的GameSettingsValidator类，可以把模型里的验证逻辑迁移到验证器中，模型只需要暴露验证所需的公共属性（通常模型的属性本身就是可访问的）：

class GameSettingsValidator < ActiveModel::Validator
  def validate(record)
    # 直接在验证器里实现原来check_max_players的逻辑
    if record.max_players > record.allowed_max_players
      record.errors.add(:max_players, "人数不能超过上限")
    end
    # 其他验证规则...
  end
end

然后模型里的check_max_players就可以删掉了，验证器直接通过模型的公共属性完成验证，不需要调用私有方法。

为什么不推荐send或改公共方法？ #

• send(:check_max_players)虽然能解决问题，但破坏了Ruby的封装性，绕过了访问控制，长期来看不利于代码维护。
• 把私有方法改成公共方法，会不必要地暴露模型内部逻辑，违背了面向对象的封装原则。

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