Rails中统计优惠券使用次数：报错及增量不生效问题排查

再解决使用次数不递增的问题 #

你现在的代码有两个明显的问题：

1. 控制器index方法里的错误逻辑：你写了@coupons.increment!(:used)，这会导致每次刷新优惠券列表页面，所有优惠券的used字段都会加1，这完全不是"实际使用优惠券时递增"的逻辑，赶紧把这行删掉！

2. 模型递增方法的竞态风险：你原来的increment_used!方法用update(:used => used + 1)，如果同时有多个用户使用同一张优惠券，会出现"竞态条件"——比如两个请求同时读取used=2，然后都加1变成3，最后数据库里只存了3，而不是正确的4。

修正方案：

• 把模型的increment_used!改成Rails内置的原子递增方法，它会直接在数据库层面执行used = used + 1，完全避免竞态问题：

  def increment_used!
    increment!(:used) # 这是Rails专门为数字字段设计的原子递增方法，安全可靠
  end
  

• 确保increment_coupon_used(coupon)这个方法是在用户实际使用优惠券的场景下被调用，比如用户提交订单、验证优惠券有效之后的代码里（比如订单创建的控制器方法），而不是随便在某个页面加载时调用。

另外，你之前试的update_attributes(:used => used +1)和原来的update问题一样，都有竞态风险，不如直接用increment!省心。

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