1.DuplicatedCode
代碼重複幾乎是最常見的異味了。他也是Refactoring的主要目标之一。代碼重複往往來自于copy-and-paste 的程式設計風格。
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。
代碼的壞味道 | 一般重構方法 | 使用模式重構 |
重複代碼 | 提煉方法 提取類 方法上移 替換算法 鍊構造方法 | 構造Template Method 以Composite取代一/多之分 引入Null Object 用Adapter統一接口 用Fatory Method引入多态建立 |
過長方法 | 提取方法 組合方法 以查詢取代臨時變量 引入參數對象 保持對象完整 | 轉移聚集操作到Vistor 以Strategy取代條件邏輯 以Command取代條件排程程式 轉移聚集操作到Collecting Parameter |
過長參數列 | 以方法取代參數 引入參數對象 保持對象完整 | |
條件邏輯過度複雜 | 分解條件式 合并條件式 合并重複的條件片段 移除控制标記 以衛語句取代嵌套條件式 以多态取代條件式 引入斷言 | 以Strategy取代條件邏輯 轉移裝飾功能到Decorator 以State取代狀态改變條件語句 引入Null Object |
分支語句 | 提取方法 轉移方法 以子類取代類型代碼 以多态取代條件式 已明确方法取代參數 | 以State/Strategy取代類型代碼 引入Null Object 以Command替換條件排程程式 轉移聚集操作到Visitor |
基本類型迷戀 程式代碼過于依賴基本類型(int,string,double,array等低層次語言要素) | 以對象取代資料值 以類型取代類型代碼 以子類取代類型代碼 提取類 引入參數對象 以對象取代數組 | 以State取代狀态改變條件語句 以Strategy取代條件邏輯 以Composite取代隐含樹 以Interpreter取代隐式語言 轉移裝飾功能到Decorator 用Builder封裝Composite |
資料泥團 在類的字段和參數列中,總是一起出現的資料 | 提取類 引入參數對象 保持對象完整 | |
令人迷惑的臨時字段 | 提取類 | 引入Null Object |
組合爆炸 許多段代碼使用不同種類或數量的資料或對象做同樣的事情(例如使用特定條件和資料庫查詢) | 以Interpreter取代隐式語言 | |
過大類 | 提取類 提取子類 提取接口 複制被監視資料 | 以Command取代條件排程程式 以State取代狀态改變條件語句 以Interpreter取代隐式語言 |
冗贅類 不再做很多工作或沒有用的類 | 折疊繼承關系 内聯Singleton | |
不恰當的暴露 在客戶代碼中不應看到類的字段和方法,卻是公開可見的 | 封裝字段 封裝群集 移除設定方法 隐藏方法 | 用Factory封裝類 |
發散式變化 類經常因為不同的原因在不同方向上發生變化,顯然是違反了單一職責原則 | 提取類 | |
霰彈式修改 如果遇到變化,必須在不同的類中作出相應的修改 | 轉移方法 轉移字段 内聯類 | 将建立知識搬移到Factory |
依戀情結 方法對于某個類的興趣高過對自己所處的宿主類 | 轉移方法 提取方法 | 引入Strategy 引入Visitor |
平行繼承體系 當為一個類增加一個子類時,也必須在另一個類中增加一個相應的子類 | 轉移方法 轉移字段 | |
誇誇其談未來性 | 折疊繼承關系 内聯類 移除參數 移除方法 | |
過度耦合的消息連 不斷的向一個對象索求另一個對象 | 隐藏委托 提取方法 轉移方法 | 使用抽象引入Chain Of Responsibility |
中間轉手人 類接口中有很多方法都委托給其他類 | 移除中間轉手人 内聯方法 以繼承取代委托 | |
狎昵關系 類之間彼此依賴于其private成員 | 轉移方法 将雙向關聯改為單向 提取類 隐藏委托 以繼承取代委托 | |
異曲同工的類 | 重命名方法 轉移方法 提取超類 | 用Adapter統一接口 |
不完善的程式庫類 | 引入外加方法 引入本地擴充 | 用Adapter統一接口 用Facade封裝類 |
純稚的資料類 隻擁有字段的資料類 | 封裝字段 封裝集合 移除設定方法 轉移方法 隐藏方法 | |
被拒絕的遺贈 繼承父類時,子類想要選擇繼承的成員 | 以委托取代繼承 | |
過多的注釋 為糟糕的代碼寫大量的注釋 | 使用一起重構方法,使方法本身達到自說明的效果,讓注釋顯得多餘 | |
怪異解決方案 在同一系統中使用不同的方式解決同一問題 | 替換算法 | 用Adapter統一接口 |