社区应用 最新帖子 精华区 社区服务 会员列表 统计排行 社区论坛任务 迷你宠物
  • 2679阅读
  • 0回复

审查Java代码的十一种常见错误

级别: 终身会员
发帖
3743
铜板
8
人品值
493
贡献值
9
交易币
0
好评度
3746
信誉值
0
金币
0
所在楼道
代码审查是消灭Bug最重要的方法之一,这些审查在大多数时候都特别奏效。由于代码审查本身所针对的对象,就是俯瞰整个代码在测试过程中的问题和Bug。并且,代码审查对消除一些特别细节的错误大有裨益,尤其是那些能够容易在阅读代码的时候发现的错误,这些错误往往不容易通过机器上的测试识别出来。本文就常见的Java代码中容易出现的问题提出一些建设性建议,以便您在审查代码的过程中注意到这些常见的细节性错误。 euZ(}+N&  
zD_H yGf  
VD;j[~/Z  
  通常给别人的工作挑错要比找自己的错容易些。别样视角的存在也解释了为什么作者需要编辑,而运动员需要教练的原因。不仅不应当拒绝别人的批评,我们应该欢迎别人来发现并指出我们的编程工作中的不足之处,我们会受益匪浅的。 &>=#w"skb6  
Bcaw~WD  
bF6gBM@*  
S:Xs '0K_  
 正规的代码审查(code inspection)是提高代码质量的最强大的技术之一,代码审查?由同事们寻找代码中的错误?所发现的错误与在测试中所发现的错误不同,因此两者的关系是互补的,而非竞争的。 (Jpm KO  
lPS*-p#IZ  
&7][@v  
/co%:}ln  
  如果审查者能够有意识地寻找特定的错误,而不是靠漫无目的的浏览代码来发现错误,那么代码审查的效果会事半功倍。在这篇文章中,我列出了11个Java编程中常见的错误。你可以把这些错误添加到你的代码审查的检查列表(checklist)中,这样在经过代码审查后,你可以确信你的代码中不再存在这类错误了。 j`9Nwa  
BTs0o&}e  
"_)|8|gN  
#JS`e_3Rr  
  一、常见错误1# :多次拷贝字符串 SsRVd^=;x  
JN^bo(kb  
u+DX$#-n!]  
j |td,82.  
  测试所不能发现的一个错误是生成不可变(immutable)对象的多份拷贝。不可变对象是不可改变的,因此不需要拷贝它。最常用的不可变对象是String。 5B|,S1b  
2FT-}w0;  
AfE%a-;:  
ZY Kd  
  如果你必须改变一个String对象的内容,你应该使用StringBuffer。下面的代码会正常工作: G+C} <S}  
