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

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

级别: 终身会员
发帖
3743
铜板
8
人品值
493
贡献值
9
交易币
0
好评度
3746
信誉值
0
金币
0
所在楼道
代码审查是消灭Bug最重要的方法之一,这些审查在大多数时候都特别奏效。由于代码审查本身所针对的对象,就是俯瞰整个代码在测试过程中的问题和Bug。并且,代码审查对消除一些特别细节的错误大有裨益,尤其是那些能够容易在阅读代码的时候发现的错误,这些错误往往不容易通过机器上的测试识别出来。本文就常见的Java代码中容易出现的问题提出一些建设性建议,以便您在审查代码的过程中注意到这些常见的细节性错误。 E}eu]2=nU}  
yt,xA;g  
Br w-"tmx  
  通常给别人的工作挑错要比找自己的错容易些。别样视角的存在也解释了为什么作者需要编辑,而运动员需要教练的原因。不仅不应当拒绝别人的批评,我们应该欢迎别人来发现并指出我们的编程工作中的不足之处,我们会受益匪浅的。 lq0@)'D  
Y rq-(  
%qG nvQ  
i,HafY  
 正规的代码审查(code inspection)是提高代码质量的最强大的技术之一,代码审查?由同事们寻找代码中的错误?所发现的错误与在测试中所发现的错误不同,因此两者的关系是互补的,而非竞争的。 ygt7;};!  
