我爱电脑技术论坛 » 编程语言 » 使用Java语言编写程序最容易犯的21种错误

bbs23 发表于 2008-4-8 08:40

使用Java语言编写程序最容易犯的21种错误

1.Duplicated Code
c&S Un(G4U,Vn.u
Lw9k
iH.Q2uw

$NV\7f*?Y)p 代码重复几乎是最常见的异味了。他也是Refactoring的主要目标之一。代码重复往往来自于copy-and-paste的编程风格。与他相对应OAOO是一个好系统的重要标志。 K*[N
u)qH5cMP%W
$t q$X&Y/oI

J$V6`&?\^^u 2.Long method
+@4\+k1e
u5j_v8t
;oZZ$z*}'^'~ ]^
/a4J(t zK(T 它是传统结构化的“遗毒”。一个方法应当具有自我独立的意图，不要把几个意图放在一起。
o Z6EF:N7__HVN
/M]"L%qP!ZB-I9y
%OG2N2\DL5f 3.Large Class
x0[r)oF Q&M;h
CuA0V ^y,?2Z nK']1mVz F8^
大类就是你把太多的责任交给了一个类。这里的规则是One Class One Responsibility. }uql"F1F-\
K/h
tW_QbS7I&}#H

w'B4V`~F 4.Divergent Change 1g/Y[#yq/}h6k
)QG6eg@~\r

yC7jm"T a 一个类里面的内容变化率不同。某些状态一个小时变一次，某些则几个月一年才变一次；某些状态因为这方面的原因发生变化，而另一些则因为其他方面的原因变一次。面向对象的抽象就是把相对不变的和相对变化相隔离。把问题变化的一方面和另一方面相隔离。这使得这些相对不变的可以重用。问题变化的每个方面都可以单独重用。这种相异变化的共存使得重用非常困难。 (@;z#V9N$sQj0[9b

$\v,w l0G3f
X!]|4ChNJ)Y)p 5.Shotgun Surgery i]7WOt8qG
'DDPa@ n3B
Bk)v
H|/d)G;k
t
这正好和上面相反。对系统一个地方的改变涉及到其他许多地方的相关改变。这些变化率和变化内容相似的状态和行为通常应当放在同一个类中。
@+Pz*aO$g
+A6Z,UL?;Kd b 7`k.D
G~]6H1N
6.Feature Envy
u5hU kGw
1xk;v.q1H
K&i%~'pl
`4BsOd"i 对象的目的就是封装状态以及与这些状态紧密相关的行为。如果一个类的方法频繁用get 方法存取其他类的状态进行计算，那么你要考虑把行为移到涉及状态数目最多的那个类。 '~#Qw*J5k4?(`J

8F5_/S`n| 'L W]
c:h3_P`
7.Data Clumps
E'L%`M4?:xZ#l
PKki0i j+^"~4{f(p !^(l(u?!E c)by
某些数据通常像孩子一样成群玩耍：一起出现在很多类的成员变量中，一起出现在许多方法的参数中，这些数据或许应该自己独立形成对象。 8r;r
Xx c

7rW%\)R5X1t -d`%C I] ?q+g*b
8.Primitive Obsession [L1d'}T N/G

M-f {(h*HR
6lvV%`MO8S 面向对象的新手通常习惯使用几个原始类型的数据来表示一个概念。譬如对于范围，他们会使用两个数字。对于Money，他们会用一个浮点数来表示。因为你没有使用对象来表达问题中存在的概念，这使得代码变的难以理解，解决问题的难度大大增加。好的习惯是扩充语言所能提供原始类型，用小对象来表示范围、金额、转化率、邮政编码等等。 0M-T$W'aPh[z`5?
+X:u8kH/v/v

