Duplicated Code
代碼重復幾乎是最常見的異味了他也是Refactoring的主要目標之一代碼重復往往來自於copyandpaste的編程風格與他相對應OAOO是一個好系統的重要標志
Long method
它是傳統結構化的遺毒一個方法應當具有自我獨立的意圖不要把幾個意圖放在一起
Large Class
大類就是你把太多的責任交給了一個類這裡的規則是One Class One Responsibility
Divergent Change
一個類裡面的內容變化率不同某些狀態一個小時變一次某些則幾個月一年才變一次;某些狀態因為這方面的原因發生變化而另一些則因為其他方面的原因變一次面向對象的抽象就是把相對不變的和相對變化相隔離把問題變化的一方面和另一方面相隔離這使得這些相對不變的可以重用問題變化的每個方面都可以單獨重用這種相異變化的共存使得重用非常困難
Shotgun Surgery
這正好和上面相反對系統一個地方的改變涉及到其他許多地方的相關改變這些變化率和變化內容相似的狀態和行為通常應當放在同一個類中
Feature Envy
對象的目的就是封裝狀態以及與這些狀態緊密相關的行為如果一個類的方法頻繁用get 方法存取其他類的狀態進行計算那麼你要考慮把行為移到涉及狀態數目最多的那個類
Data Clumps
某些數據通常像孩子一樣成群玩耍一起出現在很多類的成員變量中一起出現在許多方法的參數中這些數據或許應該自己獨立形成對象
Primitive Obsession
面向對象的新手通常習慣使用幾個原始類型的數據來表示一個概念譬如對於范圍他們會使用兩個數字對於Money他們會用一個浮點數來表示因為你沒有使用對象來表達問題中存在的概念這使得代碼變的難以理解解決問題的難度大大增加好的習慣是擴充語言所能提供原始類型用小對象來表示范圍金額轉化率郵政編碼等等
Switch Statement
基於常量的開關語句是OO 的大敵你應當把他變為子類state或strategy
Parallel Inheritance Hierarchies
並行的繼承層次是shotgun surgery的特殊情況因為當你改變一個層次中的某一個類時你必須同時改變另外一個層次的並行子類
Lazy Class
一個干活不多的類類的維護需要額外的開銷如果一個類承擔了太少的責任應當消除它
Speculative Generality
一個類實現了從未用到的功能和通用性通常這樣的類或方法唯一的用戶是testcase不要猶豫刪除它
Temporary Field
一個對象的屬性可能只在某些情況下才有意義這樣的代碼將難以理解專門建立一個對象來持有這樣的孤兒屬性把只和他相關的行為移到該類最常見的是一個特定的算法需要某些只有該算法才有用的變量
Message Chain
消息鏈發生於當一個客戶向一個對象要求另一個對象然後客戶又向這另一對象要求另一個對象再向這另一個對象要求另一個對象如此如此這時你需要隱藏分派
Middle Man
對象的基本特性之一就是封裝而你經常會通過分派去實現封裝但是這一步不能走得太遠如果你發現一個類接口的一大半方法都在做分派你可能需要移去這個中間人
Inappropriate Intimacy
某些類相互之間太親密它們花費了太多的時間去磚研別人的私有部分對人類而言我們也許不應該太假正經但我們應當讓自己的類嚴格遵守禁欲主義
Alternative Classes with Different Interfaces
做相同事情的方法有不同的函數signature一致把它們往類層次上移直至協議一致
Incomplete Library Class
要建立一個好的類庫非常困難我們大量的程序工作都基於類庫實現然而如此廣泛而又相異的目標對庫構建者提出了苛刻的要求庫構建者也不是萬能的有時候我們會發現庫類無法實現我們需要的功能而直接對庫類的修改有非常困難這時候就需要用各種手段進行Refactoring
Data Class
對象包括狀態和行為如果一個類只有狀態沒有行為那麼肯定有什麼地方出問題了
Refused Bequest
超類傳下來很多行為和狀態而子類只是用了其中的很小一部分這通常意味著你的類層次有問題
Comments
經常覺得要寫很多注釋表示你的代碼難以理解如果這種感覺太多表示你需要Refactoring
From:http://tw.wingwit.com/Article/program/Java/hx/201311/27238.html