cQkH4>C~  
awP ']iE  
4o7(cP  
  如果审查者能够有意识地寻找特定的错误,而不是靠漫无目的的浏览代码来发现错误,那么代码审查的效果会事半功倍。在这篇文章中,我列出了11个Java编程中常见的错误。你可以把这些错误添加到你的代码审查的检查列表(checklist)中,这样在经过代码审查后,你可以确信你的代码中不再存在这类错误了。 bvHF;Qywg  
EB8=*B8  
>y&Db  
f-6hcd@Ca  
  一、常见错误1# :多次拷贝字符串 E`vCYhf{  
 d+FS  
,_HSvs7-  
E/x2LYH  
  测试所不能发现的一个错误是生成不可变(immutable)对象的多份拷贝。不可变对象是不可改变的,因此不需要拷贝它。最常用的不可变对象是String。 (`S32,=TS  
V %k #M  
Z"spua5  
tbz?th\#  
  如果你必须改变一个String对象的内容,你应该使用StringBuffer。下面的代码会正常工作: r![RRa^  
j2GO ZKy  
J:6wFmU  
]fc9m~0N,\  
String s = new String ("Text here"); QLNQE6-  
r$3{1HXc  
+ 3+^J?N  
#i$/qk= N  
  但是,这段代码性能差,而且没有必要这么复杂。你还可以用以下的方式来重写上面的代码: R7~H}>uaF  
E]G#"EV!Y  
?UD2}D[M  
k-5Enbkr  
String temp = "Text here"; 0*?/s\>PS;  
String s = new String (temp); EW;R^?Z  
a.P7O!2Lp  
}T<[JXh=J  
);4lM%]eb  
  但是这段代码包含额外的String,并非完全必要。更好的代码为: r>v_NKS]t  
eq^<5 f  
_TF\y@hF*D  
t;wfp>El  
String s = "Text here"; X\X* -.]{  
GLI 5AbQK  
h\+U+ ?u  
oK cgP  
  二、常见错误2#: 没有克隆(clone)返回的对象 Rw4"co6  
u=a5Z4N'  
?6h65GO{  
6{2LV&T=u  
  封装(encapsulation)是面向对象编程的重要概念。不幸的是,Java为不小心打破封装提供了方便??Java允许返回私有数据的引用(reference)。下面的代码揭示了这一点: bs-O3w  
hoy+J/  
CV/ei,=9  
DPENYr  
import java.awt.Dimension; IyTL|W6  
/***Example class.The x and y values should never*be negative.*/ t__UqCq~h  
public class Example{ j$Ttoo  
  private Dimension d = new Dimension (0, 0); c.5?Q >!+  
  public Example (){ } #BA=?7  
bMT1(edm  
  /*** Set height and width. Both height and width must be nonnegative * or an exception is thrown.*/ Jt4&%b-T  
  public synchronized void setValues (int height,int width) throws IllegalArgumentException{ EdQ:8h  
   if (height < 0 || width < 0) nAc02lJh|  
    throw new IllegalArgumentException(); S}=d74(/n  
    d.height = height; 3+6s}u)  
     d.width = width; pk&kJ307  
  } dP8b\H  
~*-(_<FH  
  public synchronized Dimension getValues(){ c^^[~YW j  
   // Ooops! Breaks encapsulation XA8{N  
   return d; MB$K ?"Y  
  } $JKR,   
} 9qIdwDRY  
cID{X&or  
H{*~d+:ol  
H,r>@Y  
  Example类保证了它所存储的height和width值永远非负数,试图使用setValues()方法来设置负值会触发异常。不幸的是,由于getValues()返回d的引用,而不是d的拷贝,你可以编写如下的破坏性代码: w+ZeVZv!r  
CA2 ,  
q,k/@@Qd9  
F+.:Ry FS  
Example ex = new Example(); *ea%KE":  
Dimension d = ex.getValues(); #R_IF&7  
d.height = -5; y,$kU1yH7  
d.width = -10; fmH"&>Loc  
9 yH/5'  
<gU^#gsGra  
Rs@2Pe$3  
  现在,Example对象拥有负值了!如果getValues() 的调用者永远也不设置返回的Dimension对象的width 和height值,那么仅凭测试是不可能检测到这类的错误。 J7q]|9Hus|  
u&)+~X  
(n'Mf  
MCN}p i  
  不幸的是,随着时间的推移,客户代码可能会改变返回的Dimension对象的值,这个时候,追寻错误的根源是件枯燥且费时的事情,尤其是在多线程环境中。 FJ}RT*7_C  
sQt]Y&_/@  
GX4HW \>a  
B+:'Ld](  
  更好的方式是让getValues()返回拷贝: 1EvAV,v"  
JA!O,4  
6?-vj2,  
$~M#msK9  
public synchronized Dimension getValues(){ /15e-(Zz/  
return new Dimension (d.x, d.y); QdaYP  
} 5mNd5IM  
&WW|! 6  
I;dc[m  
cx?XJ)  
  现在,Example对象的内部状态就安全了。调用者可以根据需要改变它所得到的拷贝的状态,但是要修改Example对象的内部状态,必须通过setValues()才可以。 'gYUyl  
32[lsU>1  
h-B&m:gD_U  
N*.JQvbnr  
  三、常见错误3#:不必要的克隆 zZ3Ko3L%g_  
V+7x_>!&)  
NP%Y\%;l6  
|G.|ocj;  
  我们现在知道了get方法应该返回内部数据对象的拷贝,而不是引用。但是,事情没有绝对: 96fzSZS,  
qaK9E@l  
P]*,955*)  
L\L/+yNv:G  
/*** Example class.The value should never * be negative.*/ T;(k  
public class Example{ zcCX;N  
  private Integer i = new Integer (0); ha6jbni  
  public Example (){ } T/NeoU3 p  
0)/L+P5  
  /*** Set x. x must be nonnegative* or an exception will be thrown*/ <dxc"A  
  public synchronized void setValues (int x) throws IllegalArgumentException{ Ps3wg=ni[  
   if (x < 0) <ptZY.8N  
    throw new IllegalArgumentException(); 7TCY$RcF,I  
    i = new Integer (x); T_}9b  
  } t!MGSB~  
7hPiPv  
  public synchronized Integer getValue(){ > %5<fK2  
   // We can’t clone Integers so we makea copy this way. Y#3<w  
   return new Integer (i.intValue()); E0XfM B]+  
  } b(8#*S!U  
} 7MXi_V;p<  
eR,ePyA;  
5[Sa7Mk  
octBt`\Of  
  这段代码是安全的,但是就象在错误1#那样,又作了多余的工作。Integer对象,就象String对象那样,一旦被创建就是不可变的。因此,返回内部Integer对象,而不是它的拷贝,也是安全的。 Ba$&4?8  
HIUB:  
{ETuaFDM   
*n $=2v^A  
  方法getValue()应该被写为: gkDyWZG B  
\XaKq8uE  
$[5S M>e]  
&)?ECj0`  
public synchronized Integer getValue(){ 2y/|/IW=  
// ’i’ is immutable, so it is safe to return it instead of a copy. eh=.Q<N  
return i; HyKvDJ 3_  
} 92|\`\LP%  
}G,PUjg_^3  
FsI51@V72Q  
E<u6 js,  
  Java程序比C++程序包含更多的不可变对象。JDK 所提供的若干不可变类包括: I^h^QeBis  
$@t]0  
d>j`|(\  
APydZ  
  ?Boolean +C4UM9  
   ?Byte 2H7b2%  
   ?Character dQSX&.<c,  
   ?Class b}DxD1*nsI  
   ?Double SGi(Zkc  
   ?Float @J"Gn-f~  
   ?Integer L4bx [  
   ?Long "<f"r#   
   ?Short '1|FqQ\.  
   ?String :\I88 -N@'  
   ?大部分的Exception的子类 |G^w2"D_Z  
@edx]H1~^  
k/MrNiC  
=+{SZh@  
 四、常见错误4# :自编代码来拷贝数组 xY] Y  
J&mZsa)4  
i,5mH$a&u:  
hS<lUG!9UJ  
  Java允许你克隆数组,但是开发者通常会错误地编写如下的代码,问题在于如下的循环用三行做的事情,如果采用Object的clone方法用一行就可以完成: QDO.&G2  
d\% |!ix  
^Ec);Z  
bb@@QzR  
public class Example{ t= =+SHGP  
  private int[] copy; `cee tr=  
  /*** Save a copy of ’data’. ’data’ cannot be null.*/ b^WTX  
  public void saveCopy (int[] data){ Bf {h\>q  
   copy = new int[data.length]; q~QB?+ x&  
   for (int i = 0; i < copy.length; ++i) s,&tD WU  
    copy = data; sFh mp  
  } ~?l>QP|o  
} v<+5B5"1  
8t4o}3>  
|Nx7jGd:i  
Tf [o'=2  
  这段代码是正确的,但却不必要地复杂。saveCopy()的一个更好的实现是: #^|"dIZ_M  
vumA W*  
"UUzLa_  
;JQ:S~K9  
void saveCopy (int[] data){ !% 'dyj  
  try{ 'Z^-(xG,+  
   copy = (int[])data.clone(); lOk'stLNa&  
  }catch (CloneNotSupportedException e){ -?T:> *]p  
   // Can’t get here. E?,O>bCJ5  
  } >93I|C|  
} X8l|^ [2F  
;8!L*uMI  
(yh zjN~  
A.`) 0dV  
  如果你经常克隆数组,编写如下的一个工具方法会是个好主意: IrIF 853g  
IdP"]Sv{<  
F^La\cZ*'  
fpESuVKr  
static int[] cloneArray (int[] data){ 'Ipp1a Z_M  
  try{ UBj"m<  
   return(int[])data.clone(); ^5{M@o  
  }catch(CloneNotSupportedException e){ t@hE}R  
   // Can’t get here. B4 XN  
  } X,+M?  
} G)|s(C!  
X:3W9`s )*  
s2`:NS  
-SF *DZ  
  这样的话,我们的saveCopy看起来就更简洁了: ~57.0?IK  
l)1FCDV  
#* KmPc+  
Ze?(N~  
void saveCopy (int[] data){ 1?!z<<  
  copy = cloneArray ( data); gHL v zm  
} &\w:jI44Bs  
Pl2ZA)[g  
$G $147z  
x(r>iy  
  五、常见错误5#:拷贝错误的数据 TOH!vQP  
h3.6<vM  
57nSyd] PR  
1/hk3m(C  
  有时候程序员知道必须返回一个拷贝,但是却不小心拷贝了错误的数据。由于仅仅做了部分的数据拷贝工作,下面的代码与程序员的意图有偏差: tN-U,6c]  
*3A`7usU  
BH@b]bEJ  
Hu4\4x$?  
import java.awt.Dimension; oQsls9t  
/*** Example class. The height and width values should never * be 'h]sq {  
negative. */ !:zWhu,  
public class Example{ i'6>_,\(  
  static final public int TOTAL_VALUES = 10; p`oSI}ZwB  
  private Dimension[] d = new Dimension[TOTAL_VALUES]; r]6X  
  public Example (){ } ;";#{B:  