n_;S2KM  
'z](xG<  
1 PIzV:L\  
String s = new String ("Text here"); '>]&rb09|  
`]&*`9IK{  
uQ1jwYK`7  
-$L(y@%X^  
  但是,这段代码性能差,而且没有必要这么复杂。你还可以用以下的方式来重写上面的代码: X 7&U3v  
@ RX`>r{_  
|D(&w+(  
{Y "8~  
String temp = "Text here"; ||fvKyKW>  
String s = new String (temp); Q 3X  
cuMc*i$w!  
&CO| Y(+  
}{=8&gA0  
  但是这段代码包含额外的String,并非完全必要。更好的代码为: /&QQ p3  
WVkG 2  
oek #^:pF  
x/_dW  
String s = "Text here"; oVEAlBm^v  
w>vmF cp  
N6BNzN}-P  
G{O\)gf  
  二、常见错误2#: 没有克隆(clone)返回的对象 8>d q=0:  
O(Td:Zdp  
'2xcce#  
wzbz }P>  
  封装(encapsulation)是面向对象编程的重要概念。不幸的是,Java为不小心打破封装提供了方便??Java允许返回私有数据的引用(reference)。下面的代码揭示了这一点: /pU|ZA.z'2  
F7V6-V{_  
}bCK  
uZ<%kV1B  
import java.awt.Dimension; , | <jjq)  
/***Example class.The x and y values should never*be negative.*/ -[<vYxX:h:  
public class Example{ K+-zY[3  
  private Dimension d = new Dimension (0, 0); &|NZ8:*+#  
  public Example (){ } *@~`d*d  
uYabJqV  
  /*** Set height and width. Both height and width must be nonnegative * or an exception is thrown.*/ _r[r8M B  
  public synchronized void setValues (int height,int width) throws IllegalArgumentException{ O&52o]k5l  
   if (height < 0 || width < 0) cp|:8 [  
    throw new IllegalArgumentException(); [xWEf#', !  
    d.height = height; LVoyA/ F  
     d.width = width; /t$*W\PL@  
  } i<bxc  
Pwg/Vhfh  
  public synchronized Dimension getValues(){ :+<t2^)rD  
   // Ooops! Breaks encapsulation EZ*t$3.T  
   return d; Dl&PL  
  } x g{VP7  
} f~U#z7  
G~`'E&/  
U-1VnX9m  
 c,.0d  
  Example类保证了它所存储的height和width值永远非负数,试图使用setValues()方法来设置负值会触发异常。不幸的是,由于getValues()返回d的引用,而不是d的拷贝,你可以编写如下的破坏性代码: l$=Gvb  
prqT(1  
u*U_7Uw$  
A%P 8c  
Example ex = new Example(); \4/:^T}*  
Dimension d = ex.getValues(); <3)|44.o&  
d.height = -5; k+f1sV[4}  
d.width = -10; t[/\KG8  
y~x#pC*w  
|1lf(\T_  
87+.pM|t%  
  现在,Example对象拥有负值了!如果getValues() 的调用者永远也不设置返回的Dimension对象的width 和height值,那么仅凭测试是不可能检测到这类的错误。 F:M/z#:~  
n$IWoIdbGN  
- *r[  
HE@-uh  
  不幸的是,随着时间的推移,客户代码可能会改变返回的Dimension对象的值,这个时候,追寻错误的根源是件枯燥且费时的事情,尤其是在多线程环境中。 $]nVr(OZ_  
avmcGyL  
]&' jP  
ZMP?'0h=  
  更好的方式是让getValues()返回拷贝: mn(/E/  
FLK"|*A  
?ISI[hoc  
"k/;`eAP  
public synchronized Dimension getValues(){ =!(S<];  
return new Dimension (d.x, d.y); W;q#ZD(;  
} )nJzSN=>$  
1bT' u5&  
]"C| qR*  
YGfA qI y  
  现在,Example对象的内部状态就安全了。调用者可以根据需要改变它所得到的拷贝的状态,但是要修改Example对象的内部状态,必须通过setValues()才可以。 gHp'3SnS  
KBd7|,j  
0&.LBv8  
zoR,RBU6  
  三、常见错误3#:不必要的克隆 CQj/e+eE4  
x`Vy<h 33  
4u@yJ?U  
(6e!09P&  
  我们现在知道了get方法应该返回内部数据对象的拷贝,而不是引用。但是,事情没有绝对: 9qnuR'BDu  
/]pX8 d  
_RN/7\  
) )fDOJ  
/*** Example class.The value should never * be negative.*/ dko[  
public class Example{ ZYrKG+fkl  
  private Integer i = new Integer (0); Z@<q/2).|  
  public Example (){ } aIQrb  
d]^\w'w$  
  /*** Set x. x must be nonnegative* or an exception will be thrown*/ @=02  
  public synchronized void setValues (int x) throws IllegalArgumentException{ yBr$ 0$  
   if (x < 0) Q~x*bMb.  
    throw new IllegalArgumentException(); j@%K*Gb`  
    i = new Integer (x); >|v=Ba6R0  
  } p Z0=  
t^`<*H  
  public synchronized Integer getValue(){ luJ{Iq  
   // We can’t clone Integers so we makea copy this way. We[<BJ o4  
   return new Integer (i.intValue()); |3s.;w K  
  } ,G916J*XA  
} jK& Nkp  
iSnIBs9\  
Kh>?!` lL  
['/;'NhdlY  
  这段代码是安全的,但是就象在错误1#那样,又作了多余的工作。Integer对象,就象String对象那样,一旦被创建就是不可变的。因此,返回内部Integer对象,而不是它的拷贝,也是安全的。 VC/R)%@%  
hdo+Qezu:  
}".\ 4B$n  
; :v]NZtc  
  方法getValue()应该被写为: |U8>:DEl  
6lB{Ao?|  
{KF7j63  
nL 1IS  
public synchronized Integer getValue(){ .t"n]X i  
// ’i’ is immutable, so it is safe to return it instead of a copy. >l7eoj  
return i; P&qy.0  
} I@8+k&nXS  
v]LFZI5  
fs]#/*RR  
.d<~a1k  
  Java程序比C++程序包含更多的不可变对象。JDK 所提供的若干不可变类包括: P58\+9d_  
jrDz7AfA  
rU/-Wq`B  
4v rm&k  
  ?Boolean #R~">g:w  
   ?Byte g_3rEvf"4  
   ?Character O JZ!|J8?  
   ?Class u.ULS3`C/X  
   ?Double f]@[4<Ny  
   ?Float !Ei Ze.K  
   ?Integer AlPL;^Y_l  
   ?Long  9'L1KQ  
   ?Short ;g;,%jdCS  
   ?String 31 KDeFg  
   ?大部分的Exception的子类 Ri^sQ<~(  
nOA ,x  
~$ cm9>  
5#9`ROT9  
 四、常见错误4# :自编代码来拷贝数组 o+)m}'T8  
VZ9e~){xA  
(E2lv#[  
ZwY`x')  
  Java允许你克隆数组,但是开发者通常会错误地编写如下的代码,问题在于如下的循环用三行做的事情,如果采用Object的clone方法用一行就可以完成: m? \#vw$  
G#_(7X&  
:epitpJ  
_*OaiEL+:  
public class Example{ *@b~f&Lx6  
  private int[] copy; "6|'& 6&  
  /*** Save a copy of ’data’. ’data’ cannot be null.*/ 7v4-hfN  
  public void saveCopy (int[] data){ Jgi{7J  
   copy = new int[data.length]; Z7K!"I  
   for (int i = 0; i < copy.length; ++i) ^*$WZMMJ1  
    copy = data; qiwQUm{  
  } 'ugR!o1  
} BP7<^`i&  
yKX:Z4I/  
vZ1D3ytfG  
s5_1}KKCs  
  这段代码是正确的,但却不必要地复杂。saveCopy()的一个更好的实现是: ^^j|0qshL  
J8`1V `$  
QrrZF.  
OI;L9\MJc  
void saveCopy (int[] data){ g%<{G/Tz  
  try{ <uWJ>sg^ 6  
   copy = (int[])data.clone(); ~@a) E+LsF  
  }catch (CloneNotSupportedException e){ W2X+N acD  
   // Can’t get here. }[hDg6i  
  } DbPBgD>Q  
} r&j+;JM5  
 ^qSf  
qB` 0^V  
qqO10~Xc  
  如果你经常克隆数组,编写如下的一个工具方法会是个好主意: 8&`T<ECq>  
v]d?6g  
I%VV4,I&pK  
b{yH4)O  
static int[] cloneArray (int[] data){ U3VsMV*Y  
  try{ N?`GZ+5  
   return(int[])data.clone(); //4p1^%  
  }catch(CloneNotSupportedException e){ `"bRjC"f]  
   // Can’t get here. B4M'Er{v  
  } DI"dY ug#  
} 4F 6ju6w  
/M{)k_V  
7\Yq]:;O  
&`\kb2uep  
  这样的话,我们的saveCopy看起来就更简洁了: l#J>It\  
$D2Ain1  
<iY 9cV|}3  
@/ovdf{  
void saveCopy (int[] data){ [3bwbfHhi  
  copy = cloneArray ( data); ~k:>Xo[|O  
} ,M9hb<:m  
,_4 KyLfBF  
+$pO  
O+3D 5*  
  五、常见错误5#:拷贝错误的数据 (t"YoWA#m  
C9^elcdv  
u.xA}yVS  
3`C3+  
  有时候程序员知道必须返回一个拷贝,但是却不小心拷贝了错误的数据。由于仅仅做了部分的数据拷贝工作,下面的代码与程序员的意图有偏差: }~0{1&  
y|2g"J  
iR4,$Nn>  
R.n`R|NOd  
import java.awt.Dimension; 5Dh&ez`oR'  
/*** Example class. The height and width values should never * be $(<*pU  
negative. */ -^SD6l$  
public class Example{ s$=B~l  
  static final public int TOTAL_VALUES = 10; fjeE.  
  private Dimension[] d = new Dimension[TOTAL_VALUES]; E rRMiT  
  public Example (){ } a} Iz  
D-;43>yi<  
  /*** Set height and width. Both height and width must be nonnegative * or an exception will be thrown. */ ='l6&3X  
  public synchronized void setValues (int index, int height, int width) throws IllegalArgumentException{ E`Zh\u)  
   if (height < 0 || width < 0) 5E!|on  
    throw new IllegalArgumentException(); gs2&0rnOy\  
    if (d[index] == null) &`9bGO  
     d[index] = new Dimension(); C J}4V!;|  
     d[index].height = height; =*O9)$b  
     d[index].width = width; O'?lW~CD.>  
  } j(2tbWg9-  
  public synchronized Dimension[] getValues() oU{-B$w  
   throws CloneNotSupportedException{ 8i+jFSZ$  
    return (Dimension[])d.clone(); C^ k3*N  
  } e1Z;\U$&.  
} # xE>]U  
s9)8{z  
hrtN.4p[  
%>QSeX  
  这儿的问题在于getValues()方法仅仅克隆了数组,而没有克隆数组中包含的Dimension对象,因此,虽然调用者无法改变内部的数组使其元素指向不同的Dimension对象,但是调用者却可以改变内部的数组元素(也就是Dimension对象)的内容。方法getValues()的更好版本为: e[Ul"pMvS`  
l=.InSuLT  
DyV[+P  
(j\UoKLRt  
public synchronized Dimension[] getValues() throws CloneNotSupportedException{ TTjjyZ@  
  Dimension[] copy = (Dimension[])d.clone(); _M[[o5{  
  for (int i = 0; i < copy.length; ++i){ (>/Dw|,m  
   // NOTE: Dimension isn’t cloneable. r;s3(@[,@  
   if (d != null) ~o\]K  
    copy = new Dimension (d.height, d.width); WW Kr & )  
  } }N=zn7W  
  return copy; I5AjEp  
} jq]\oY8y  
]{l O  
4?6'~G$k  
\}_7^)S;  
  在克隆原子类型数据的多维数组的时候,也会犯类似的错误。原子类型包括int,float等。简单的克隆int型的一维数组是正确的,如下所示: L``mF(R^  
dYttse'  
dXr !_)i  
@C6DOB  
public void store (int[] data) throws CloneNotSupportedException{ ?%TM7Z4  
  this.data = (int[])data.clone(); - &LZle&M  
  // OK I5 7<0  
} K%~Kg9  
{s^n|b}  
So0,)  
W!Os ci  
  拷贝int型的二维数组更复杂些。Java没有int型的二维数组,因此一个int型的二维数组实际上是一个这样的一维数组:它的类型为int[]。简单的克隆int[][]型的数组会犯与上面例子中getValues()方法第一版本同样的错误,因此应该避免这么做。下面的例子演示了在克隆int型二维数组时错误的和正确的做法: kO O~%|1CP  
O#ajoE  
N,'qMoNf  
( ]uoN4  
public void wrongStore (int[][] data) throws CloneNotSupportedException{ ;{#M  
  this.data = (int[][])data.clone(); // Not OK! /t2 <OU9  
} 4rCqN.J  
public void rightStore (int[][] data){ X\:(8C;+  
  // OK! Bk(XJAjY  
  this.data = (int[][])data.clone(); dXy"yQ>{  
  for (int i = 0; i < data.length; ++i){ &ppZRdq]  
   if (data != null) Pn){xfqDl  
    this.data = (int[])data.clone(); t7& GCZ  
  } oML K!]a  
} D}C*8s bC}  
C'#)bX{  
6j.(l4}  
o]k]pNO  
2H0q\zZ  
   六、常见错误6#:检查new 操作的结果是否为null "VhrsVT  
