寫Java程式最容易犯的21種錯誤

程式|錯誤

1．DuplicatedCode

代碼重複幾乎是最常見的異味了。他也是Refactoring的主要目標之一。代碼重複往往來自於copy-and-paste

的編程風格。與他相對應OAOO是一個好系統的重要標誌（請參見我的duplicatedcode一文：http://www.erptao.org/download.php?op=viewsdownload&sid=6）。

2．Longmethod

它是傳統結構化的“遺毒“。一個方法應當具有自我獨立的意圖，不要把幾個意圖放在一起，我的《大類和長方法》一文中有詳細描述。

3．LargeClass

大類就是你把太多的責任交給了一個類。這裡的規則是OneClassOneResponsibility。

4．DivergentChange

一個類裡面的內容變動率不同。某些狀態一個小時變一次，某些則幾個月一年才變一次；某些狀態因為這方面的原因發生變化，而另一些則因為其他方面的原因變一次。物件導向的抽象就是把相對不變的和相對變化相隔離。把問題變化的一方面和另一方面相隔離。這使得這些相對不變的可以重用。問題變化的每個方面都可以單獨重用。這種相異變化的共存使得重用非常困難。

5．ShotgunSurgery

這正好和上面相反。對系統一個地方的改變涉及到其他許多地方的相關改變。這些變動率和變化內容相似的狀態和行為通常應當放在同一個類中。

6．FeatureEnvy

對象的目的就是封裝狀態以及與這些狀態緊密相關的行為。如果一個類的方法頻繁用get方法存取其他類的狀態進行計算，那麼你要考慮把行為移到涉及狀態數目最多的那個類。

7．DataClumps

某些資料通常像孩子一樣成群玩耍：一起出現在很多類的成員變數中，一起出現在許多方法的參數中…..，這些資料或許應該自己獨立形成對象。

8．PrimitiveObsession

物件導向的新手通常習慣使用幾個原始類型的資料來表示一個概念。譬如對於範圍，他們會使用兩個數字。對於Money，他們會用一個浮點數來表示。因為你沒有使用對象來表達問題中存在的概念，這使得代碼變的難以理解，解決問題的難度大大增加。

好的習慣是擴充語言所能提供原始類型，用小對象來表示範圍、金額、轉化率、郵遞區號等等。

9．SwitchStatement

基於常量的開關語句是OO的大敵，你應當把他變為子類、state或strategy。

10．ParallelInheritanceHierarchies

並行的繼承層次是shotgunsurgery的特殊情況。因為當你改變一個層次中的某一個類時，你必須同時改變另外一個層次的並行子類。

11．LazyClass

一個幹活不多的類。類的維護需要額外的開銷，如果一個類承擔了太少的責任，應當消除它。

12．SpeculativeGenerality

一個類實現了從未用到的功能和通用性。通常這樣的類或方法唯一的使用者是testcase。不要猶豫，刪除它。

13．TemporaryField

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

14．MessageChain

訊息鏈發生於當一個客戶向一個對象要求另一個對象，然後客戶又向這另一對象要求另一個對象，再向這另一個對象要求另一個對象，如此如此。這時，你需要隱藏指派。

15．MiddleMan

對象的基本特性之一就是封裝，而你經常會通過指派去實現封裝。但是這一步不能走得太遠，如果你發現一個類介面的一大半方法都在做指派，你可能需要移去這個中間

人。

16．InappropriateIntimacy

某些類相互之間太親密，它們花費了太多的時間去磚研別人的私人部分。對人類而言，我們也許不應該太假正經，但我們應當讓自己的類嚴格遵守禁慾主義。

17．AlternativeClasseswithDifferentInterfaces

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

18．IncompleteLibraryClass

要建立一個好的類庫非常困難。我們大量的程式工作都基於類庫實現。然而，如此廣泛而又相異的目標對庫構建者提出了苛刻的要求。庫構建者也不是萬能的。有時候我們會發現庫類無法實現我們需要的功能。而直接對庫類的修改有非常困難。這時候就需要用各種手段進行Refactoring。

19．DataClass

對象包括狀態和行為。如果一個類只有狀態沒有行為，那麼肯定有什麼地方出問題了。

20．RefusedBequest

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

21．Comments

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