Rails模型中修改status为no时清除table_number值失败问题

你的代码存在三个关键问题，导致功能未生效且无报错：

• 条件判断误用赋值运算符：if self.status = "no"里的=是赋值操作，会强制把status设为"no"，而非判断当前值是否为"no"，逻辑完全错误。
• 回调时机选择错误：after_commit是数据库事务提交后触发的回调，此时修改self.table_number仅会改变内存中的对象属性，不会自动同步到数据库，所以看不到实际效果。
• 回调条件不够精准：status_changed?会在status发生任何变化时触发方法，但你只需要在status改为"no"时执行逻辑，多余触发会造成不必要的资源消耗。

修复后的代码 #

class Rsvp < ApplicationRecord
  # 仅当status改为"no"时，在更新前执行清除操作
  before_update :remove_table_assignment, if: -> { status_changed? && status == "no" }

  def remove_table_assignment
    self.table_number = nil
  end
end

修复说明 #

1. 替换回调时机：用before_update替代after_commit，在记录更新到数据库前执行方法，修改后的table_number会被自动保存，无需额外调用save方法。
2. 精准限定触发条件：通过lambda表达式，仅在status发生变化且新值为"no"时触发方法，避免无效逻辑执行。
3. 简化方法逻辑：由于回调条件已经确保当前status为"no"，方法内可直接赋值nil，无需重复判断。

如果需要支持创建记录时（比如初始设置status为"no"）也清除table_number，将before_update替换为before_save即可。

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