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

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

级别: 终身会员
发帖
3743
铜板
8
人品值
493
贡献值
9
交易币
0
好评度
3746
信誉值
0
金币
0
所在楼道
代码审查是消灭Bug最重要的方法之一,这些审查在大多数时候都特别奏效。由于代码审查本身所针对的对象,就是俯瞰整个代码在测试过程中的问题和Bug。并且,代码审查对消除一些特别细节的错误大有裨益,尤其是那些能够容易在阅读代码的时候发现的错误,这些错误往往不容易通过机器上的测试识别出来。本文就常见的Java代码中容易出现的问题提出一些建设性建议,以便您在审查代码的过程中注意到这些常见的细节性错误。 _ q1|\E%`h  
LR.+C xQ  
'.v^seU  
  通常给别人的工作挑错要比找自己的错容易些。别样视角的存在也解释了为什么作者需要编辑,而运动员需要教练的原因。不仅不应当拒绝别人的批评,我们应该欢迎别人来发现并指出我们的编程工作中的不足之处,我们会受益匪浅的。 *g}&&$b0  
XsMphZnK  
Lu5.$b  
)xs,  
 正规的代码审查(code inspection)是提高代码质量的最强大的技术之一,代码审查?由同事们寻找代码中的错误?所发现的错误与在测试中所发现的错误不同,因此两者的关系是互补的,而非竞争的。 j ZafwBi  
7l EwQ  
55en D  
=&xoyF  
  如果审查者能够有意识地寻找特定的错误,而不是靠漫无目的的浏览代码来发现错误,那么代码审查的效果会事半功倍。在这篇文章中,我列出了11个Java编程中常见的错误。你可以把这些错误添加到你的代码审查的检查列表(checklist)中,这样在经过代码审查后,你可以确信你的代码中不再存在这类错误了。 <08V-   
Kt0Tuj@CY  
<;"=ah7A  
cC]1D*Bn  
  一、常见错误1# :多次拷贝字符串 LxDhthZi_  
%P6!vx:&^b  
N* -Z Jv  
_ h-X-s Y  
  测试所不能发现的一个错误是生成不可变(immutable)对象的多份拷贝。不可变对象是不可改变的,因此不需要拷贝它。最常用的不可变对象是String。 HK.J/Zr  
H!=BjU1Pmg  
J4X35H=Z  
<5^(l$IBj  
  如果你必须改变一个String对象的内容,你应该使用StringBuffer。下面的代码会正常工作: !d )i6W?  
VG7#6)sQoK  
q,Q|Uvpk  
h}_q  
String s = new String ("Text here"); J8'zvH&I  
m @ ?e <$  
Z}f_\d'  
fe/6JV  
  但是,这段代码性能差,而且没有必要这么复杂。你还可以用以下的方式来重写上面的代码: e8v=n@0  
g>CQO,s;w  
M*uG`Eo&  
hglt D8,  
String temp = "Text here"; 1i2w<VG1  
String s = new String (temp); rpWy 6oD  
#+\G- =-  
9mm(?O~'p  
/ep~/#Ia  
  但是这段代码包含额外的String,并非完全必要。更好的代码为: ?8/h3xV;  
]vErF=[U,  
';F][x5j  
b>WT-.b0  
String s = "Text here"; )P])0Y-  
{D#`+uw  
!}7m^  
_%B`Y ?I`  
  二、常见错误2#: 没有克隆(clone)返回的对象 j+/*NM_y3  
