Rails日历应用：添加用户前验证重复用户的问题求助

嗨，别担心！新手刚接触Rails关联和验证的时候，很容易踩这类小坑，我来帮你一步步解决～

问题根源分析 #

你现在遇到的核心问题是判断用户是否已存在的方法用错了：
你代码里的@calendar.users.includes(@new_user)是Rails的查询预加载方法（用来优化N+1查询），它返回的是一个ActiveRecord集合，永远会是"truthy"值，所以不管用户有没有被添加，这个判断都会走提示已添加的分支。

另外关于<<添加用户的疑问：这个用法是完全正确的！Rails的has_many关联支持用<<直接添加关联对象，它会自动创建中间关联记录（如果是through关联的话），甚至会自动保存，不需要额外调用@calendar.save（当然调用也不会有问题）。

解决方案 #

1. 修正控制器的判断逻辑 #

把检查用户是否存在的方法改成Ruby集合的include?（注意是带问号的版本），这才是用来判断元素是否在集合里的正确方法：

def update
  if @calendar.calendar_admin?(current_user)
    @new_user = User.find(params[:calendar][:user_ids])
    if @calendar.users.include?(@new_user)
      redirect_to user_calendar_path(@calendar), notice: 'This user has already been added to this calendar.'
    else
      @calendar.users << @new_user
      redirect_to user_calendar_path(@calendar), notice: 'Your calendar has been updated.'
    end
  else
    # 补充权限不足的提示，提升用户体验
    redirect_to user_calendar_path(@calendar), alert: 'You are not authorized to update this calendar.'
  end
end

2. 模型层面添加唯一性验证（关键！） #

只在控制器判断还不够，最好从模型层面彻底拦截重复添加的情况，防止因为控制器逻辑漏洞或者直接操作数据库导致的重复。

假设你的日历和用户是通过中间表calendar_users关联的（也就是Calendar模型里有has_many :users, through: :calendar_users），那在CalendarUser模型里添加唯一性验证：

class CalendarUser < ApplicationRecord
  belongs_to :calendar
  belongs_to :user

  # 验证同一个用户不能重复加入同一个日历
  validates :user_id, uniqueness: { 
    scope: :calendar_id, 
    message: "has already been added to this calendar" 
  }
end

3. 可选：添加异常处理，捕获验证失败 #

如果想更健壮，可以在控制器里捕获模型验证失败的异常，这样即使控制器的判断漏了（比如并发场景），也能给出正确提示：

def update
  if @calendar.calendar_admin?(current_user)
    @new_user = User.find(params[:calendar][:user_ids])
    begin
      @calendar.users << @new_user
      redirect_to user_calendar_path(@calendar), notice: 'Your calendar has been updated.'
    rescue ActiveRecord::RecordInvalid => e
      redirect_to user_calendar_path(@calendar), notice: e.message
    end
  else
    redirect_to user_calendar_path(@calendar), alert: 'You are not authorized to update this calendar.'
  end
end

这样一来，不管是控制器提前判断，还是模型层面的验证，都能有效防止重复添加用户的问题啦！

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