假設一段代碼是不穩(wěn)定或者有一些潛在問題的,那么代碼往往會包括一些明顯的痕跡。
正如食物要腐壞之前召娜,常常會發(fā)出一些異味一樣恢暖。
我們管這些痕跡叫做“代碼異味”排监。
參考資料:
Duplicated Code --------(反復代碼)難維護。
解決方法:提取公共函數(shù)杰捂。
- Long Method ------------(函數(shù)長)難理解舆床。
解決方法:拆分成若干函數(shù)。
- Large Class ------------(類大)難理解嫁佳。
解決方法:拆分成若干類挨队。
- Long Parameter List ----(參數(shù)多)難用,難理解蒿往。
解決方法:將參數(shù)封裝成結(jié)構(gòu)或者類盛垦。
- Divergent Change -------(萬能類)發(fā)散試改動,改好多需求,都會動他瓤漏。
解決方法:拆腾夯,將總是一起變化的東西放在一塊兒。
- Shotgun Surgery --------(天女散花的邏輯)散彈試改動,改某個需求的時候蔬充,要改非常多類蝶俱。
解決方法:將各個改動點,集中起來娃惯,抽象成一個新類跷乐。
- Feature Envy -----------(紅杏出墻的函數(shù))使用了大量其它類的成員
解決方法:將這個函數(shù)挪到那個類里面。
- Data Clumps ------------(數(shù)據(jù)團)常一起出現(xiàn)的一坨數(shù)據(jù)趾浅。
解決方法:他們那么有基情愕提,就在一起吧馒稍,給他們一個新的類。
- Primitive Obsession ----(偏愛基本類型)熱衷于使用int,long,String等基本類型浅侨。
解決方法:重復出現(xiàn)的一組參數(shù)纽谒,有關(guān)聯(lián)的多個數(shù)組換成類吧。
- Switch Statements ------(switch語句)
解決方法:state/strategy 或者僅僅是簡單的多態(tài)如输。
- Parallel Inheritance Hierarchies ---------------(平行繼承)添加A類的子類ax鼓黔,B類也須要對應的添加一個bx。
解決方法:應該有一個類是能夠去掉繼承關(guān)系的不见。
- Lazy Class -------------------------------------(冗贅類)假設他不干活了澳化,炒掉他吧。
解決方法:把這些不再重要的類里面的邏輯稳吮,合并到相關(guān)類缎谷,刪掉舊的。
- Speculative Generality -------------------------(夸夸其談未來性)
解決方法:刪掉
- Temporary Field --------------------------------(暫時字段)僅在特定環(huán)境下使用的變量
解決方法:將這些暫時變量集中到一個新類中管理灶似。
- Message Chains ---------------------------------(消息鏈)過度耦合的才是壞的列林。
解決方法:拆函數(shù)或者移動函數(shù)。
- Middle Man -------------------------------------(中介)大部分都交給中介來處理了酪惭。
解決方法:用繼承替代托付希痴。
- Inappropriate Intimacy -------------------------(太親熱)兩個類彼此使用對方的私有的東西。
解決方法:劃清界限拆散春感,或合并砌创,或改成單項聯(lián)系。
- Alternative Classes with Different Interfaces --(相似的類甥厦,有不同接口)
解決方法:重命名纺铭,移動函數(shù),或抽象子類刀疙。
- Incomplete Library Class -----------------------(不完好的類庫)
解決方法:包一層函數(shù)或包成新的類舶赔。
- Data Class -------------(純數(shù)據(jù)類)類非常easy,僅有公共成員變量谦秧,或簡單操作函數(shù)竟纳。
解決方法:將相關(guān)操作封裝進去,降低public成員變量疚鲤。
- Refused Bequest --------(繼承過多)父類里面方法非常多锥累,子類僅僅用有限幾個。
解決方法:用代理替代繼承關(guān)系集歇。
- Comments ---------------(太多凝視)這里指代碼太難懂了桶略,不得不用凝視解釋。
解決方法:避免用凝視解釋代碼,而是說明代碼的目的际歼,背景等惶翻。好代碼會說話。