Rails 5：如何基于DRY原则精简多条件判断代码

嘿，我懂这种想精简代码却踩坑的滋味！多用户类型的内容控制确实容易在ApplicationController里出问题，我先给你梳理几个常见的坑和正确的实现思路，你可以对照看看自己的问题在哪：

1. 用户类型判断逻辑不严谨 #

很多时候出错是因为判断条件写得不够精准，比如用了if current_user.type == 'Admin'，但实际用户类型的存储方式是枚举或者单表继承（STI），这时候判断逻辑就会失效。

举两个常见场景的正确写法：

• 如果是单表继承（STI）（比如User有AdminUser、RegularUser子类）：

# 在ApplicationController里
def admin_user?
  current_user.is_a?(AdminUser)
end
helper_method :admin_user? # 必须加这个，视图才能直接调用

def regular_user?
  current_user.is_a?(RegularUser)
end
helper_method :regular_user?

• 如果是枚举字段（比如user表有role字段，值为admin/regular）：

def admin_user?
  current_user&.role == 'admin'
end
helper_method :admin_user?

这里的&.是关键，能避免current_user为nil时抛出NoMethodError，这是超级高频的踩坑点！

2. 忘记把方法暴露给视图 #

如果在ApplicationController里定义了判断方法，但没加helper_method :方法名，视图里调用的时候就会报错“方法不存在”，这个细节很容易被忽略。

3. 复杂权限逻辑导致的混乱 #

如果你的用户类型不止两种，或者有嵌套权限（比如管理员下还有超级管理员），直接在ApplicationController堆判断逻辑会越写越乱，容易出bug。这时候推荐用Pundit或者CanCanCan这类权限管理gem，把权限逻辑抽离到单独的Policy类里，代码更清晰也更易维护。

比如用Pundit的话，流程大概是这样：

# ApplicationController
include Pundit
after_action :verify_authorized, except: :index # 可选，强制权限校验

然后创建app/policies/user_policy.rb，把所有权限判断逻辑放在这里：

class UserPolicy < ApplicationPolicy
  def can_view_dashboard?
    user.role == 'admin' || user.role == 'editor'
  end

  def can_edit_content?
    user.role == 'admin'
  end
end

之后在视图里就能直接用policy(current_user).can_view_dashboard?来判断内容是否显示，控制器里用authorize current_user做权限校验，彻底把逻辑从ApplicationController里解放出来。

4. before_action执行顺序错误 #

如果你用before_action来初始化用户权限相关的实例变量，一定要注意执行顺序——必须先执行用户认证的方法（比如devise的authenticate_user!），再执行权限初始化，不然current_user还没赋值就会报错。

正确的写法示例：

# ApplicationController
before_action :authenticate_user!
before_action :set_user_permissions

private

def set_user_permissions
  @is_admin = current_user.admin?
  @is_editor = current_user.editor?
end

如果你能把具体的错误日志（比如控制台的报错信息）和你写的ApplicationController代码片段贴出来，我能更精准地帮你定位问题！

内容的提问来源于stack exchange，提问作者Bao Lam Nguyen