优先用 if 判断权限,因其可安全处理 null;非空固定角色可用 switch 但需 default;字符串比较用 objects.equals 或 equalsignorecase;权限逻辑应分层,controller 做粗粒度校验,service 处理行级规则;推荐用 enum 替代 string 提升类型安全与可维护性。
权限判断该用 if 还是 switch?
Java 里最直接的权限判断就是 if 链,比如检查用户角色是否为 "ADMIN"、"EDITOR" 或 "VIEWER"。别硬套 switch——JDK 7+ 虽支持字符串 switch,但角色值若来自数据库或配置文件,存在 null 风险,switch 会直接抛 NullPointerException,而 if 可以先判空。
- 优先用
if (role != null && "ADMIN".equals(role)),避免role.equals("ADMIN")的空指针 - 多个固定角色且确定非空时,
switch (role)语义更清晰,但记得加default处理未知角色 - 别在条件里调用可能变状态的方法(如
user.getRole().toUpperCase()),防止重复计算或副作用
如何安全比较用户权限字符串?
权限字段常来自 HTTP 请求头、JWT payload 或数据库查询结果,天然不可信。直接用 == 比较字符串引用必然出错;用 .equals() 前必须确认不为 null;忽略大小写时别用 .toUpperCase(),它依赖系统默认 Locale,中文环境可能异常。
- 统一用
Objects.equals(role, "ADMIN"),自动处理null - 需忽略大小写时,用
"ADMIN".equalsIgnoreCase(role),而非role.toUpperCase().equals("ADMIN") - 从 JWT 解析出的
role字段,建议在解析后立即校验非空并归一化(如转为全大写),后续判断才可放心用==常量
权限逻辑该写在 Controller 还是 Service?
Controller 层只做轻量级准入控制,比如拦截未登录或角色明显越权(如普通用户访问 /api/admin/users);真正涉及数据行级权限(如“只能看自己创建的文章”)或组合规则(“编辑需同时满足角色=EDITOR 且 status=ACTIVE”)必须下沉到 Service 层。