b<7f:drVC  
S\(_"xJPp  
N|}`p"  
  封装(encapsulation)是面向对象编程的重要概念。不幸的是,Java为不小心打破封装提供了方便??Java允许返回私有数据的引用(reference)。下面的代码揭示了这一点: r1] e:  
@xE Q<g  
J>35q'nN]F  
:P~Owz  
import java.awt.Dimension; 7a net  
/***Example class.The x and y values should never*be negative.*/ ] fB{  
public class Example{ GAKJc\o  
  private Dimension d = new Dimension (0, 0); <rs]@J'p  
  public Example (){ } ks$G6WC  
Q!v[b{]8  
  /*** Set height and width. Both height and width must be nonnegative * or an exception is thrown.*/ NBX/V^  
  public synchronized void setValues (int height,int width) throws IllegalArgumentException{ ^Du_e(TiyK  
   if (height < 0 || width < 0) ZxQP,Ys_Y  
    throw new IllegalArgumentException(); 8b!_b2Za  
    d.height = height; WTx;,TNG  
     d.width = width; L8Q!6oO=<  
  } Y`uCDfcQ  
(Bz(KyD[  
  public synchronized Dimension getValues(){ ).xWjVC  
   // Ooops! Breaks encapsulation 3}+ \&[  
   return d; S{6u\Vy  
  } `<q5RuU  
} 1wt]J!hgV  
X*Zv,Wm  
$)!Z"2T  
4NIfQYC.  
  Example类保证了它所存储的height和width值永远非负数,试图使用setValues()方法来设置负值会触发异常。不幸的是,由于getValues()返回d的引用,而不是d的拷贝,你可以编写如下的破坏性代码: $P_Y8:  
clNP9{  
jC%I]#!n  
! ZEKvW  
Example ex = new Example(); /_\4( vvf  
Dimension d = ex.getValues(); /Y:Zqk3  
d.height = -5; q-#fuD^  
d.width = -10; p(Mv^ea  
;f Gi5=-  
4tjRju?  
Hw? J1#1IE  
  现在,Example对象拥有负值了!如果getValues() 的调用者永远也不设置返回的Dimension对象的width 和height值,那么仅凭测试是不可能检测到这类的错误。 >B0S5:S$W  
??PpHB J')  
FmPF7  
H'2 =yhtVh  
  不幸的是,随着时间的推移,客户代码可能会改变返回的Dimension对象的值,这个时候,追寻错误的根源是件枯燥且费时的事情,尤其是在多线程环境中。 ^E^:=Q?'_  
$ }53f'QjW  
al/~  
c@`P{ 6  
  更好的方式是让getValues()返回拷贝: Wj&s5;2a  
&n|gPp77$  
*O~D lf  
zPe .  
public synchronized Dimension getValues(){ >\ W" 3.  
return new Dimension (d.x, d.y); 0dW1I|jR  
} 9EEHLx"  
K4"as9oFP  
}O/Nn0,  
E2MpMR  
  现在,Example对象的内部状态就安全了。调用者可以根据需要改变它所得到的拷贝的状态,但是要修改Example对象的内部状态,必须通过setValues()才可以。 aH_&=/-Tz  
Dp8(L ]6  
eR CGr?e4  
P\JpE  
  三、常见错误3#:不必要的克隆 j*"s~8u4  
|@RO&F  
2k_Bo~.  
sdLFBiR  
  我们现在知道了get方法应该返回内部数据对象的拷贝,而不是引用。但是,事情没有绝对: >:=TS"}yS}  
2r,fF<WQ  
15COwc*k  
@OpcS>:R  
/*** Example class.The value should never * be negative.*/ ; OsN^   
public class Example{ #qWEyb2UZ  
  private Integer i = new Integer (0); 0:*$i(2  
  public Example (){ } n2E2V<#   
hY!G>d{J  
  /*** Set x. x must be nonnegative* or an exception will be thrown*/ MEu-lM7v  
  public synchronized void setValues (int x) throws IllegalArgumentException{ KGIz)/eSg  
   if (x < 0) [ LCi,  
    throw new IllegalArgumentException(); m<E7cY3mX  
    i = new Integer (x); kHO\#fF<  
  } Nn$$yUkMX  
08Q:1 '  
  public synchronized Integer getValue(){ 0pQ>V)  
   // We can’t clone Integers so we makea copy this way. 5Ai Yx}  
   return new Integer (i.intValue()); IH5thL@D  
  } ;(V=disU/  
} tc[PJH&P  
v[x`I;  
*M{1RMc  
hRP0Djc  
  这段代码是安全的,但是就象在错误1#那样,又作了多余的工作。Integer对象,就象String对象那样,一旦被创建就是不可变的。因此,返回内部Integer对象,而不是它的拷贝,也是安全的。 ,#crtX  
sEoS|"  
&+\wYa,  
;(XSw%Y H  
  方法getValue()应该被写为: SV.*Z|"^N  
t5&$ y`  
, Le_PJY)  
n}l Z  
public synchronized Integer getValue(){ HBt?cA '  
// ’i’ is immutable, so it is safe to return it instead of a copy. t/3veDh@  
return i; "783F:mPh  
} _u>+H#  
8)i\d`  
?mV[TM{p  
}Eav@3h6  
  Java程序比C++程序包含更多的不可变对象。JDK 所提供的若干不可变类包括: P5N"7/PfW  
DT*/2TH*l  
RR"#z'zQ  
r )T`?y  
  ?Boolean ;,viE~n  
   ?Byte :A[ Gtc(_  
   ?Character HjK|9  
   ?Class ^3e l-dZ  
   ?Double '!_o`t@  
   ?Float uuq?0t2Z  
   ?Integer VR'w$mp  
   ?Long b c+' n  
   ?Short hJ|z8Sy@1  
   ?String TqWvHZX  
   ?大部分的Exception的子类 \UXQy{Ex  
PgVM>_nHk  
LE7o[<>  
MFC= oKD  
 四、常见错误4# :自编代码来拷贝数组 (F @IUbnl  
]Y3ALQr!  
zR e0z2  
b&LhydaJ  
  Java允许你克隆数组,但是开发者通常会错误地编写如下的代码,问题在于如下的循环用三行做的事情,如果采用Object的clone方法用一行就可以完成: =/zQJzN  
R)#"Ab Z'  
n44j]+P  
C ZJW`c/  
public class Example{ ddw!FH2W (  
  private int[] copy; !XK p_v  
  /*** Save a copy of ’data’. ’data’ cannot be null.*/ 5~\W!|j/  
  public void saveCopy (int[] data){ L|c01  
   copy = new int[data.length]; )&*&ZL0  
   for (int i = 0; i < copy.length; ++i) Jap v<lV%  
    copy = data; 0hPm,H*Y]  
  } aUd6 33  
} h322^24-2  
il:+O08_  
hRME;/r]X  
A>X#[qx  
  这段代码是正确的,但却不必要地复杂。saveCopy()的一个更好的实现是: o<x2,uT  
p}C3<[Nk  
RlpW)\{j?  
jML}{>Gy8S  
void saveCopy (int[] data){ -`rz[";n  
  try{ ](%-5G1<  
   copy = (int[])data.clone(); r1,RloyZS  
  }catch (CloneNotSupportedException e){ V;>p@uE,P  
   // Can’t get here. `LNRl'Z m  
  } ~x824xW  
} J H6\;G6  
P,,@&* :  
d=q2Or   
eQMY3/#  
  如果你经常克隆数组,编写如下的一个工具方法会是个好主意: W4Zi?@L>'  
/H}83 C  
?:UDK?  
p`Ax)L\f  
static int[] cloneArray (int[] data){ `2GHB@S"k  
  try{ nL\BB&  
   return(int[])data.clone(); [^aow-4z  
  }catch(CloneNotSupportedException e){ y%43w4  
   // Can’t get here. ,;UVQwY  
  } Qp{{OjD  
} ~~>D=~B0'  
>YD? pDPb/  
d6wsT\S  
[0  3Aej  
  这样的话,我们的saveCopy看起来就更简洁了: i/~A7\:8%  
x#'# ~EO-G  
uQrD}%GI  
P.LMu  
void saveCopy (int[] data){ nd-y`@z  
  copy = cloneArray ( data); %|4Nmf$:Og  
} `NrxoU=  
]Rz]"JZ\S  
"`16-g97  
]>&au8  
  五、常见错误5#:拷贝错误的数据 )~rN{W<s`H  
GBN^ *I  
~fEgrF d  
2}t2k>  
  有时候程序员知道必须返回一个拷贝,但是却不小心拷贝了错误的数据。由于仅仅做了部分的数据拷贝工作,下面的代码与程序员的意图有偏差: TN(1oJ:  
W,}C*8{+  
m\[r6t]V  
|6$6Za]:  
import java.awt.Dimension; Hj LY\.S  
/*** Example class. The height and width values should never * be L= hPu#&/  
negative. */ @MTm8E6au  
public class Example{ ShFSBD\M#  
  static final public int TOTAL_VALUES = 10; GJU84Xn7  
  private Dimension[] d = new Dimension[TOTAL_VALUES]; ,{7Z OzA  
  public Example (){ } 8h}o5B  
7@5}WNr  
  /*** Set height and width. Both height and width must be nonnegative * or an exception will be thrown. */ 9tWu>keu  
  public synchronized void setValues (int index, int height, int width) throws IllegalArgumentException{ iq=<LOx  
   if (height < 0 || width < 0) L3,p8-d9Z  
    throw new IllegalArgumentException(); Beq zw0  
    if (d[index] == null) Z_Hc":4i  
     d[index] = new Dimension(); Y0 Ta&TYZ0  
     d[index].height = height; *e!0ZB3J  
     d[index].width = width; ^ola5wD  
  } k#&d`?X  
  public synchronized Dimension[] getValues() wm !Y5  
   throws CloneNotSupportedException{ BH0].-)[y!  
    return (Dimension[])d.clone(); YR^J7b\  
  } ma,H<0R  
} ;5?$q  
hxGZ}zq*S  
0DV .1  
5_9mA4gs@  
  这儿的问题在于getValues()方法仅仅克隆了数组,而没有克隆数组中包含的Dimension对象,因此,虽然调用者无法改变内部的数组使其元素指向不同的Dimension对象,但是调用者却可以改变内部的数组元素(也就是Dimension对象)的内容。方法getValues()的更好版本为: ^,qi` Tk  
=Z2Cg{z  
ZXh6Se4o  
4DaLmQ2O  
public synchronized Dimension[] getValues() throws CloneNotSupportedException{ 9])dLL0  
  Dimension[] copy = (Dimension[])d.clone(); V)=!pT  
  for (int i = 0; i < copy.length; ++i){ FIDV5Y/f  
   // NOTE: Dimension isn’t cloneable. gDa}8!+i  
   if (d != null) =`Pgo5A  
    copy = new Dimension (d.height, d.width); ,C1}gPQ6<  
  } |>Qj]  
  return copy; 1/:WA:]1 ,  
} ozy~`$;c  
0[/>> !ws  
Y/?V%X  
}4b 4<Sm_h  
  在克隆原子类型数据的多维数组的时候,也会犯类似的错误。原子类型包括int,float等。简单的克隆int型的一维数组是正确的,如下所示: a6cq0g[#z  
aSkH<5i`v  
hRHqG  
;shhg z$  
public void store (int[] data) throws CloneNotSupportedException{ UJ* D  
  this.data = (int[])data.clone(); % \IB_M  
  // OK 4}E|CD/pZ  
} %F_)!M;x  
F<39eDNpz  
-|YG**i/  
D,b'1=  
  拷贝int型的二维数组更复杂些。Java没有int型的二维数组,因此一个int型的二维数组实际上是一个这样的一维数组:它的类型为int[]。简单的克隆int[][]型的数组会犯与上面例子中getValues()方法第一版本同样的错误,因此应该避免这么做。下面的例子演示了在克隆int型二维数组时错误的和正确的做法: 3copJS  
