Rails中Contact模型仅修改邮箱时创建新记录的实现求助

推荐方案：在控制器中处理业务逻辑 #

把判断逻辑放在ContactsController的update动作里，这更符合Rails MVC原则，能彻底避免模型回调带来的递归问题：

def update
  @contact = Contact.find(params[:id])
  incoming_email = params[:contact][:email]

  if @contact.email != incoming_email
    # 创建新的Contact记录
    new_contact_attrs = contact_params.except(:id, :verified, :start_date, :end_date)
    new_contact_attrs.merge!(
      user_id: @contact.user_id,
      verified: false,
      start_date: Date.today,
      end_date: Date.today + 10.years
    )
    Contact.create!(new_contact_attrs)

    # 更新原记录的结束日期，标记为已归档
    @contact.update!(end_date: Date.today)

    redirect_to @contact, notice: '邮箱地址已更新，旧联系方式已归档'
  else
    # 仅修改其他字段，直接更新原记录
    if @contact.update(contact_params)
      redirect_to @contact, notice: '联系方式已更新'
    else
      render :edit
    end
  end
end

private

def contact_params
  params.require(:contact).permit(:address, :city, :state, :email, :verified, :start_date, :end_date)
end

模型回调优化方案（不推荐，仅作备选） #

如果一定要在模型层处理，需避免递归触发回调，且不要在回调内部手动调用save（before_save执行后Rails会自动完成保存）：

# 在Contact模型中
before_save :archive_old_contact_and_create_new, if: -> { email_changed? && persisted? }

def archive_old_contact_and_create_new
  # 复制属性并排除不需要的字段
  new_contact_attrs = attributes.except('id', 'verified', 'start_date', 'end_date', 'created_at', 'updated_at')
  new_contact_attrs.merge!(
    verified: false,
    start_date: Date.today,
    end_date: Date.today + 10.years
  )

  # 创建新记录：由于新记录未持久化，persisted?为false，不会触发当前回调
  Contact.create!(new_contact_attrs)

  # 还原当前记录的邮箱地址
  self.email = email_was
  # 设置当前记录的结束日期
  self.end_date = Date.today
end

原代码问题分析 #

1. 栈溢出原因：你在before_save回调中手动调用self.save，导致递归触发save流程，即使还原了邮箱地址，多次调用也会引发栈溢出。before_save回调执行后，Rails会自动执行保存操作，无需手动调用save。
2. 回调范围不明确：原回调未判断persisted?，虽然新建记录的email_changed?实际为false（无原始值对比），但加上persisted?可明确仅针对已存在的记录更新时触发逻辑。

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