Ug2^cgL  
  /*** Set height and width. Both height and width must be nonnegative * or an exception will be thrown. */ ?G|*=-8  
  public synchronized void setValues (int index, int height, int width) throws IllegalArgumentException{ qFV }Y0w  
   if (height < 0 || width < 0) `XmT)C  
    throw new IllegalArgumentException(); PPj_NV  
    if (d[index] == null) &O&;v|!9  
     d[index] = new Dimension(); G; onJ>  
     d[index].height = height; G\\0N^v  
     d[index].width = width; /E%r@Rui3$  
  } Uu}a! V  
  public synchronized Dimension[] getValues() K |Z]  
   throws CloneNotSupportedException{ :4HZ >!i  
    return (Dimension[])d.clone(); #RN"Ul-B|  
  } aC2cyUuaN  
} cz{5-;$9Z  
TmH'_t.*T~  
f>i" j  
S(&]?!  
  这儿的问题在于getValues()方法仅仅克隆了数组,而没有克隆数组中包含的Dimension对象,因此,虽然调用者无法改变内部的数组使其元素指向不同的Dimension对象,但是调用者却可以改变内部的数组元素(也就是Dimension对象)的内容。方法getValues()的更好版本为: Pr>Pxsr&  
>I*Qc<X91  
*{#l0My  
=x QLf4>  
public synchronized Dimension[] getValues() throws CloneNotSupportedException{ \R}`S`fIw`  
  Dimension[] copy = (Dimension[])d.clone(); I4Y; 9Gg  
  for (int i = 0; i < copy.length; ++i){ v"Z`#Bi  
   // NOTE: Dimension isn’t cloneable. QOfqW@g  
   if (d != null) f\/'Fy0  
    copy = new Dimension (d.height, d.width); K4.GAGd  
  } _,G^#$pH  
  return copy; Y<h [5  
} [UW%(N  
AJ%x"  
H.H$5(?O  
IegZ)&_n  
  在克隆原子类型数据的多维数组的时候,也会犯类似的错误。原子类型包括int,float等。简单的克隆int型的一维数组是正确的,如下所示: I"_``*/1  
+DpiX&^h   
 o(q][:,h  
li`4&<WGC  
public void store (int[] data) throws CloneNotSupportedException{ >}?4;:.=  
  this.data = (int[])data.clone(); M@wQ6ow  
  // OK "i5Rh^  
} OS.oknzZZ  
zA<Hj;9SM  
XH"-sZt  
M8,_E\*  
  拷贝int型的二维数组更复杂些。Java没有int型的二维数组,因此一个int型的二维数组实际上是一个这样的一维数组:它的类型为int[]。简单的克隆int[][]型的数组会犯与上面例子中getValues()方法第一版本同样的错误,因此应该避免这么做。下面的例子演示了在克隆int型二维数组时错误的和正确的做法: 0r|mg::'  
Da@H^  
"&Y5Nh  
p,cw- lN  
public void wrongStore (int[][] data) throws CloneNotSupportedException{ Wwf],Ya  
  this.data = (int[][])data.clone(); // Not OK! Q r n^T  
} hU]Gv)B  
public void rightStore (int[][] data){ Y)7LkZO(y  
  // OK! uyfH;9L5$  
  this.data = (int[][])data.clone(); Gl@-RLo  
  for (int i = 0; i < data.length; ++i){ a YC[15?'  
   if (data != null) wv6rjg:7  
    this.data = (int[])data.clone(); F~C9,`#Wf@  
  } < gtqwH]   
} G\I DgPj`  
N8-!}\,  
bq}hj Cy  
QnJ(C]cW  
'x{E#4A  
   六、常见错误6#:检查new 操作的结果是否为null ;FI"N@z  
kCuIEv@  
#xlT,:_:)  
BY&+fK ae  
  Java编程新手有时候会检查new操作的结果是否为null。可能的检查代码为: Kx@Papn|6  
w4"4(SR.  
=Eimbk  
3r]m8Hp  
Integer i = new Integer (400); GK>.R<[  
if (i == null) iW\Q>~0#_  
throw new NullPointerException(); kz UP   
REaU=-m-  
~\ C.Nm  
^rP` . Z  
  检查当然没什么错误,但却不必要,if和throw这两行代码完全是浪费,他们的唯一功用是让整个程序更臃肿,运行更慢。 |+|q`SwJ  
E#T6rd P  
e ; #"t  
)q>mt/,  
  C/C++程序员在开始写java程序的时候常常会这么做,这是由于检查C中malloc()的返回结果是必要的,不这样做就可能产生错误。检查C++中new操作的结果可能是一个好的编程行为,这依赖于异常是否被使能(许多编译器允许异常被禁止,在这种情况下new操作失败就会返回null)。在java 中,new 操作不允许返回null,如果真的返回null,很可能是虚拟机崩溃了,这时候即便检查返回结果也无济于事。 [!Jd.zm  
.]IidsgM  
 七、常见错误7#:用== 替代.equals SZ*Nr=X  
P%nN#Qm  
  在Java中,有两种方式检查两个数据是否相等:通过使用==操作符,或者使用所有对象都实现的.equals方法。原子类型(int, flosat, char 等)不是对象,因此他们只能使用==操作符,如下所示: lZI?k=rWv  
m%[Ul@!V  
:I)WSXP9h  
= ;!$Qw4  
int x = 4; jJ B+UF=  
int y = 5; = MP?aH [  
if (x == y) ;%/Kh :Vg  
  System.out.println ("Hi"); b;AGw3SF  
// This ’if’ test won’t compile. w:0=L`<Eu  
if (x.equals (y)) jIOrB}  
  System.out.println ("Hi"); x U1](O  
ux 7^PTgcO  
Te:4 z@?  
;hcOD4or  
  对象更复杂些,==操作符检查两个引用是否指向同一个对象,而equals方法则实现更专门的相等性检查。 uv}?8$<\  