uShjDif/C;J6JQ 9.Switch Statement lUyR8zcG
];s
[?,H
VTA9i,h#w6G,I
基于常量的开关语句是OO 的大敌，你应当把他变为子类、state或strategy. )mg;G&x:Wd-^L
h,loFS9E`;I(q\om

S
N j:J_;b!ms%h 10. Parallel Inheritance Hierarchies
"@'k,hO.I*EU
+A^(v(?(u]GG3eN)e P[\O"|&e&x|
并行的继承层次是shotgun surgery的特殊情况。因为当你改变一个层次中的某一个类时，你必须同时改变另外一个层次的并行子类。
w+BU}oB*vc
1tH-f1a7ItGD
zQ
X6e%b`wMV w8i1i*PC 11. Lazy Class
ak*N]T,P,cV
!hP[W8iS5v
:@
lE:CD;{*V0J O-e"e 一个干活不多的类。类的维护需要额外的开销，如果一个类承担了太少的责任，应当消除它。
w$iK hz
]V_%qAV7t
hq^oAa 12. Speculative Generality
dSx
d.}j5t
s6D{(mI? O6H!RxP.F(]Nl
一个类实现了从未用到的功能和通用性。通常这样的类或方法唯一的用户是testcase.不要犹豫，删除它。 9@CP*gM~+eD

5Fj*C6gm0f%w J5[4B,zV!U4^ p@
13. Temporary Field +GOl*_.~ w
U2b+egU_6G
q

w6PJ(yeoq0~r 一个对象的属性可能只在某些情况下才有意义。这样的代码将难以理解。专门建立一个对象来持有这样的孤儿属性，把只和他相关的行为移到该类。最常见的是一个特定的算法需要某些只有该算法才有用的变量。 |hSs6n-TJVaI
:H1W6Q+n2J'LdA

jqe| {j$_/E'vaVk 14. Message Chain cqN?F
.mJLTr

[R%z%}{[A0t0lZ~x 消息链发生于当一个客户向一个对象要求另一个对象，然后客户又向这另一对象要求另一个对象，再向这另一个对象要求另一个对象，如此如此。这时，你需要隐藏分派。 5B{8H,i7I%c'A

7a)Y(aMm uk3A
DW~9P)Uvp)ej7P E 15. Middle Man
!u%k&A
YQL
}t%J~+H
@JO^6uQM#h 对象的基本特性之一就是封装，而你经常会通过分派去实现封装。但是这一步不能走得太远，如果你发现一个类接口的一大半方法都在做分派，你可能需要移去这个中间人。 1ut? q(t5aA W(~
+\\Nko%i@8uG
W&i}v0e8hB-Lw([
16. Inappropriate Intimacy
y1M9t?.K-M]
an9]/s)~v 0R"X6BW/X H
某些类相互之间太亲密，它们花费了太多的时间去砖研别人的私有部分。对人类而言，我们也许不应该太假正经，但我们应当让自己的类严格遵守禁欲主义。
kCov(~&J2p;} |(q'p-A])B
TgK(a(q*B
@
17. Alternative Classes with Different Interfaces h fF,yq+Z9v
+BpN\!T{y9iY
.tk4j:p&r!P;jp
做相同事情的方法有不同的函数signature，一致把它们往类层次上移，直至协议一致。
t6}.|!b+Q;q
%e%bJC/t d_AG*h7x)xK
18. Incomplete Library Class
0Y%@}*r wP1p(g
1[
`]5J-G4A-EH7j_u(O;M
-G6clEDVQV 要建立一个好的类库非常困难。我们大量的程序工作都基于类库实现。然而，如此广泛而又相异的目标对库构建者提出了苛刻的要求。库构建者也不是万能的。有时候我们会发现库类无法实现我们需要的功能。而直接对库类的修改有非常困难。这时候就需要用各种手段进行Refactoring.
uPO7FH7Y9tW |
V8P:dI-e
X#m%ap%Bz
19. Data Class -e k*|;q-X_
O'WIX0]l%Em

C,lK!^9X0a.cf 对象包括状态和行为。如果一个类只有状态没有行为，那么肯定有什么地方出问题了。
1I6wdrV;R
-H5j/N}6k
*_n1YX8Z;LA 20. Refused Bequest
nl"\7d"X8d2V.C +Aooi-JD)GHRT
&N,X.dS_G8XiT
超类传下来很多行为和状态，而子类只是用了其中的很小一部分。这通常意味着你的类层次有问题。
w.Q Q:oTJ.K:Q
nH GvQ&i5fc$i/x
Sf5[|Yr~U 21. Comments
1\\ zXBlH2d0~P+[ f .`r'B+C0o9b
LT!x1@8e {q e-Z$u
经常觉得要写很多注释表示你的代码难以理解。如果这种感觉太多，表示你需要Refactoring。

查看完整版本: 使用Java语言编写程序最容易犯的21种错误