dZ K /v  
;89 `!V O  
T)? : q  
public void wrongStore (int[][] data) throws CloneNotSupportedException{ :[YHJaK  
  this.data = (int[][])data.clone(); // Not OK! LX2rg\a+%  
} [|.IXdJ!  
public void rightStore (int[][] data){ =bgzl=A`  
  // OK! _FR_6*C)5  
  this.data = (int[][])data.clone(); K[r<-6TS  
  for (int i = 0; i < data.length; ++i){ %38HGjS  
   if (data != null) 1fUg  
    this.data = (int[])data.clone(); ova4  
  } cNOtfn6?F  
} yq]=+X>(  
WR,MqM20  
KcKdhqdN-  
/enlkZx=8  
UEHJ? }  
   六、常见错误6#:检查new 操作的结果是否为null &y_Ya%Z3*e  
X?whyD)vE@  
RC?gozBFJ  
>%LZ|*U  
  Java编程新手有时候会检查new操作的结果是否为null。可能的检查代码为: [}:;B$,  
pZHx  
v,]-;V~<  
i[L5,%5<H  
Integer i = new Integer (400); )S"!)\4 b  
if (i == null) GWd71ZtFO  
throw new NullPointerException(); 5,dKha  
^m pWQ`R  
I8};t b#  
uIh68UM  
  检查当然没什么错误,但却不必要,if和throw这两行代码完全是浪费,他们的唯一功用是让整个程序更臃肿,运行更慢。 b$FK}D5  
F/p/&9  
Hw29V //  
O?,Grn%'.  
  C/C++程序员在开始写java程序的时候常常会这么做,这是由于检查C中malloc()的返回结果是必要的,不这样做就可能产生错误。检查C++中new操作的结果可能是一个好的编程行为,这依赖于异常是否被使能(许多编译器允许异常被禁止,在这种情况下new操作失败就会返回null)。在java 中,new 操作不允许返回null,如果真的返回null,很可能是虚拟机崩溃了,这时候即便检查返回结果也无济于事。 ./5LV)_`  