10C,\  
vp#AD9h1  
Fhr5)Z  
  更显得混乱的是由java.lang.Object 所提供的缺省的equals方法的实现使用==来简单的判断被比较的两个对象是否为同一个。 SCUsDr+.  
&E(KOfk#  
|hlc#t ?  
];n3H~2  
  许多类覆盖了缺省的equals方法以便更有用些,比如String类,它的equals方法检查两个String对象是否包含同样的字符串,而Integer的equals方法检查所包含的int值是否相等。 7[)IP:I>  
wE4:$+R};  
 Q9!T@  
, (Bo .(]  
  大部分时候,在检查两个对象是否相等的时候你应该使用equals方法,而对于原子类型的数据,你用该使用==操作符。 c-dOb.v0  
i- v PJg1  
|d@%Vb_  
 #"6O3.P  
  八、常见错误8#: 混淆原子操作和非原子操作 c[h{C!d1  
DviRD[+q"  
;#goC N.  
3a_=e B  
  Java保证读和写32位数或者更小的值是原子操作,也就是说可以在一步完成,因而不可能被打断,因此这样的读和写不需要同步。以下的代码是线程安全(thread safe)的: Rb8wq.LqD  
8pEiU/V  
Tw{}Ht_Qq  
v_7?Zik8E  
public class Example{ [J`%i U  
  private int value; // More code here... O8\dMb  
  public void set (int x){ 2A5R3x= \  
   // NOTE: No synchronized keyword |IL/F]I  
   this.value = x; { !;I4W%!  
  } 2c Pd$j  
} l[G&=/R@H  
h:J0d~u  
h yPVt6Gkj  
v*pN~}5  
  不过,这个保证仅限于读和写,下面的代码不是线程安全的: &ml7368@  
{&bjjM  
V2&O]bR  
zK5/0zMZ  
public void increment (){ ZYi."^l  
  // This is effectively two or three instructions: +;ILj<!Z7  
  // 1) Read current setting of ’value’. C1V@\mRi  
  // 2) Increment that setting. _(R1En1  
  // 3) Write the new setting back. p#yq'kY  
  ++this.value; L93PDp4v  
} q3CcXYY  
Tf|?j=f  
|v'_Co0ki  
VN5UJ!$?J  
  在测试的时候,你可能不会捕获到这个错误。首先,测试与线程有关的错误是很难的,而且很耗时间。其次,在有些机器上,这些代码可能会被翻译成一条指令,因此工作正常,只有当在其它的虚拟机上测试的时候这个错误才可能显现。因此最好在开始的时候就正确地同步代码: p,)~w1|  
D;@nrj`.  
~eVq Fc  
Ui^~A  
public synchronized void increment (){ zn=Ifz)#|  
  ++this.value; YEg(QOn3Q  
} 19r4J(pV  
vzr?#FG  
Vg>\@ C .s  
#%=6DHsK  
  九、常见错误9#:在catch 块中作清除工作 &"h 9Awn2  
,k,RXgQ  
_ yu d  
=tS1|_  
  一段在catch块中作清除工作的代码如下所示: 0pC}+ +  
C'_^DPzj  
V\!6K  
323zR*\m  
OutputStream os = null; cg]\R1Gm  
try{ d&@>P&AT  
  os = new OutputStream (); lVw77bZ  
  // Do something with os here. n B5:X  
  os.close(); MPtn$@  
}catch (Exception e){ doERBg`Jh  
  if (os != null) MHm=X8eg  
  os.close(); x$6` k  
} ~$bkWb*RJ  
0# )I :5  
aLWNqe&1  
swfcA\7R  
  尽管这段代码在几个方面都是有问题的,但是在测试中很容易漏掉这个错误。下面列出了这段代码所存在的三个问题: 3Y L  
Hju7gP=y}  
lU}y%J@  
QO-R>  
  1.语句os.close()在两处出现,多此一举,而且会带来维护方面的麻烦。 >R9_ ;  
Zs(I]^w;d  
g}vOp3 ^  
`2B,+ytW8  
  2.上面的代码仅仅处理了Exception,而没有涉及到Error。但是当try块运行出现了Error,流也应该被关闭。 QXQ'QEG  
e1EFZ,EcaO  
<#[_S$54  
6c?;-5.  
  3.close()可能会抛出异常。 U:a-Wi+  
5*q!:$ W  
_>6xU t  
 L$Uy  
  上面代码的一个更优版本为: :skNEY].  
tm"9`   
Qh0tU<jG  
/9K,W)h_  
OutputStream os = null; AB.gVw| 4  
try{  /z0X  
  os = new OutputStream (); &yQilyU{V  
  // Do something with os here. pZYcCc>6&  
}finally{ &sbKN[xM  
  if (os != null) (eG9b pqr  
   os.close(); t7t?xk!2  
} ~)Z MGx  
8Moe8X#3  
FR7DuH/f)  
DR d|m<Z  
  这个版本消除了上面所提到的两个问题:代码不再重复,Error也可以被正确处理了。但是没有好的方法来处理第三个问题,也许最好的方法是把close()语句单独放在一个try/catch块中。 t,YnweH  
cJ}J4?  
-=tf)  
)r9l T*z  
  十、常见错误10#: 增加不必要的catch 块 \hm;p  
