有一个函数,完全取决于参数值而采取不同行为。
针对该参数的每一个可能值,建立一个独立函数。
动机:
如果某个参数有多种可能的值,而函数内又以条件表达式检查这些参数值,并根据不同参数值做出不同的行为,那么就应该使用本项重构。如果以参数决定函数行为,那么函数用户不但需要观察该函数,而且还要判断参数值是否合法,而“合法的参数值”往往很少在文档中被清除提出。
就算不考虑编译期检查的好处,只是为了获得一个清晰地接口,也值得执行本想重构。
如果参数值不会对函数行为有太多影响,就不应该使用此重构。
做法:
针对参数的每一种可能值,新建一个明确函数。
修改条件表达式的每个分支,使其调用合适的新函数。
修改每个分支后,编译测试。
修改原函数的每一个被调用点,改而调用上述的某个合适的新函数。
旧代码
static final int ENGINEER = 0;
static final int SALESMAN = 1;
static final int MANAGER = 2;
static Employee create(int type){
switch(type){
case ENGINEER:
return new Engineer();
case SALESMAN:
return new Salesman();
case MANAGER:
return new Manager();
default:
throw new IllegalArgumentException();
}
}
新代码
static employee createEngineer(){
return new Engineer();
}
static employee createSalesman(){
return new Salesman();
}
static employee createManager(){
return new Manager();
}