Rails 6+Ruby 2.7下ActiveRecord更新回调未执行计算问题

• 循环触发的本质：在当前记录的回调内部反复调用update修改同一条记录，每一次update都会重新拉起全链路更新回调，只要complete没被持久化到数据库，判断分支就会反复命中，形成无限递归。
• 更新逻辑不执行的直接原因：before_update_calculations的判断逻辑完全反向：代码写的是unless Sheet.find(self.id).complete == true，意思是“如果记录不是complete状态，就把它改成非complete”。实际场景里更新前的记录本来就是complete: true，这个分支根本不会进入；等走到after_update的计算逻辑时，complete字段还是数据库里存的true，所有sheet.complete != true的判断全部不成立，计算逻辑直接跳过。
• 状态错位问题：代码里反复通过Sheet.find(self.id)、Store.find(self.store_id)重新查库，拿到的对象和当前回调上下文的对象不是同一个实例，状态判断很容易因为读写时序错位出错。
• 非必要的性能浪费：单条记录的保存拆成了5-6次零散的update调用，每一次update都要走一遍数据库查询、校验、回调流程，既慢又容易触发递归。

• 从根上打断递归：如果需要在回调内修改当前记录的字段，不要用默认会触发回调的update方法，改用update_columns直接写库，跳过后续的回调链路，不会再触发无限循环。
• 抽离公共计算逻辑：把创建、更新场景下重复的计算代码抽成同一个私有方法，避免两边逻辑不一致的问题。
• 修正触发判断逻辑：不要在before_update里硬改complete字段，只要把所有计算需要的字段一次性算完、批量赋值后一次性保存，不需要拆成多次零散更新。
• 补全空值兜底：对查询关联得到的past_sheet这类可能为空的对象，提前判断，避免第一条记录没有上序数据时触发方法调用错误。

belongs_to :user
belongs_to :store
after_create :run_calculations
after_update :run_calculations, if: :need_recalculate?

private

def run_calculations
  # 空值兜底：识别当前是否为店铺下第一张表单
  is_first_sheet = id == store.sheets.minimum(:id)
  past_sheet = is_first_sheet ? nil : store.sheets.second_to_last

  # 计算销售额
  current_sold = if past_sheet
                   past_sheet.draw - returned
                 else
                   sold
                 end

  # 按店铺售货机类型匹配对应系数
  rate = store.vending_box? ? 0.75 : 0.55
  current_shortage = (current_sold * rate) - collected
  current_pilferage = current_shortage / rate

  # 直接写库跳过回调，避免循环
  update_columns(
    complete: true,
    sold: current_sold,
    shortage: current_shortage,
    pilferage: current_pilferage
  )
end

def need_recalculate?
  # 仅当和计算相关的字段发生变更时才触发重算，减少不必要的执行
  saved_change_to_returned? || saved_change_to_collected? || saved_change_to_store_id?
end

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