']bpsn  
!zu YO3:  
{c7ZA%T~R  
  一些开发者听到try/catch块这个名字后,就会想当然的以为所有的try块必须要有与之匹配的catch块。 J$]-)`[G&  
61XLL/=P  
Ve]ufn6  
e(5 :XHe  
  C++程序员尤其是会这样想,因为在C++中不存在finally块的概念,而且try块存在的唯一理由只不过是为了与catch块相配对。 :jJ;&t^^  
#[Z1W8e  
(P+TOu-y\  
sQ)D.9\~  
  增加不必要的catch块的代码就象下面的样子,捕获到的异常又立即被抛出: 8RA]h?$$J  
H}Jdnu|ko  
&gP/<!#  
3?R56$-+  
try{ z]^u@]@NC  
  // Nifty code here B8f BX!u/  
}catch(Exception e){ 5$<\  
  throw e; 17\5 NgB  
}finally{ xrXfLujn%  
  // Cleanup code here I 3ZlKI  
} +b9gP\Hke  
/M0A9ZT[  
\!+#9sq0  
NSsLuM=.  
  不必要的catch块被删除后,上面的代码就缩短为: UdIl5P  
z'W8t|m}Pb  
C1x"q9| \`  
mMz^I7$  
try{ 9AA_e ~y  
  // Nifty code here kF1Tg KSd  
}finally{ (oftq!X2  
  // Cleanup code here |8|_^`  
} L"_l(<g  
$jc&Tk#  
dN8@ 0AMSf  
LU=<? "N6  
  常见错误11#;没有正确实现equals,hashCode,或者clone 等方法 *hk8[  
