22種代碼的壞味道(轉載)

假設一段代碼是不穩定或者有一些潛在問題的,那么代碼往往會包括一些明顯的痕跡。
正如食物要腐壞之前,常常會發出一些異味一樣。
我們管這些痕跡叫做“代碼異味”。

參考資料:

Bad Smell(代碼的壞味道)

解決方法:提取公共函數。

解決方法:拆分成若干函數。

解決方法:拆分成若干類。

解決方法:將參數封裝成結構或者類。

  • Divergent Change -------(萬能類)發散試改動,改好多需求,都會動他。

解決方法:拆,將總是一起變化的東西放在一塊兒。

  • Shotgun Surgery --------(天女散花的邏輯)散彈試改動,改某個需求的時候,要改非常多類。

解決方法:將各個改動點,集中起來,抽象成一個新類。

  • Feature Envy -----------(紅杏出墻的函數)使用了大量其它類的成員

解決方法:將這個函數挪到那個類里面。

  • Data Clumps ------------(數據團)常一起出現的一坨數據。

解決方法:他們那么有基情,就在一起吧,給他們一個新的類。

  • Primitive Obsession ----(偏愛基本類型)熱衷于使用int,long,String等基本類型。

解決方法:重復出現的一組參數,有關聯的多個數組換成類吧。

解決方法:state/strategy 或者僅僅是簡單的多態。

解決方法:應該有一個類是能夠去掉繼承關系的。

  • Lazy Class -------------------------------------(冗贅類)假設他不干活了,炒掉他吧。

解決方法:把這些不再重要的類里面的邏輯,合并到相關類,刪掉舊的。

解決方法:刪掉

  • Temporary Field --------------------------------(暫時字段)僅在特定環境下使用的變量

解決方法:將這些暫時變量集中到一個新類中管理。

  • Message Chains ---------------------------------(消息鏈)過度耦合的才是壞的。

解決方法:拆函數或者移動函數。

  • Middle Man -------------------------------------(中介)大部分都交給中介來處理了。

解決方法:用繼承替代托付。

  • Inappropriate Intimacy -------------------------(太親熱)兩個類彼此使用對方的私有的東西。

解決方法:劃清界限拆散,或合并,或改成單項聯系。

解決方法:重命名,移動函數,或抽象子類。

解決方法:包一層函數或包成新的類。

  • Data Class -------------(純數據類)類非常easy,僅有公共成員變量,或簡單操作函數。

解決方法:將相關操作封裝進去,降低public成員變量。

  • Refused Bequest --------(繼承過多)父類里面方法非常多,子類僅僅用有限幾個。

解決方法:用代理替代繼承關系。

  • Comments ---------------(太多凝視)這里指代碼太難懂了,不得不用凝視解釋。

解決方法:避免用凝視解釋代碼,而是說明代碼的目的,背景等。好代碼會說話。

最后編輯于
?著作權歸作者所有,轉載或內容合作請聯系作者
平臺聲明:文章內容(如有圖片或視頻亦包括在內)由作者上傳并發布,文章內容僅代表作者本人觀點,簡書系信息發布平臺,僅提供信息存儲服務。
  • 序言:七十年代末,一起剝皮案震驚了整個濱河市,隨后出現的幾起案子,更是在濱河造成了極大的恐慌,老刑警劉巖,帶你破解...
    沈念sama閱讀 230,182評論 6 543
  • 序言:濱河連續發生了三起死亡事件,死亡現場離奇詭異,居然都是意外死亡,警方通過查閱死者的電腦和手機,發現死者居然都...
    沈念sama閱讀 99,489評論 3 429
  • 文/潘曉璐 我一進店門,熙熙樓的掌柜王于貴愁眉苦臉地迎上來,“玉大人,你說我怎么就攤上這事。” “怎么了?”我有些...
    開封第一講書人閱讀 178,290評論 0 383
  • 文/不壞的土叔 我叫張陵,是天一觀的道長。 經常有香客問我,道長,這世上最難降的妖魔是什么? 我笑而不...
    開封第一講書人閱讀 63,776評論 1 317
  • 正文 為了忘掉前任,我火速辦了婚禮,結果婚禮上,老公的妹妹穿的比我還像新娘。我一直安慰自己,他們只是感情好,可當我...
    茶點故事閱讀 72,510評論 6 412
  • 文/花漫 我一把揭開白布。 她就那樣靜靜地躺著,像睡著了一般。 火紅的嫁衣襯著肌膚如雪。 梳的紋絲不亂的頭發上,一...
    開封第一講書人閱讀 55,866評論 1 328
  • 那天,我揣著相機與錄音,去河邊找鬼。 笑死,一個胖子當著我的面吹牛,可吹牛的內容都是我干的。 我是一名探鬼主播,決...
    沈念sama閱讀 43,860評論 3 447
  • 文/蒼蘭香墨 我猛地睜開眼,長吁一口氣:“原來是場噩夢啊……” “哼!你這毒婦竟也來了?” 一聲冷哼從身側響起,我...
    開封第一講書人閱讀 43,036評論 0 290
  • 序言:老撾萬榮一對情侶失蹤,失蹤者是張志新(化名)和其女友劉穎,沒想到半個月后,有當地人在樹林里發現了一具尸體,經...
    沈念sama閱讀 49,585評論 1 336
  • 正文 獨居荒郊野嶺守林人離奇死亡,尸身上長有42處帶血的膿包…… 初始之章·張勛 以下內容為張勛視角 年9月15日...
    茶點故事閱讀 41,331評論 3 358
  • 正文 我和宋清朗相戀三年,在試婚紗的時候發現自己被綠了。 大學時的朋友給我發了我未婚夫和他白月光在一起吃飯的照片。...
    茶點故事閱讀 43,536評論 1 374
  • 序言:一個原本活蹦亂跳的男人離奇死亡,死狀恐怖,靈堂內的尸體忽然破棺而出,到底是詐尸還是另有隱情,我是刑警寧澤,帶...
    沈念sama閱讀 39,058評論 5 363
  • 正文 年R本政府宣布,位于F島的核電站,受9級特大地震影響,放射性物質發生泄漏。R本人自食惡果不足惜,卻給世界環境...
    茶點故事閱讀 44,754評論 3 349
  • 文/蒙蒙 一、第九天 我趴在偏房一處隱蔽的房頂上張望。 院中可真熱鬧,春花似錦、人聲如沸。這莊子的主人今日做“春日...
    開封第一講書人閱讀 35,154評論 0 28
  • 文/蒼蘭香墨 我抬頭看了看天上的太陽。三九已至,卻和暖如春,著一層夾襖步出監牢的瞬間,已是汗流浹背。 一陣腳步聲響...
    開封第一講書人閱讀 36,469評論 1 295
  • 我被黑心中介騙來泰國打工, 沒想到剛下飛機就差點兒被人妖公主榨干…… 1. 我叫王不留,地道東北人。 一個月前我還...
    沈念sama閱讀 52,273評論 3 399
  • 正文 我出身青樓,卻偏偏與公主長得像,于是被迫代替她去往敵國和親。 傳聞我的和親對象是個殘疾皇子,可洞房花燭夜當晚...
    茶點故事閱讀 48,505評論 2 379