Rails项目中ItemUtil模块初始化调用及eql?方法优化问题

核心错误原因：你将ItemUtil作为混入模块引入ActiveRecord模型时，模块中自定义的initialize方法不会被自动调用——Item模型的实例化由ActiveRecord控制，你自定义的带参initialize和ActiveRecord的实例化逻辑冲突，导致@field等实例变量未赋值，触发空值错误。

1. 如何让new_item和current_item方法返回预期结果？ #

不需要单独定义initialize方法，直接在eql?入口处给实例变量赋值即可，完全兼容原有调用方式，修正后代码如下：

# lib/item_util.rb
module ItemUtil
  def eql?(item, field: "cost", op: "==")
    # 入口处统一赋值实例变量
    @compare_item = item
    @compare_field = field
    @compare_op = op

    new_item.present? && new_item.send(@compare_op, current_item)
  end

  private
  def new_item
    @compare_item.send(@compare_field)
  end

  def current_item
    self.send(@compare_field)
  end
end

2. 如何在eql?方法内部触发initialize方法调用？ #

不建议在当前使用场景下触发自定义的initialize方法：
你当前是将模块作为mixin混入Item模型，模块的方法都是Item实例的实例方法，Item实例本身就是self，不需要额外把自身存入@item变量。如果硬要在eql?中调用你写的initialize，会覆盖Item实例原有属性，导致更多ActiveRecord相关的异常。
如果坚持要使用带initialize的结构，需要将ItemUtil改为独立工具类，但会修改原有调用方式，不符合你的需求。

3. 是否有其他替代实现方案？ #

提供两种更稳妥的实现：

• 极简安全版：不需要拆分方法，原有逻辑本身足够简洁，用public_send和安全调用优化，避免意外调用私有方法：

module ItemUtil
  def eql?(item, field: "cost", op: "==")
    item&.public_send(field)&.public_send(op, self.public_send(field)) == true
  end
end

• 线程安全版：用参数传递代替实例变量，避免同一实例并发调用eql?时的变量覆盖问题：

module ItemUtil
  def eql?(item, field: "cost", op: "==")
    target_val = get_field_value(item, field)
    current_val = get_field_value(self, field)
    target_val.present? && target_val.public_send(op, current_val)
  end

  private
  def get_field_value(record, field)
    record.public_send(field)
  end
end

4. 模块中是否有类似控制器before_action的方式可以统一访问参数？ #

可以借助Rails默认提供的ActiveSupport::Callbacks实现类似before_action的回调逻辑，示例如下：

module ItemUtil
  extend ActiveSupport::Concern

  included do
    define_callbacks :eql_run
    set_callback :eql_run, :before, :init_compare_params
  end

  def eql?(item, field: "cost", op: "==")
    @raw_params = {item: item, field: field, op: op}
    run_callbacks :eql_run do
      new_item.present? && new_item.public_send(op, current_item)
    end
  end

  private
  def init_compare_params
    @compare_item = @raw_params[:item]
    @compare_field = @raw_params[:field]
  end

  def new_item
    @compare_item.send(@compare_field)
  end

  def current_item
    self.send(@compare_field)
  end
end

如果没有复杂的回调复用需求，直接在eql?开头统一处理参数即可，逻辑更清晰，性能也更高。

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