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