5?l8;xe`{f  
 七、常见错误7#:用== 替代.equals x Zp`  
gi {rqM  
  在Java中,有两种方式检查两个数据是否相等:通过使用==操作符,或者使用所有对象都实现的.equals方法。原子类型(int, flosat, char 等)不是对象,因此他们只能使用==操作符,如下所示: k4T`{s}e  
HE!"3S2S&+  
Uzh#z eZ`<  
Z;/QB6|%  
int x = 4; Y]!WPJ`f2  
int y = 5; zD^*->`p  
if (x == y) Aq 5CF`e{  
  System.out.println ("Hi"); R ?62g H  
// This ’if’ test won’t compile. {:;6 *W  
if (x.equals (y)) OTe h8h  
  System.out.println ("Hi"); (fNG51h!  
qkXnpv  
l(A)Gd5>  
<=nOyT9  
  对象更复杂些,==操作符检查两个引用是否指向同一个对象,而equals方法则实现更专门的相等性检查。 2 o)8'Lp  
d)>b/0CZ  
fM/~k>wl  
L0\~ K~q  
  更显得混乱的是由java.lang.Object 所提供的缺省的equals方法的实现使用==来简单的判断被比较的两个对象是否为同一个。 /aV;EkyO,  
5]f6YlJZ  
R<djW5()f  
i1dE.f ;  
  许多类覆盖了缺省的equals方法以便更有用些,比如String类,它的equals方法检查两个String对象是否包含同样的字符串,而Integer的equals方法检查所包含的int值是否相等。 8yCt(ms  
s@ 02 ?+/  
PPAcEXsIu  
Kj53"eW  
  大部分时候,在检查两个对象是否相等的时候你应该使用equals方法,而对于原子类型的数据,你用该使用==操作符。 e@h (Zwp  
h-.xx 4D  
"%(SLQOyy  
9QP-~V{$  
  八、常见错误8#: 混淆原子操作和非原子操作 ~n=oPm$pR  
6L<Y   
jWL%*dJrN  
JIc(hRf9>  
  Java保证读和写32位数或者更小的值是原子操作,也就是说可以在一步完成,因而不可能被打断,因此这样的读和写不需要同步。以下的代码是线程安全(thread safe)的: O,PTY^  
w%1-_;.aU6  
;IOM3'5 T@  
B@j2^Dr~!  
public class Example{ P9 w);jp;  
  private int value; // More code here... O2qy[]km  
  public void set (int x){ A Xpg_JC  
   // NOTE: No synchronized keyword .QU]  
   this.value = x; U3**x5F_  
  } v? Zo5uVoq  
} DuQW?9^232  
:y.~IQN  
Y 'y yrn}  
Y(RB@+67  
  不过,这个保证仅限于读和写,下面的代码不是线程安全的: &>f]  
#HDP ha  
0^3n#7m;K  
b($9gre>mI  
public void increment (){ QQ,V35Vp[  
  // This is effectively two or three instructions: ;#bDz}|\AN  
  // 1) Read current setting of ’value’. 6Vgxfic  
  // 2) Increment that setting. 7v&>d,  
  // 3) Write the new setting back. &#zx/$  
  ++this.value; FLo`EE":O(  
} 7#&e0fw/I  
8S` j6  
;w7s>(ITZ  
T' =6_?7K4  
  在测试的时候,你可能不会捕获到这个错误。首先,测试与线程有关的错误是很难的,而且很耗时间。其次,在有些机器上,这些代码可能会被翻译成一条指令,因此工作正常,只有当在其它的虚拟机上测试的时候这个错误才可能显现。因此最好在开始的时候就正确地同步代码: {TXfi'\  
_vJ(F  
<2af&-EG s  
4L bll%[9  
public synchronized void increment (){ XL7||9,(h  
  ++this.value; :85QwN]\  
} TKp2C5bX  
gNJdP!(t  
!bIE%cq  
EQtYb"_  
  九、常见错误9#:在catch 块中作清除工作 5?Ukf$)x  
oj/#wF+  
I5@8=rFk  
K&VMhMVb  
  一段在catch块中作清除工作的代码如下所示: r=HL!XFk  
;i?rd f  
G<-<>)zO!  
:K~sazs7J  
OutputStream os = null; G0A\"2U  
try{ ,8.$!Zia  
  os = new OutputStream (); >,ABE2t5  
  // Do something with os here. 99tUw'w  
  os.close(); Y(u`K=*  
}catch (Exception e){ 9;Q|" T  
  if (os != null) VAo`R9^D#  
  os.close(); 2bOl`{x  
} nDS\2  
OZ33w-X<  
9#>nFs"H  
#KNl<V+c}1  
  尽管这段代码在几个方面都是有问题的,但是在测试中很容易漏掉这个错误。下面列出了这段代码所存在的三个问题: 0|<9eD\I=  
 {FX]1:  
BRa9j:_b  
^xgqs $`7  
  1.语句os.close()在两处出现,多此一举,而且会带来维护方面的麻烦。 Vr@tSc&  
gFd*\Dk  
|c>.xt~  
c^rWS&)P  
  2.上面的代码仅仅处理了Exception,而没有涉及到Error。但是当try块运行出现了Error,流也应该被关闭。 Zoy)2E{  
18Vn[}]"  
VsJKxa4  
==UYjbuU  
  3.close()可能会抛出异常。  |e<$  
Gr>CdB>~+  
~0h@p4  
&=f?:UZ%  
  上面代码的一个更优版本为: xYZ,.  
.4ZOm'ko{  
q6ZewuV.  
k }{o: N  
OutputStream os = null; .Cf!5[0E  
try{ PC HKH  
  os = new OutputStream (); 5$$# d_Gj  
  // Do something with os here. `8r$b/6  
}finally{ J$PlI  
  if (os != null) F9Af{*Jw?x  
   os.close(); 4K\o2p?4  
} l,~`o$ _  
x]@z.Yj  
Qea"49R  
F2\&rC4v  
  这个版本消除了上面所提到的两个问题:代码不再重复,Error也可以被正确处理了。但是没有好的方法来处理第三个问题,也许最好的方法是把close()语句单独放在一个try/catch块中。 t(dVd%   
/OYa1,  
E%( s=YhW  
Ex Q\qp3  
  十、常见错误10#: 增加不必要的catch 块 tJ7F.}\;C  
#.!#"8{0_  
UCXRF  
xHqF_10S#  
  一些开发者听到try/catch块这个名字后,就会想当然的以为所有的try块必须要有与之匹配的catch块。 fs:yx'mxV  
?pcbso  
N:CQ$7T{ j  
*dxm|F98  
  C++程序员尤其是会这样想,因为在C++中不存在finally块的概念,而且try块存在的唯一理由只不过是为了与catch块相配对。 %% /8B  
1Q!kk5jE  
BY&{fWUo  
cly}[<w!  
  增加不必要的catch块的代码就象下面的样子,捕获到的异常又立即被抛出: 7#W]Qj  
ZyDNtX%  
jO/cdLKX(  
^_i)XdPU  
try{ G"m?2$^-A  
  // Nifty code here `qYiic%  
}catch(Exception e){ {F2Rv  
  throw e; e&2,cQRFV  
}finally{ Te[v+jgLY,  
  // Cleanup code here E .28G2&  
} [& Z- *a  
1r};cY6  
@?3^ Ks_  
fm@Pa} ,  
  不必要的catch块被删除后,上面的代码就缩短为: _5H~1G%q  
(~%NRH<\  
[u$|/  
i39ZBs@  
try{ <i4]qO(0u  
  // Nifty code here /t< &  
}finally{ o[}Dj6e\t  
  // Cleanup code here [\#ANA"  
} G0|}s&$yL  
$,J0) ~  
4H (8BNgzV  
+7o1&D*v  
  常见错误11#;没有正确实现equals,hashCode,或者clone 等方法 P3]K'*Dyd  
c|JQ0] K  
N mXRA(m  
s9a`2Wm  
  方法equals,hashCode,和clone 由java.lang.Object提供的缺省实现是正确的。不幸地是,这些缺省实现在大部分时候毫无用处,因此许多类覆盖其中的若干个方法以提供更有用的功能。但是,问题又来了,当继承一个覆盖了若干个这些方法的父类的时候,子类通常也需要覆盖这些方法。在进行代码审查时,应该确保如果父类实现了equals,hashCode,或者clone等方法,那么子类也必须正确。正确的实现equals,hashCode,和clone需要一些技巧。 h=,h Yz?]  
:o ~'\:/  
7Mq{Py1  
bhGRD{=  
  小结 _/z_ X  
<tBT?#C9+  
9 " t;6  
z@,(^~C_  
  我在代码审查的时候至少遇到过一次这些错误,我自己也犯过其中的几个错误。好消息是只要你知道你在找什么错误,那么代码审查就很容易管理,错误也很容易被发现和修改。即便你找不到时间来进行正规的代码审查,以自审的方式把这些错误从你的代码中根除会大大节省你的调试时间。花时间在代码审查上是值得的。 Z$g'h1,zW  
vanV|O  
 
评价一下你浏览此帖子的感受

精彩

感动

搞笑

开心

愤怒

无聊

灌水
描述
快速回复

您目前还是游客,请 登录注册
批量上传需要先选择文件,再选择上传
认证码:
验证问题:
10+5=?,请输入中文答案:十五