Ruby on Rails中before_save导致新建记录后立即更新的问题

嘿，我来帮你分析下这个问题：你用了before_save回调来处理金额的正负转换，但这个回调会在创建（create）和更新（update）操作前都触发，这就是导致记录创建后值不正确的核心原因！

举个例子：当你新建一笔debit类型的交易时，第一次保存（create）已经把amount乘以-1变成负数了；如果之后因为某些原因（比如关联账户的自动同步、控制器里的额外逻辑，甚至是Rails内部的某些触发机制）触发了第二次保存（update），这时候trx_type作为内存中的attr_accessor属性可能还存在，回调会再次执行，把已经是负数的金额又乘一次-1，结果就变回正数，或者出现其他不符合预期的值。

下面给你几个可行的解决方案：

方案1：改用before_create回调（最推荐） #

既然你只需要在创建记录时调整金额，完全可以把before_save换成before_create，这个回调只会在新建记录时触发，后续更新操作不会再执行这段逻辑：

class Transaction < ApplicationRecord
  belongs_to :account
  attr_accessor :trx_type
  before_create do
    if self.trx_type == "debit"
      self.amount = self.amount * -1
    end
  end
end

方案2：在before_save里增加判断，避免重复修改 #

如果之后有更新时也需要处理金额的场景，可以给回调加个判断，比如只在新记录时执行，或者判断金额的符号是否已经符合要求：

class Transaction < ApplicationRecord
  belongs_to :account
  attr_accessor :trx_type
  before_save do
    # 只有当是新记录且trx_type为debit时，才转换金额
    if self.trx_type == "debit" && self.new_record?
      self.amount = self.amount * -1
    end
  end
end

或者通过判断金额符号避免重复转换：

before_save do
  next unless self.trx_type == "debit"
  # 确保金额是负数，避免重复乘以-1
  self.amount = -self.amount.abs if self.amount > 0
end

方案3：把逻辑移到控制器里处理 #

如果觉得回调的隐式执行不太直观，也可以直接在创建交易的控制器动作里处理金额转换：

class TransactionsController < ApplicationController
  def create
    @transaction = Transaction.new(transaction_params)
    # 创建前直接处理金额
    if @transaction.trx_type == "debit"
      @transaction.amount = -@transaction.amount.abs
    end
    
    if @transaction.save
      redirect_to @transaction, notice: "Transaction created successfully."
    else
      render :new
    end
  end

  private
  def transaction_params
    params.require(:transaction).permit(:account_id, :amount, :trx_type)
  end
end

这样就能彻底解决创建后记录被意外更新导致值不正确的问题啦！

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