代码审查是消灭Bug最重要的方法之一,这些审查在大多数时候都特别奏效。由于代码审查本身所针对的对象,就是俯瞰整个代码在测试过程中的问题和Bug。并且,代码审查对消除一些特别细节的错误大有裨益,尤其是那些能够容易在阅读代码的时候发现的错误,这些错误往往不容易通过机器上的测试识别出来。本文就常见的Java代码中容易出现的问题提出一些建设性建议,以便您在审查代码的过程中注意到这些常见的细节性错误。 WVP?Ie8
x@NfN*?/+i
.p[uIRd`
通常给别人的工作挑错要比找自己的错容易些。别样视角的存在也解释了为什么作者需要编辑,而运动员需要教练的原因。不仅不应当拒绝别人的批评,我们应该欢迎别人来发现并指出我们的编程工作中的不足之处,我们会受益匪浅的。 Kb; *"@LX
WtOjPW
g}_2T\$k
T?8BAxC?K
正规的代码审查(code inspection)是提高代码质量的最强大的技术之一,代码审查?由同事们寻找代码中的错误?所发现的错误与在测试中所发现的错误不同,因此两者的关系是互补的,而非竞争的。 _XZ
Gj:V
lp`j3)
0#V"
be+-p
如果审查者能够有意识地寻找特定的错误,而不是靠漫无目的的浏览代码来发现错误,那么代码审查的效果会事半功倍。在这篇文章中,我列出了11个Java编程中常见的错误。你可以把这些错误添加到你的代码审查的检查列表(checklist)中,这样在经过代码审查后,你可以确信你的代码中不再存在这类错误了。 6#z8 %kaX
E !kN h
'2^}de!E
Phn^0 iF
一、常见错误1# :多次拷贝字符串 GfONm6A
L3eF BF/
,DFN:uf=l
P(aBJ*((~
测试所不能发现的一个错误是生成不可变(immutable)对象的多份拷贝。不可变对象是不可改变的,因此不需要拷贝它。最常用的不可变对象是String。 UC`h o%OBF
KL$.E!d
a%%7Ew ?
EyK!'9~a
如果你必须改变一个String对象的内容,你应该使用StringBuffer。下面的代码会正常工作: ZF7n]LgSc&
g QBS#NY
PQaTS*0SXJ
dz^HN`AlzC
String s = new String ("Text here"); Gu$/rb?
cH_qHXi[G
L^qCE-[
,^9+G"H:I
但是,这段代码性能差,而且没有必要这么复杂。你还可以用以下的方式来重写上面的代码: fI1CT)0<e
A7L; ims7
[4"(\r\f
P^te
String temp = "Text here"; f ,e]jw@
String s = new String (temp); /pF8S!,z
d+DO}=]
;hQ[-
j/t%7,
但是这段代码包含额外的String,并非完全必要。更好的代码为: 8ZtJvk`
"Q@m7j)(
@`[e1KQ
k$$SbStD
String s = "Text here"; Z_GGH2u
ct\msG }b:
i!YfR]"}
_hY6NMw
二、常见错误2#: 没有克隆(clone)返回的对象 ?o(284sV3
:!Ci#[g
OU{c|O
Kw-<