Java開發中存在這樣的代碼,反而影響整體整潔和可讀性

不完美的庫類

不完美的庫類(Incomplete Library Class)

當一個類庫已經不能滿足實際需要時,你就不得不改變這個庫(如果這個庫是隻讀的,那就沒轍了)。

問題原因

許多編程技術都建立在庫類的基礎上。庫類的作者沒用未卜先知的能力,不能因此責怪他們。麻煩的是庫往往構造的不夠好,而且往往不可能讓我們修改其中的類以滿足我們的需要。

解決方法

如果你只想修改類庫的一兩個函數,可以運用引入外加函數(Introduce Foreign Method);

如果想要添加一大堆額外行爲,就得運用引入本地擴展(Introduce Local Extension)。

收益

減少代碼重複(你不用一言不合就自己動手實現一個庫的全部功能,代價太高)

何時忽略

如果擴展庫會帶來額外的工作量。

重構方法說明

引入外加函數(Introduce Foreign Method)

問題

你需要爲提供服務的類增加一個函數,但你無法修改這個類。

class Report {

  //...

  void sendReport() {

    Date nextDay = new Date(previousEnd.getYear(),

      previousEnd.getMonth(), previousEnd.getDate() + 1);

    //...

  }

}

解決

在客戶類中建立一個函數,並一個第一個參數形式傳入一個服務類實例。

class Report {

  //...

  void sendReport() {

    Date newStart = nextDay(previousEnd);

    //...

  }

  private static Date nextDay(Date arg) {

    return new Date(arg.getYear(), arg.getMonth(), arg.getDate() + 1);

  }

}

引入本地擴展(Introduce Local Extension)

問題

你需要爲服務類提供一些額外函數,但你無法修改這個類。

解決

建立一個新類,使它包含這些額外函數,讓這個擴展品成爲源類的子類或包裝類。

中間人

中間人(Middle Man)

如果一個類的作用僅僅是指向另一個類的委託,爲什麼要存在呢?

問題原因

對象的基本特徵之一就是封裝:對外部世界隱藏其內部細節。封裝往往伴隨委託。但是人們可能過度運用委託。比如,你也許會看到一個類的大部分有用工作都委託給了其他類,類本身成了一個空殼,除了委託之外不做任何事情。

解決方法

應該運用移除中間人(Remove Middle Man),直接和真正負責的對象打交道。

收益

減少笨重的代碼。

何時忽略

如果是以下情況,不要刪除已創建的中間人:

添加中間人是爲了避免類之間依賴關係。

一些設計模式有目的地創建中間人(例如代理模式和裝飾器模式)。

重構方法說明

移除中間人(Remove Middle Man)

問題

某個類做了過多的簡單委託動作。

解決

讓客戶直接調用委託類。

依戀情結

依戀情結(Feature Envy)

一個函數訪問其它對象的數據比訪問自己的數據更多。

問題原因

這種氣味可能發生在字段移動到數據類之後。如果是這種情況,你可能想將數據類的操作移動到這個類中。

解決方法

As a basic rule, if things change at the same time, you should keep them in the same place. Usually data and functions that use this data are changed together (although exceptions are possible).

有一個基本原則:同時會發生改變的事情應該被放在同一個地方。通常,數據和使用這些數據的函數是一起改變的。

如果一個函數明顯應該被移到另一個地方,可運用搬移函數(Move Method)。

如果僅僅是函數的部分代碼訪問另一個對象的數據,運用提煉函數(Extract Method)將這部分代碼移到獨立的函數中。

如果一個方法使用來自其他幾個類的函數,首先確定哪個類包含大多數使用的數據。然後,將該方法與其他數據一起放在此類中。或者,使用提煉函數(Extract Method)將方法拆分爲幾個部分,可以放置在不同類中的不同位置。

收益

減少重複代碼(如果數據處理的代碼放在中心位置)。

更好的代碼組織性(處理數據的函數靠近實際數據)。

何時忽略

有時,行爲被有意地與保存數據的類分開。這通常的優點是能夠動態地改變行爲(見策略設計模式,訪問者設計模式和其他模式)。

重構方法說明

搬移函數(Move Method)

問題

你的程序中,有個函數與其所駐類之外的另一個類進行更多交流:調用後者,或被後者調用。

解決

在該函數最常引用的類中建立一個有着類似行爲的新函數。將舊函數變成一個單純的委託函數,或是舊函數完全移除。

提煉函數(Extract Method)

問題

你有一段代碼可以組織在一起。

void printOwing() {

  printBanner();

  //print details

  System.out.println("name: " + name);

  System.out.println("amount: " + getOutstanding());

}

解決

移動這段代碼到一個新的函數中,使用函數的調用來替代老代碼。

void printOwing() {

  printBanner();

  printDetails(getOutstanding());

}

void printDetails(double outstanding) {

  System.out.println("name: " + name);

  System.out.println("amount: " + outstanding);

}

狎暱關係

