多条件校验失败：满足任一条件重定向至root_path的Ruby方法问题咨询

核心问题 #

原代码的校验逻辑本身和你给出的需求是匹配的，但存在两个明显缺陷：

• 代码冗余：重定向和提示文案的逻辑重复写了两次，后续维护需要同步修改多处
• 嵌套if判断降低了代码可读性，容易出错
  如果你的校验结果和预期不符，大概率是需求表述时的逻辑运算符混淆，我们会同时给出两种常见场景的修复方案。

修复代码 #

场景1：严格匹配你给出的需求（用户必须同时是admin和professional才能访问） #

只要满足「用户不存在」/「不是admin」/「不是professional」任一条件就重定向：

def check_user_pro!
  if @current_user.blank? || !@current_user.is_admin || !@current_user.profile.professional?
    redirect_to root_path, alert: "Reserved"
  end
end

场景2：常见的权限需求（用户只要是admin或professional其中一个就可以访问） #

只有满足「用户不存在」/「既不是admin也不是professional」任一条件才重定向：

def check_user_pro!
  if @current_user.blank? || (!@current_user.is_admin && !@current_user.profile.professional?)
    redirect_to root_path, alert: "Reserved"
  end
end

优化说明 #

• 用blank?方法直接判断用户不存在的情况，简化逻辑
• 将重复的重定向逻辑合并，后续修改只需调整一次
• 直接在redirect_to方法中传入alert参数，简化flash赋值操作

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