Java 最喜欢的重构技术

Question

我在其他人的 Java 代码中发现了一些常见的代码模式，可以从一些简单的重构中受益。

您讨厌的代码模式及其修复方法是什么（以及原因（如果不明显的话））？

我冒昧地回答了一些我自己最讨厌的事情。

Answer 1

我最喜欢的重构之一是使用策略模式而不是长 if-else/switch 语句。
例如。

String chooser = ""//some sting

if(testCond1(chooser)){
doSomething1();
} else if(testCond2(chooser)){
doSomethingElse2();
} else if(testCond2(chooser)){
doSomethingElse3();
} else if(testCond4(chooser)){
doSomethingElse4();
} else if(testCond5(chooser)){
doSomethingElse5();
} else if(testCond6(chooser)){
doSomethingElse6();
}

可以改为：

    Map<String, Handler> handlers = new HashMap<String, Handler>();

handlers.get(chooser).handle();

然后我们定义一个Handler接口

interface Handler{
    handle();
}

，并且对于每个条件我们都有一个实现该处理程序的新类。

class CondOne implements Handler{
    handle(){
        //some code
    }
}

优点。面向对象的方法，代码更易于维护。在不更改代码重要部分的情况下添加新条件也很容易。

Answer 2

boolean someMethod() {
    if (<some test>) {
        return true;
    } else {
        return false;
    }
}

替换为

boolean someMethod() {
    return <some test>;
}

指导原则/模式：

• 更少的代码是好的，
• 冗余的代码是坏的

Answer 3

void someMethod(SomeClass param) {
    if (param != null) { // or some other test
        // Rest of method code
    }
}

替换为：

void someMethod(SomeClass param) {
    if (param == null) { // or some other test
        return; // or throw exception if test expected to "always" pass
    }
    // Rest of method code
}

指导原则/模式：

• 缩进
• 如果块很好，则
• 越少越好，越短参数检查应该尽早完成并在出现问题时退出，而不是允许代码在传递时执行

Answer 4

if (<some test>) {
    return someObject;
} else {
    return someOtherObject;
}

替换为：

if (<some test>) {
    return someObject;
}

return someOtherObject;

或者为了更简洁，如果该行不太长（即您没有内联创建对象）：

return <some test> ? someObject : someOtherObject;

指导原则/模式：

• 更少的代码是好的，
• 冗余的代码是坏的 - 当 if 返回时，不需要“其他”，
• 缩进少一点就好

Answer 5

我喜欢枚举并尝试通过大量字符串检查来重构代码。

if (str.equals("A") {...}
else if (str.equals("B") {...}
else if (str.equals("C") {...}

到

switch (str){
 case A: ... ; break;
 case B: ... ; break;
 case C: ... ; break;

}

Answer 6

我经常看到这样的代码，但不应该比较引用，而是应该比较内容。

if (someObject == anotherObject) {
   doSomething();
}

所以我将其替换为（如果需要的话覆盖 equals() 和 hashCode()）：

if (someObject.equals(anotherObject))  {
   doSomething();
}