d,hKy2  
[i9.#*  
-$x5[6bN  
  方法equals,hashCode,和clone 由java.lang.Object提供的缺省实现是正确的。不幸地是,这些缺省实现在大部分时候毫无用处,因此许多类覆盖其中的若干个方法以提供更有用的功能。但是,问题又来了,当继承一个覆盖了若干个这些方法的父类的时候,子类通常也需要覆盖这些方法。在进行代码审查时,应该确保如果父类实现了equals,hashCode,或者clone等方法,那么子类也必须正确。正确的实现equals,hashCode,和clone需要一些技巧。 ;Nd,K C0k  
r?:zKj8/u  
nn1T5;  
bm</qF'T6  
  小结 C:?mOM#_  
dR^7d _!  
}.L\O]~{  
pPa3byWf  
  我在代码审查的时候至少遇到过一次这些错误,我自己也犯过其中的几个错误。好消息是只要你知道你在找什么错误,那么代码审查就很容易管理,错误也很容易被发现和修改。即便你找不到时间来进行正规的代码审查,以自审的方式把这些错误从你的代码中根除会大大节省你的调试时间。花时间在代码审查上是值得的。 $4*wK@xu  
 .# Jusd  
 
评价一下你浏览此帖子的感受

精彩

感动

搞笑

开心

愤怒

无聊

灌水
描述
快速回复

您目前还是游客,请 登录注册
温馨提示:欢迎交流讨论,请勿纯表情、纯引用!
认证码:
验证问题:
3+5=?,请输入中文答案:八 正确答案:八