DRY（don’t repeat yourself），重复往往是代码腐烂的开始，我们一般的处理手法是将重复的代码提取成一个方法，然后用新方法替换掉原来的代码。
但是对于for循环里面的重复代码要如何处理呢？比如有下面2个方法：
A.class
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
|
public boolean methodA(List<String> list) {
for(String string : list) {
if (string.startWith("A")) {
return true;
}
}
return false;
}
public boolean methodB(List<String> list) {
for(String string : list) {
if (string.endWith("B")) {
return true;
}
}
return false;
}
|
2个方法极其相似，唯一不同的地方只有 if 里面的判断条件。要如何去除这里的重复呢？提取for里面的代码？虽然能去除一部分，但for这个循环体的代码不能一起去掉，而且要是for里面的代码复杂一点，这个方法就不适用了。我今天想介绍自己最近用的比较多的一种重构手法：用模板方法去除for循环的重复代码。
模板方法大家都知道，我可以先建1个抽象类和2个子类要代替这2个方法，但在java1.5+以后，我们可以用enum（枚举）来轻易实现模板方法。重构后的代码如下：
For.class
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
|
enum For {
METHOD_A {
@Override
protected boolean match(String string) {
return string.startsWith("A");
}
}, METHOD_B {
@Override
protected boolean match(String string) {
return string.endsWith("B");
}
};
public boolean method(List<String> list) {
for(String string : list) {
if(match(string)) {
return true;
}
}
return false;
}
protected abstract boolean match(String string);
}
|
这里将相同的代码写成一个公共方法，不同的代码提取成一个抽象方法，让子类去具体实现。这样就实现了相同代码和不同代码的分离了。由于使用了enum，使得我们的创建类的代价为最小，只需一个enum类就搞定整个模板方法。
这里只是抛砖引玉，我相信肯定其他人一定还有更好的方法，希望和大家一起共同探讨。