21個(gè)java程序經(jīng)常犯的錯(cuò)誤

1.Duplicated Code

代碼重復(fù)幾乎是常見的異味了。他也是Refactoring的主要目標(biāo)之一。代碼重復(fù)往往來自于copy-and-paste的編程風(fēng)格。與他相對應(yīng)OAOO是一個(gè)好系統(tǒng)的重要標(biāo)志。

2.Long method

它是傳統(tǒng)結(jié)構(gòu)化的“遺毒”。一個(gè)方法應(yīng)當(dāng)具有自我獨(dú)立的意圖，不要把幾個(gè)意圖放在一起。

3.Large Class

大類就是你把太多的責(zé)任交給了一個(gè)類。這里的規(guī)則是One Class One Responsibility.

4.Divergent Change

一個(gè)類里面的內(nèi)容變化率不同。某些狀態(tài)一個(gè)小時(shí)變一次，某些則幾個(gè)月一年才變一次；某些狀態(tài)因?yàn)檫@方面的原因發(fā)生變化，而另一些則因?yàn)槠渌矫娴脑蜃円淮?。面向?qū)ο蟮某橄缶褪前严鄬Σ蛔兊暮拖鄬ψ兓喔綦x。把問題變化的一方面和另一方面相隔離。這使得這些相對不變的可以重用。問題變化的每個(gè)方面都可以單獨(dú)重用。這種相異變化的共存使得重用非常困難。

5.Shotgun Surgery

這正好和上面相反。對系統(tǒng)一個(gè)地方的改變涉及到其他許多地方的相關(guān)改變。這些變化率和變化內(nèi)容相似的狀態(tài)和行為通常應(yīng)當(dāng)放在同一個(gè)類中。

6.Feature Envy

對象的目的就是封裝狀態(tài)以及與這些狀態(tài)緊密相關(guān)的行為。如果一個(gè)類的方法頻繁用get 方法存取其他類的狀態(tài)進(jìn)行計(jì)算，那么你要考慮把行為移到涉及狀態(tài)數(shù)目多的那個(gè)類。

7.Data Clumps

某些數(shù)據(jù)通常像孩子一樣成群玩耍：一起出現(xiàn)在很多類的成員變量中，一起出現(xiàn)在許多方法的參數(shù)中，這些數(shù)據(jù)或許應(yīng)該自己獨(dú)立形成對象。

8.Primitive Obsession

面向?qū)ο蟮男率滞ǔＡ?xí)慣使用幾個(gè)原始類型的數(shù)據(jù)來表示一個(gè)概念。譬如對于范圍，他們會(huì)使用兩個(gè)數(shù)字。對于Money，他們會(huì)用一個(gè)浮點(diǎn)數(shù)來表示。因?yàn)槟銢]有使用對象來表達(dá)問題中存在的概念，這使得代碼變的難以理解，解決問題的難度大大增加。好的習(xí)慣是擴(kuò)充語言所能提供原始類型，用小對象來表示范圍、金額、轉(zhuǎn)化率、郵政編碼等等。

9.Switch Statement

基于常量的開關(guān)語句是OO 的大敵，你應(yīng)當(dāng)把他變?yōu)樽宇悺tate或strategy.

10. Parallel Inheritance Hierarchies

并行的繼承層次是shotgun surgery的特殊情況。因?yàn)楫?dāng)你改變一個(gè)層次中的某一個(gè)類時(shí)，你必須同時(shí)改變另外一個(gè)層次的并行子類。

11. Lazy Class

一個(gè)干活不多的類。類的維護(hù)需要額外的開銷，如果一個(gè)類承擔(dān)了太少的責(zé)任，應(yīng)當(dāng)消除它。

12. Speculative Generality

一個(gè)類實(shí)現(xiàn)了從未用到的功能和通用性。通常這樣的類或方法唯一的用戶是testcase.不要猶豫，刪除它。

13. Temporary Field

一個(gè)對象的屬性可能只在某些情況下才有意義。這樣的代碼將難以理解。專門建立一個(gè)對象來持有這樣的孤兒屬性，把只和他相關(guān)的行為移到該類。常見的是一個(gè)特定的算法需要某些只有該算法才有用的變量。

14. Message Chain

消息鏈發(fā)生于當(dāng)一個(gè)客戶向一個(gè)對象要求另一個(gè)對象，然后客戶又向這另一對象要求另一個(gè)對象，再向這另一個(gè)對象要求另一個(gè)對象，如此如此。這時(shí)，你需要隱藏分派。

15. Middle Man

對象的基本特性之一就是封裝，而你經(jīng)常會(huì)通過分派去實(shí)現(xiàn)封裝。但是這一步不能走得太遠(yuǎn)，如果你發(fā)現(xiàn)一個(gè)類接口的一大半方法都在做分派，你可能需要移去這個(gè)中間人。

16. Inappropriate Intimacy

某些類相互之間太親密，它們花費(fèi)了太多的時(shí)間去磚研別人的私有部分。對人類而言，我們也許不應(yīng)該太假正經(jīng)，但我們應(yīng)當(dāng)讓自己的類嚴(yán)格遵守禁欲主義。

17. Alternative Classes with Different Interfaces

做相同事情的方法有不同的函數(shù)signature，一致把它們往類層次上移，直至協(xié)議一致。

18. Incomplete Library Class

要建立一個(gè)好的類庫非常困難。我們大量的程序工作都基于類庫實(shí)現(xiàn)。然而，如此廣泛而又相異的目標(biāo)對庫構(gòu)建者提出了苛刻的要求。庫構(gòu)建者也不是萬能的。有時(shí)候我們會(huì)發(fā)現(xiàn)庫類無法實(shí)現(xiàn)我們需要的功能。而直接對庫類的修改有非常困難。這時(shí)候就需要用各種手段進(jìn)行Refactoring.

19. Data Class

對象包括狀態(tài)和行為。如果一個(gè)類只有狀態(tài)沒有行為，那么肯定有什么地方出問題了。

20. Refused Bequest

超類傳下來很多行為和狀態(tài)，而子類只是用了其中的很小一部分。這通常意味著你的類層次有問題。

21. Comments

經(jīng)常覺得要寫很多注釋表示你的代碼難以理解。如果這種感覺太多，表示你需要Refactoring。