关联两个模型Email字段时出现stack level too deep错误求助

错误原因 #

• 你的代码中update操作会触发ActiveRecord的回调（比如after_save、before_update这类钩子），如果模型间的回调存在互相调用逻辑，就会形成无限递归，导致栈溢出。
• 原遍历逻辑效率极低（第一个方法是O(n*m)的时间复杂度），大量的对象实例化和数据库操作也可能加剧栈内存消耗。

重构方案 #

方案一：直接用SQL批量更新（推荐，效率最高） #

绕开ActiveRecord的对象实例化和回调，直接通过SQL完成关联匹配，避免递归问题：

# 1. 批量匹配邮箱，更新author的user_id
Author.connection.execute(<<-SQL)
  UPDATE authors
  SET user_id = users.id
  FROM users
  WHERE authors.email = users.email
SQL

# 2. 给未设置chosen_author_id的用户，匹配对应邮箱的第一个author（取id最小的）
User.connection.execute(<<-SQL)
  UPDATE users
  SET chosen_author_id = authors.id
  FROM authors
  WHERE users.email = authors.email
    AND users.chosen_author_id IS NULL
    AND authors.id = (SELECT MIN(id) FROM authors WHERE authors.email = users.email)
SQL

方案二：用ActiveRecord跳过回调更新 #

如果需要保留部分业务逻辑但避免回调触发，使用update_columns（跳过所有回调和验证）：

# 第一步：构建邮箱到用户ID的映射，减少重复查询
user_email_to_id = User.where.not(email: nil).pluck(:email, :id).to_h

# 批量更新author的user_id
Author.where(email: user_email_to_id.keys).find_each do |author|
  author.update_columns(user_id: user_email_to_id[author.email])
end

# 批量更新用户的chosen_author_id（仅处理未设置的）
User.where(chosen_author_id: nil).find_each do |user|
  author = Author.find_by(email: user.email)
  user.update_columns(chosen_author_id: author.id) if author
end

注意事项 #

• update_columns会直接修改数据库字段，跳过所有ActiveRecord回调、验证和关联缓存刷新，使用前确认不需要这些逻辑。
• 如果必须保留回调，需要检查模型中的钩子代码，确保没有循环调用的逻辑（比如Author更新后触发User修改，User修改又触发Author更新）。

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