基于Cocoon Gem的嵌套记录模型create方法重写问题：实现存在则更新、不存在则创建逻辑

首先，你遇到的错误是因为Rails的ActiveRecord模型已经内置了一个私有方法create_or_update，这个方法是框架内部用来处理保存逻辑的，不需要任何参数。你自己定义了同名方法，导致在调用时（不管是你自己的代码还是Rails内部调用）参数不匹配，抛出了wrong number of arguments的错误。

接下来，我们来一步步解决你的需求：实现"存在则更新，不存在则创建"的逻辑，同时适配Cocoon的AJAX嵌套表单场景。

方案1：自定义类方法（推荐） #

避免和内置方法冲突，我们定义一个自定义的类方法来处理逻辑，同时使用Rails推荐的查询语法：

class PayrollEmployee < ApplicationRecord
  belongs_to :payroll
  belongs_to :employee_contract

  # 自定义类方法，处理创建或更新逻辑
  def self.create_or_update_by_payroll_and_contract(payroll_id, employee_contract_id, additional_attrs = {})
    # 查找或初始化记录
    payroll_employee = find_or_initialize_by(
      payroll_id: payroll_id,
      employee_contract_id: employee_contract_id
    )
    # 合并并更新属性，使用update!确保错误会抛出
    payroll_employee.update!(additional_attrs.merge(
      payroll_id: payroll_id,
      employee_contract_id: employee_contract_id
    ))
    # 返回处理后的记录
    payroll_employee
  end
end

在控制器中调用这个方法：

# 示例：在payrolls_controller的update动作中
def update
  # 获取需要的参数
  payroll_id = @payroll.id
  employee_contract_id = params[:payroll_employee][:employee_contract_id]
  other_attrs = params[:payroll_employee].except(:employee_contract_id)
  
  PayrollEmployee.create_or_update_by_payroll_and_contract(payroll_id, employee_contract_id, other_attrs)
  # 其他AJAX响应逻辑...
end

方案2：适配Cocoon嵌套表单的属性处理 #

如果是通过Cocoon的嵌套表单提交数据，你可以在父模型（Payroll）中自定义嵌套属性的处理逻辑，自动处理每个子记录的创建/更新：

class Payroll < ApplicationRecord
  has_many :payroll_employees, dependent: :destroy
  # 先注释掉默认的accepts_nested_attributes_for，改用自定义方法
  # accepts_nested_attributes_for :payroll_employees

  def payroll_employees_attributes=(attributes)
    attributes.each_value do |attr|
      # 处理删除逻辑（如果需要）
      if attr[:id].present? && attr[:_destroy] == '1'
        payroll_employees.find(attr[:id]).destroy
        next
      end

      # 查找或初始化对应的PayrollEmployee记录
      payroll_employee = payroll_employees.find_or_initialize_by(
        employee_contract_id: attr[:employee_contract_id]
      )
      # 赋值并保存
      payroll_employee.assign_attributes(attr.except(:id, :_destroy))
      payroll_employee.save!
    end
  end
end

这样，当你提交嵌套表单时，每个employee_contract_id对应的记录会自动匹配：存在则更新，不存在则创建，不会重复生成。

重要优化：添加数据库唯一约束 #

为了从根源防止重复数据，建议给payroll_employees表添加联合唯一索引，确保同一个payroll_id和employee_contract_id的组合只能存在一条记录：

生成迁移文件：

rails generate migration AddUniqueIndexToPayrollEmployees

编辑迁移文件：

class AddUniqueIndexToPayrollEmployees < ActiveRecord::Migration[6.1]
  def change
    add_index :payroll_employees, [:payroll_id, :employee_contract_id], unique: true
  end
end

执行迁移：

rails db:migrate

进阶方案：使用Rails的Upsert（Rails 6+） #

如果你的Rails版本在6.0及以上，可以使用upsert方法，这是数据库层面的原子操作，效率更高：

PayrollEmployee.upsert(
  {
    payroll_id: payroll_id,
    employee_contract_id: employee_contract_id,
    # 其他需要更新的属性
    salary: params[:payroll_employee][:salary]
  },
  unique_by: [:payroll_id, :employee_contract_id]
)

这个方法会直接在数据库层面执行"插入或更新"操作，不需要先查询再修改，性能更好，前提是已经添加了上述的联合唯一索引。

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