狎暱關係(Inappropriate Intimacy)

一個類大量使用另一個類的內部字段和方法。

問題原因

類和類之間應該儘量少的感知彼此(減少耦合)。這樣的類更容易維護和複用。

解決方法

最簡單的解決方法是運用搬移函數(Move Method)和搬移字段(Move Field)來讓類之間斬斷羈絆。


你也可以看看是否能運用將雙向關聯改爲單向關聯(Change Bidirectional Association to Unidirectional)讓其中一個類對另一個說分手。

如果這兩個類實在是情比金堅,難分難捨,可以運用提煉類(Extract Class)把二者共同點提煉到一個新類中,讓它們產生愛的結晶。或者,可以嘗試運用隱藏委託關係(Hide Delegate)讓另一個類來爲它們牽線搭橋。

繼承往往造成類之間過分緊密,因爲子類對超類的瞭解總是超過後者的主觀願望,如果你覺得該讓這個子類自己闖蕩,請運用以委託取代繼承(Replace Inheritance with Delegation)來讓超類和子類分家。

收益

提高代碼組織性。

提高代碼複用性。

重構方法說明

搬移函數(Move Method)

問題

你的程序中,有個函數與其所駐類之外的另一個類進行更多交流:調用後者,或被後者調用。

解決

在該函數最常引用的類中建立一個有着類似行爲的新函數。將舊函數變成一個單純的委託函數,或是舊函數完全移除。

搬移字段(Move Field)

問題

在你的程序中,某個字段被其所駐類之外的另一個類更多地用到。

解決

在目標類新建一個字段,修改源字段的所有用戶,令他們改用新字段。

將雙向關聯改爲單向關聯(Change Bidirectional Association to Unidirectional)

問題

兩個類之間有雙向關聯,但其中一個類如今不再需要另一個類的特性。

解決

去除不必要的關聯。

提煉類(Extract Class)

問題

某個類做了不止一件事。

解決

建立一個新類,將相關的字段和函數從舊類搬移到新類。

隱藏委託關係(Hide Delegate)

問題

客戶通過一個委託類來調用另一個對象。

解決

在服務類上建立客戶所需的所有函數,用以隱藏委託關係。

以委託取代繼承(Replace Inheritance with Delegation)

問題

某個子類只使用超類接口中的一部分,或是根本不需要繼承而來的數據。

解決

在子類中新建一個字段用以保存超類;調整子類函數,令它改而委託超類;然後去掉兩者之間的繼承關係。

過度耦合的消息鏈

過度耦合的消息鏈(Message Chains)

消息鏈的形式類似於:obj.getA().getB().getC()。

問題原因

如果你看到用戶向一個對象請求另一個對象,然後再向後者請求另一個對象,然後再請求另一個對象……這就是消息鏈。實際代碼中你看到的可能是一長串 getThis()或一長串臨時變量。採取這種方式,意味客戶代碼將與查找過程中的導航緊密耦合。一旦對象間關係發生任何變化,客戶端就不得不做出相應的修改。

解決方法

可以運用隱藏委託關係(Hide Delegate)刪除一個消息鏈。

有時更好的選擇是:先觀察消息鏈最終得到的對象是用來幹什麼的。看看能否以提煉函數(Extract Method)把使用該對象的代碼提煉到一個獨立函數中,再運用搬移函數(Move Method)把這個函數推入消息鏈。

收益

能減少鏈中類之間的依賴。

能減少代碼量。

何時忽略

過於侵略性的委託可能會使程序員難以理解功能是如何觸發的。

重構方法說明

隱藏委託關係(Hide Delegate)

問題

客戶通過一個委託類來調用另一個對象。


解決

在服務類上建立客戶所需的所有函數,用以隱藏委託關係。

提煉函數(Extract Method)

問題

你有一段代碼可以組織在一起。

void printOwing() {

  printBanner();

  //print details

  System.out.println("name: " + name);

  System.out.println("amount: " + getOutstanding());

}

解決

移動這段代碼到一個新的函數中,使用函數的調用來替代老代碼。

void printOwing() {

  printBanner();

  printDetails(getOutstanding());

}

void printDetails(double outstanding) {

  System.out.println("name: " + name);

  System.out.println("amount: " + outstanding);

}

搬移函數(Move Method)

問題

你的程序中,有個函數與其所駐類之外的另一個類進行更多交流:調用後者,或被後者調用。

解決

在該函數最常引用的類中建立一個有着類似行爲的新函數。將舊函數變成一個單純的委託函數,或是舊函數完全移除。

覺得不錯請點贊支持,歡迎留言或進我的個人羣855801563領取【架構資料專題目合集90期】、【BATJTMD大廠JAVA面試真題1000+】,本羣專用於學習交流技術、分享面試機會,拒絕廣告,我也會在羣內不定期答題、探討

發表評論
所有評論
還沒有人評論,想成為第一個評論的人麼? 請在上方評論欄輸入並且點擊發布.
相關文章