Rails模型调用方法报NoMethodError: undefined private method解决方法

• 直接触发原因：你在OrderItem模型中设置了before_validation :generate_order回调，该回调会在OrderItem实例执行保存操作时，自动调用实例层面的generate_order方法。但你仅定义了类方法def self.generate_order，没有编写对应的实例方法，因此实例触发回调时找不到对应方法，抛出报错。
• 业务逻辑隐患：当前generate_order类方法查询status: 1的未关闭订单时，没有关联用户ID过滤，会导致所有用户共用同一个未关闭订单，业务逻辑完全错误。
• 代码写法不规范：
  1. 你将generate_order类方法放在了private块下，Ruby中普通的private关键字不会限制类方法的访问权限，虽然本次控制器调用没有受影响，但属于不规范写法，后续可能引发访问权限问题。
  2. 控制器中你使用if @order_item.save!的写法错误，save!方法在验证失败时会直接抛出异常，不会返回布尔值，永远无法走到else分支。

1. 首先删除OrderItem模型中多余的before_validation :generate_order回调，你已经在控制器中手动为OrderItem绑定了order关联，不需要再通过回调重复生成订单。
2. 修复generate_order类方法的查询逻辑，限定仅查询当前用户的未关闭订单，同时将方法移到公共方法区域：

class OrderItem < ApplicationRecord
  belongs_to :order
  belongs_to :product

  # 公共类方法，移到private块之外
  def self.generate_order(user)
    # 增加user_id过滤条件，只查询当前用户的未关闭订单
    order = Order.find_by(user_id: user.id, status: 1)
    unless order
      order = Order.new(status: 1, total: 0, subtotal: 0, date: Date.today)
      order.user = user 
      order.save!
    end
    order
  end

  private
  # 其他私有实例方法写在这里
end

3. 修正控制器中save的写法，把save!改为普通save，保证验证失败时能走else分支：

class OrderItemsController < ApplicationController
  def create
    @product = Product.find(params[:product_id])
    @order = OrderItem.generate_order(current_user)
    @order_item = OrderItem.new
    @order_item.product = @product
    @order_item.order = @order
    if @order_item.save
      redirect_to cart_path
    else
      redirect_to root_path
    end
  end

  def delete
  end
end

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