[TOC] ## 代碼重復 ## 冗余的子程序 ## 循環過長或嵌套過深 ## 內聚性太差的類 內聚性太差的類如果看到有某個類大包大攬了許多彼此無關的任務，那么這個類就該被拆分成多個類，每個類負責一組具有內在的相互關聯的任務。 ## 類的接口未能提供層次一致的抽象 即使是那些從誕生之日起就具有內聚接口的類也可能漸漸失去最初的一致性。 為了維護接口的完整性,程序員常常會在怒之下對類動手,這樣的草率修改會使得類隨著時間的推移不斷地發生變化。最終,類的接口會變成維護的法蘭肯斯坦( Frankensteinian,寓人造怪物)惡魔,對程序的可管理性毫無裨益 ## 擁有太多參數的參數列表 ## 類的內部修改往往被局限于某個部分 有時一個類會有這兩種或更多獨立的功能。如果你發現自己要么修改類里的這一部分,要么修改另一部分,但極少的修改會同時影響類中的兩個部分,這就表明該類應該根據相互獨立的功能被拆分為多個類。 ## 變化導致對多個類的相同修改 應該使修改僅僅影響一個類 ## 對繼承體系的同樣修改 ## 同時使用的相關數據并未以類的方式進行組織 如果看到自己常常對同樣組數據進行操作,你也應當問問自己是否該將這些數據及其操作組織到一個類里面 ## 成員函數使用其他類的特征比使用自身類的特征還要多 這一狀況暗示著這子程序應該被放到另一個類中,然后在原來的類里調用 ## 過多使用基本數據類型 基礎類型添加別名,可以預防兩個不相等的類型之間比較 ## 子程序命名不恰當 ## 數據成員被設置為公用 ## 某個派生類僅使用了基類的很少一部分成員函數 ## 注釋被用于解釋難懂的代碼 ## 使用了全局變量 ## 在子程序調用前使用了設置代碼( setup code) ## 在調用后使用了收尾代碼( takedown code)這樣的代碼應當看作是一種警告