Java常见缺陷模式

Java常见缺陷模式

---

指出下列各代码片段存在的缺陷，陈述你的理由并修改缺陷（共15题）

1.字符串相等问题

private void bug_method(String str){
    if ( str == "Red")
        dosthRed();
    else
        dosthBlack();
}

错误：str==”Red”

改正：str.equals(“Red”)

2.相等判断错误

SomeObject cmpObj = new SomeObject();
private boolean bug_method(SomeObject obj1){
    //want to compare if the two object's value is same   
    return obj1 == cmpObj;
}

错误：return obj1 == cmpObj

改正：代码想要比较两个对象是否相等并返回结果，但是直接采用了==运算符，==运算符是比较两个对象地址是否相等，理论上，只要obj1与cmpObj不是指向同一个堆中new对象那么return一定会返回false。这并不是我们想要的结果。在这道题目中说讲两个对象比较，说明这两个对象是有比较的概念的，所以应该实现该类继承自Object的equals方法。采用obj1.equals(cmpObj)的方法进行比较。

3.浮点数精度问题

public static double bugMethod(Func f, double x1, double x2) {
    double x = x1;
    double result = 0;
    double step = (x2 - x1) / 700;
    while (x != x2) { 
        result = result + f.valueFor(x) * step;
        x = x + step;
    }
    return result;
}

错误：while(x!=x2)

改正：浮点数精度有限，在step=(x2 - x1) / 700;这一步中很有可能不能整除导致step是一个很多小数位的浮点数，在此情况下用x+step更新的到的x值的精度大于或小于x2，导致循环不能正确结束，形成死循环，在代码中应该尽量避免用double做判断循环的条件。

4.&与&&区别

private void bugMethod(int arr[]){
    if((arr!=null) & (arr