Rails中after_create添加Post观察者时用户重复插入问题排查

我来帮你拆解这个问题——这种重复插入的坑在Rails的HABTM关联里太常见了，主要有这几个核心原因：

1. uniq: true不是用来阻止重复插入的 #

你在has_and_belongs_to_many里加了uniq: true，但这个选项的作用仅对查询生效：当你查询post.watchers时，它会自动帮你去重返回结果，但完全不会阻止你往关联表post_watchers里插入重复的(user_id, post_id)记录。

也就是说，哪怕你已经执行过watchers << author，只要你的回调里没做“是否已存在”的检查，每次触发回调都会往关联表插一条重复数据，只是查的时候看不到而已。

2. 回调可能被意外多次触发 #

虽然这种情况不多见，但也要排查：

• 有没有手动调用了多次after_create相关的方法？
• 有没有用某些gem（比如Devise、PaperTrail）或者自定义逻辑，导致Post的create动作被触发了多次？
• 会不会是在事务里的某些操作，间接触发了回调重复执行？

3. 你代码里的存在性判断不完整 #

从你贴的代码看，!...后面的内容没写完，大概率是原本要写“作者不在观察者列表里”的判断，但没写对。比如如果你的判断是!watchers.include?(author)，但因为HABTM的查询特性，可能这个判断在回调执行时没生效？或者你漏写了这个判断，导致不管作者是否已存在，都执行添加操作。

怎么修复？ #

给你两个靠谱的方案：

• 方案一：在回调里明确做存在性检查，用更高效的数据库查询判断：

  def author_watches_me
    if author.present? && !watchers.exists?(id: author.id)
      watchers << author
    end
  end
  

  这里用watchers.exists?比include?更高效，因为它直接查数据库，而不是把所有watchers加载到内存里。

• 方案二：改用has_many :through关联（推荐长期方案），因为HABTM本身有很多限制，has_many :through更灵活，你可以从数据库层面阻止重复插入：

  1. 生成关联模型：rails generate model PostWatcher post:references user:references
  2. 在PostWatcher模型里加唯一约束：

     class PostWatcher < ApplicationRecord
       belongs_to :post
       belongs_to :user
       validates :post_id, uniqueness: { scope: :user_id }
     end
     

  3. 修改Post模型的关联：

     class Post < ActiveRecord::Base
       after_create :author_watches_me
       has_many :post_watchers
       has_many :watchers, through: :post_watchers, class_name: "User"
       # ...
       private
       def author_watches_me
         watchers << author if author.present?
       end
     

  这样哪怕回调重复触发，数据库的唯一约束也会阻止重复插入，同时validations也会在应用层拦截。

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