z[I/ AORl  
,}$x'8v  
%1l80Z  
  Java编程新手有时候会检查new操作的结果是否为null。可能的检查代码为: st^N QL  
UVi/Be#|  
9(\N+  
I;PO$T  
Integer i = new Integer (400); <. ]&FPJ  
if (i == null) BwA~*5TFu  
throw new NullPointerException(); <i @jD  
LWR &(p.%  
{ca^yHgGy  
&_TjRj"  
  检查当然没什么错误,但却不必要,if和throw这两行代码完全是浪费,他们的唯一功用是让整个程序更臃肿,运行更慢。 Q#AHEm{9;s  
M(gWd8?#  
)Syf5I  
G\+MT(&5  
  C/C++程序员在开始写java程序的时候常常会这么做,这是由于检查C中malloc()的返回结果是必要的,不这样做就可能产生错误。检查C++中new操作的结果可能是一个好的编程行为,这依赖于异常是否被使能(许多编译器允许异常被禁止,在这种情况下new操作失败就会返回null)。在java 中,new 操作不允许返回null,如果真的返回null,很可能是虚拟机崩溃了,这时候即便检查返回结果也无济于事。 =*G'.D /*  
c35vjYQx0  
 七、常见错误7#:用== 替代.equals <Gt{(is  
|L#r)$n{1  
  在Java中,有两种方式检查两个数据是否相等:通过使用==操作符,或者使用所有对象都实现的.equals方法。原子类型(int, flosat, char 等)不是对象,因此他们只能使用==操作符,如下所示: 6aK2 {-+  
tWy<9TF  
'cCj@bZ9X  
[WSIC *|;  
int x = 4; m|?J^_  
int y = 5; mAERZ<I  
if (x == y) T[II;[EiE  
  System.out.println ("Hi"); :9< r(22  
// This ’if’ test won’t compile. <J uJ`t  
if (x.equals (y)) 6R,b 8  
  System.out.println ("Hi"); YuuG:Kk  
"+C\f)  
--hnv/AjI  
mh SsOmJ5  
  对象更复杂些,==操作符检查两个引用是否指向同一个对象,而equals方法则实现更专门的相等性检查。 vWga>IGM  
LU=)\U@Q  
f*@:{2I.v  
Z1}zf( JU  
  更显得混乱的是由java.lang.Object 所提供的缺省的equals方法的实现使用==来简单的判断被比较的两个对象是否为同一个。 ooxzM `  
_^A NJ7  
YR`rg;n#  
F#R\Ot,hv  
  许多类覆盖了缺省的equals方法以便更有用些,比如String类,它的equals方法检查两个String对象是否包含同样的字符串,而Integer的equals方法检查所包含的int值是否相等。 Z0@ImhejuB  
]@g$<&  
h2*&>Mc  
?Gu>!7  
  大部分时候,在检查两个对象是否相等的时候你应该使用equals方法,而对于原子类型的数据,你用该使用==操作符。 =)>q.R9  
3`!KndY1  
fN>|X\-  
C\h<02  
  八、常见错误8#: 混淆原子操作和非原子操作 )}lV41u  
Gi2Ey37]O  
O/~^}8TLL  
=&xoyF  
  Java保证读和写32位数或者更小的值是原子操作,也就是说可以在一步完成,因而不可能被打断,因此这样的读和写不需要同步。以下的代码是线程安全(thread safe)的: <08V-   
(KU@hp-\  
0u9h2/ma  
BGjTa.&  
public class Example{ |ZzBCL8q  
  private int value; // More code here... nA j2k  
  public void set (int x){ B)NB6dCp  
   // NOTE: No synchronized keyword eF\C?4  
   this.value = x; J4X35H=Z  
  } jzw?V9Ijb  
} U /Fomu  
VG7#6)sQoK  
EF~PM  
pdu  
  不过,这个保证仅限于读和写,下面的代码不是线程安全的: ' qVa/GJ  
,6L>f.V^(U  
e@By@r&nql  
%j; cXN  
public void increment (){ G-<~I#k  
  // This is effectively two or three instructions: aC` c^'5  
  // 1) Read current setting of ’value’. v Rs5-T  
  // 2) Increment that setting. m$g^On  
  // 3) Write the new setting back. C_)>VPD  
  ++this.value; iB-s*b<`~  
} (iP,YKG1?  
>4b39/BM  
z5/O8}Gz@  
</p.OaNe  
  在测试的时候,你可能不会捕获到这个错误。首先,测试与线程有关的错误是很难的,而且很耗时间。其次,在有些机器上,这些代码可能会被翻译成一条指令,因此工作正常,只有当在其它的虚拟机上测试的时候这个错误才可能显现。因此最好在开始的时候就正确地同步代码: \]El%j4  
Z;s-t\C  
g&wQ^  
v,B\+q/  
public synchronized void increment (){ _Y=yR2O  
  ++this.value; mAa]E t.  
} kMXl {  
s9>!^MzBK  
S#dS5OX  
}IL@j A  
  九、常见错误9#:在catch 块中作清除工作 Awh)@iTL  
m ws.)  
A@r,A?(  
J>35q'nN]F  
  一段在catch块中作清除工作的代码如下所示: xcA:Q`c.{  
D$;/ l}s?  
E .5xzY  
}fZBP]<I(  
OutputStream os = null; UJ:B:hh''  
try{  j C?  
  os = new OutputStream (); (0S7  
  // Do something with os here. rJ>8|K[kt  
  os.close(); f6)H!SI  
}catch (Exception e){ ^Du_e(TiyK  
  if (os != null) |zKcL3*  
  os.close(); 6np wu5!  
} 7&1: ]{_  
EK_^#b  
sP%.o7&n  
>rubMGb  
  尽管这段代码在几个方面都是有问题的,但是在测试中很容易漏掉这个错误。下面列出了这段代码所存在的三个问题: +l(}5(wc  
3OlY Ml  
.M lE1n'  
Z)%p,DiNM  
  1.语句os.close()在两处出现,多此一举,而且会带来维护方面的麻烦。 e`^j_V nEH  
|~Iw   
AP%h!b5v  
";]m]PRAam  
  2.上面的代码仅仅处理了Exception,而没有涉及到Error。但是当try块运行出现了Error,流也应该被关闭。 QTH yH   
QR4o j  
,fn=%tiUk  
; ]!  
  3.close()可能会抛出异常。 _NFJm(X.  
Pif1sL6'  
+8M{y D9#  
xmDwoLU  
  上面代码的一个更优版本为: m`~ Qr~  
&0ra a  
FmPF7  
H'2 =yhtVh  
OutputStream os = null; ^E^:=Q?'_  
try{ $ }53f'QjW  
  os = new OutputStream (); al/~  
  // Do something with os here. c@`P{ 6  
}finally{ Wj&s5;2a  
  if (os != null) &n|gPp77$  
   os.close(); *O~D lf  
} jk@]d5  
d<o  
& BkNkb0  
7+"X ^$  
  这个版本消除了上面所提到的两个问题:代码不再重复,Error也可以被正确处理了。但是没有好的方法来处理第三个问题,也许最好的方法是把close()语句单独放在一个try/catch块中。 .kVga+la?  
aO1cd_d6x_  
eR CGr?e4  
sp%7iNs  
  十、常见错误10#: 增加不必要的catch 块 ^^m%[$nw&r  
)'JSu=Ej  
SDkN  
l-&f81W  
  一些开发者听到try/catch块这个名字后,就会想当然的以为所有的try块必须要有与之匹配的catch块。 ` Oi@7 /oT  
7_RU*U^  
#p]O n87>  
(_* a4xGF  
  C++程序员尤其是会这样想,因为在C++中不存在finally块的概念,而且try块存在的唯一理由只不过是为了与catch块相配对。 dn Xc- <  
zv41Yv!x}  
ee0J;pP2#  
/bWV `*  
  增加不必要的catch块的代码就象下面的样子,捕获到的异常又立即被抛出: !E%!,  
08Q:1 '  
[kCn6\_<V  
p: o*=  
try{ ;(V=disU/  
  // Nifty code here tc[PJH&P  
}catch(Exception e){ k(MQ:9'|  
  throw e; W6pS.}  
}finally{ x\*5A,w{c]  
  // Cleanup code here -Je+7#P1  
} rP'oU V_  
&+\wYa,  
;(XSw%Y H  
G 6][@q  
  不必要的catch块被删除后,上面的代码就缩短为: z, OMR`W  
.}tL:^'~o  
7 ir T6O<.  
<,m}TTq  
try{ o` e~1  
  // Nifty code here og>f1NwS[  
}finally{ \R~Lf+q  
  // Cleanup code here /F\7_  
} KT3[{lr  
$1axZ~8sS  
eA!aUu  
,yB-jk?  
  常见错误11#;没有正确实现equals,hashCode,或者clone 等方法 2]@U$E='s  
f~]5A%=cZ  
WYq, i}S  
\UXQy{Ex  
  方法equals,hashCode,和clone 由java.lang.Object提供的缺省实现是正确的。不幸地是,这些缺省实现在大部分时候毫无用处,因此许多类覆盖其中的若干个方法以提供更有用的功能。但是,问题又来了,当继承一个覆盖了若干个这些方法的父类的时候,子类通常也需要覆盖这些方法。在进行代码审查时,应该确保如果父类实现了equals,hashCode,或者clone等方法,那么子类也必须正确。正确的实现equals,hashCode,和clone需要一些技巧。 B$\5=[U  
!l'Zar  
DU;]Q:r{  
A) qOJ(OEz  
  小结 '8dqJ`Gj  
KmMt:^9  
8J)x>6  
D,NjDIG8  
  我在代码审查的时候至少遇到过一次这些错误,我自己也犯过其中的几个错误。好消息是只要你知道你在找什么错误,那么代码审查就很容易管理,错误也很容易被发现和修改。即便你找不到时间来进行正规的代码审查,以自审的方式把这些错误从你的代码中根除会大大节省你的调试时间。花时间在代码审查上是值得的。 pD){K  
zNB G;\ W  
 
评价一下你浏览此帖子的感受

精彩

感动

搞笑

开心

愤怒

无聊

灌水
描述
快速回复

您目前还是游客,请 登录注册
如果您在写长篇帖子又不马上发表,建议存为草稿
认证码:
验证问题:
3+5=?,请输入中文答案:八 正确答案:八