[干货]如何优雅的判断角色权限,告别if/else

1,153 阅读3分钟

一般来说我们正常的后台管理系统都有所谓的角色的概念,不同管理员权限不一样,能够行使的操作也不一样,比如:

  • 系统管理员(ROLE_ROOT_ADMIN):有 A操作权限

  • 订单管理员(ROLE_ORDER_ADMIN):有 B操作权限

  • 普通用户(ROLE_NORMAL):有 C操作权限

比如一个用户进来,我们需要根据不同用户的角色来判断其有哪些行为,这时候SAO代码出现了:

public class JudgeRole {

    public String judge(String roleName) {
        String result = "";
        if (roleName.equals("ROLE_ROOT_ADMIN")) {
            // 系统管理员有A权限
            result = "ROLE_ROOT_ADMIN: " + "has AAA permission";
        } else if (roleName.equals("ROLE_ORDER_ADMIN")) {
            // 订单管理员有B权限
            result = "ROLE_ORDER_ADMIN: " + "has BBB permission";
        } else if (roleName.equals("ROLE_NORMAL")) {
            // 普通用户有C权限
            result = "ROLE_NORMAL: " + "has CCC permission";
        } else {
            result = "XXX";
        }
        return result;
    }

}

这样当系统里有几十个角色时,那几十个 if/else嵌套可以说是非常酸爽了…… 这样一来非常不优雅,别人阅读起来很费劲;二来则是以后如果再复杂一点,或者想要再加条件的话不好扩展;而且代码一改,以前的老功能肯定还得重测,岂不疯了……

下面简单列出两个优雅的方法:

1.枚举,什么角色能干什么事,这很明显有一个对应关系,所以学过的枚举为啥不用呢?

  • 首先定义一个公用接口RoleOperation,表示不同角色所能做的操作:
public interface RoleOperation {
    String op(); // 表示某个角色可以做哪些op操作
}
  • 接下来我们将不同角色的情况全部交由枚举类来做,定义一个不同角色有不同权限的枚举类RoleEnum
public enum RoleEnum implements RoleOperation {

    // 系统管理员(有A操作权限)
    ROLE_ROOT_ADMIN {
        @Override
        public String op() {
            return "ROLE_ROOT_ADMIN:" + " has AAA permission";
        }
    },

    // 订单管理员(有B操作权限)
    ROLE_ORDER_ADMIN {
        @Override
        public String op() {
            return "ROLE_ORDER_ADMIN:" + " has BBB permission";
        }
    },

    // 普通用户(有C操作权限)
    ROLE_NORMAL {
        @Override
        public String op() {
            return "ROLE_NORMAL:" + " has CCC permission";
        }
    };
}
  • 接下来调用就变得异常简单了,一行代码就行了, if/else也灰飞烟灭了:
public class JudgeRole {

    public String judge(String roleName ) {
        // 一行代码搞定!之前的if/else没了!
        return RoleEnum.valueOf(roleName).op();
    }
    
}

而且这样一来,以后假如我想扩充条件,只需要去枚举类中加代码即可,而不是去改以前的代码,这岂不爽哉!

2.工厂模式,除了用枚举来消除 if/else,工厂模式也可以实现。不同分支做不同的事情,很明显就提供了使用工厂模式的契机,我们只需要将不同情况单独定义好,然后去工厂类里面聚合即可

  • 首先,针对不同的角色,单独定义其业务类:
// 系统管理员(有A操作权限)
public class RootAdminRole implements RoleOperation {

    private String roleName;

    public RootAdminRole(String roleName) {
        this.roleName = roleName;
    }

    @Override
    public String op() {
        return roleName + " has AAA permission";
    }
}

// 订单管理员(有B操作权限)
public class OrderAdminRole implements RoleOperation {

    private String roleName;

    public OrderAdminRole(String roleName) {
        this.roleName = roleName;
    }

    @Override
    public String op() {
        return roleName + " has BBB permission";
    }
}

// 普通用户(有C操作权限)
public class NormalRole implements RoleOperation {

    private String roleName;

    public NormalRole(String roleName ) {
        this.roleName = roleName;
    }

    @Override
    public String op() {
        return roleName + " has CCC permission";
    }
}
  • 接下来再写一个工厂类RoleFactory对上面不同角色进行聚合:
public class RoleFactory {

    static Map<String, RoleOperation> roleOperationMap = new HashMap <>();

    // 在静态块中先把初始化工作全部做完
    static {
        roleOperationMap.put("ROLE_ROOT_ADMIN", new RootAdminRole("ROLE_ROOT_ADMIN"));
        roleOperationMap.put("ROLE_ORDER_ADMIN", new OrderAdminRole("ROLE_ORDER_ADMIN"));
        roleOperationMap.put("ROLE_NORMAL", new NormalRole("ROLE_NORMAL"));
    }

    public static RoleOperation getOp(String roleName ) {
        return roleOperationMap.get(roleName);
    }
}
  • 接下来借助上面这个工厂,业务代码调用也只需一行代码, if/else同样被消除了:
public class JudgeRole {
    public String judge(String roleName) {
        // 一行代码搞定!之前的 if/else也没了!
        return RoleFactory.getOp(roleName).op();
    }
}

这样的话以后想扩展条件也很容易,只需要增加新代码,而不需要动以前的业务代码,非常符合“开闭原则”。