上次那篇重构-为什么 if-else 不是好代码 说到代码中的 if-else会随着代码量的增加,在迭代的过程中变的越来越难以维护, 然后用工厂模式的思路可以把 if-else代码块给剥离开来, 不过有朋友提出了不足, 虽然表面上看没有 if-else了,但其实只是给挪到工厂里面去。 这不是换汤不换药嘛?
但其实想完全不用 if-else也是可能的,还是以上次那段代码为例子来说, 这是最终在调用端的代码
TargetExecutor executor = ExecutorFactory.getExecutor(target);
executor.process();
这是工厂类的代码
public class ExecutorFactory {
public TargetExecutor getExecutor(String target) {
if (target.startsWith("#RANGE")) {
return new ExecutorWithTag();
} else if (target.contains("#RANGE")) {
....
} else if (target.startsWith("#")) {
....
} else if (target.contains("#")) {
....
} else {
....
}
}
}
我们关注的重点是 getExecutor()这个方法,这里一堆 if-else, 可以看到这里面的逻辑是根据 target 字符串的不同内容实例化不同的 executor对象给调用者使用, 也就是说,这里是一种多对多的模式,对于这种模式我们可以考虑用一个 HashMap来保存键值对,
private static HashMap<String, TargetExecutor> mMappings = new HashMap<>();
而判断 target内容的逻辑也不需要放在工厂类里了,可以重构 TargetExecutor的代码,给它增加一个抽象方法,
public abstract String matchUri();
这个方法的作用就是返回需要跟它匹配的字符串, 比如 ExecutorWithTag,那么它返回的是 "RANGE", 如果是 ExecutorWithStash,那么它返回的是 "#". 我们把新增的代码列出来,
//处理target.startsWith("#RANGE")的情况
public class ExecutorWithTag extends TargetExecutor {
public void process() {
....
}
public String matchUri(){
return "RANGE";
}
}
准备工作好,还记得我们实例化了一个 HashMap吗,以String和Executor作为键值对的, 现在我们需要另外一个方法来把 executor放到 HashMap里,
private static void addExecutor(RouterExeBase executor) {
mMappings.put(executor.matchUri(), executor);
}
static {
addExecutor(new ExecutorWithTag());
addExecutor(new ExecutorWithStash());
....
}
在准备工作做到这里后,我们就需要来把工厂中的 if-else摘除了, 我们把之前的条件判断改成了从一个 map 中遍历查找匹配的模式,虽然从逻辑上来说,遍历查找跟 if-else差不多, 但代码会变更简洁,也不会再看到一长串的 if-else嵌套了, 下面是摘完之后的工厂类的代码,
public class ExecutorFactory {
public TargetExecutor getExecutor(String target) {
for(String pattern : mMappings.keySet()) {
if(target.contains(pattern)) {
return mMappings.get(pattern);
}
}
}
}
这只是一种代码中的小技巧,可以在重构代码的时候让整个代码逻辑清晰很多, 但是也有弊端, 因为需要通过 pattern 去查找匹配,就会有可能出现 pattern 类似或者重叠的情况,一不小心就会导致bug... 对于这种情况上面的重构方法就没那么好了, 所以我习惯的话会把 if-else 剥离到工厂中就结束,但如果涉及到多个模块的人之间的合作的话, 才会再拆分一层,让大家自己把自己的 executor 在静态方法块中注册到 mMapping里, 这样各个模块之前的耦合就能降的很低,大家也就只需要关注自己的功能就行。