For循环重复代码的重构

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类就搞定整个模板方法。

这里只是抛砖引玉，我相信肯定其他人一定还有更好的方法，希望和大家一起共同探讨。