開發十年，就只剩下這套架構體系了！ >>>   

狗屎一樣的程式碼如何重構？

重構不止是程式碼整理，它提供了一種高效且受控的程式碼整理技術。

（一）重構原則

1、何謂重構

對軟體內部結構的一種調整，目的是在不改變軟體可觀察行為的前提下，提高其可理解性，降低其修改成本。

另一種解釋是：使用一系列重構手法，在不改變軟體可觀察行為的前提下，調整其結構。

2、為何重構

改進軟體設計：如果沒有重構，程式的設計會逐漸變質，重構很像是在整理程式碼，你所做的就是讓所有的東西回到應處的位置上。

幫助找到bug：對程式碼進行重構，可以深入理解程式碼的作為，在搞清楚程式結構的同時，想不把bug揪出來都難。

提高程式設計速度：良好的設計是快速開發的根本，改善設計、提高可讀性，減少錯誤，這些都是提高質量。

3、何時重構

任何情況下我都反對專門撥出時間進行重構。重構本來就不是一件應該特別撥出時間做的事情，重構應該隨時隨地的進行。

三次法則

第一次做某件事情是隻管去做；

第二次做類似的事情會產生反感；

第三次再做類似的事，你就應該重構

大多數重構都為程式引入了更多的間接層，重構往往把大型的物件拆成多個小型的物件，把大型的函式拆成多個小型的函式。但是，間接層是一把雙刃劍。每次把一個東西分成兩份，你就需要多管理一個東西。如果某個物件委託另一個物件，後者又委託另一個物件，程式會愈加難以閱讀。【文末有乾貨分享，滿滿的免費資料！】

重新組織函式

1、提煉函式

動機：看到一個過長的函式或者一段需要註釋才能讓人理解用途的程式碼，將這段程式碼放一個獨立的函式中；

做法：

創造一個新函式，根據這個函式的意圖來命名它；

只要新函式的名稱能夠以更好的方式昭示程式碼意圖，你也應該提煉它。但如果想不到一個更有意義的名稱就別動

將提煉的程式碼從原函式複製到新建的目標函式中；

將被提煉程式碼段中需要讀取的區域性變數，當作引數傳遞給目標函式；

在源函式中，將被提煉程式碼段替換為目標函式呼叫。

2、行內函數

一個函式的本體與名稱同樣清楚易懂。在函式呼叫點插入函式本體，然後移除該函式。

動機：一群組織不甚合理的函式。你可以將它們都內聯到一個大函式中，再從中提煉出組織合理的小型函式。

使用的太多的間接層，使得系統中的所有函式都似乎只是對另一個函式的簡單委託，造成在委託動作之間暈頭轉向。

做法：

1、檢查函式，確定不具備多型；

如果子類繼承了這個函式，就不要將此函式內聯，因為子類無法複寫一個根本不存在的函式。

2、找出這個函式的所有呼叫點；

3、將這個函式的所有呼叫點都替換成函式本體。

3、內聯臨時變數

有一個臨時變數，只被一個簡單的表達是賦值一次，而它妨礙了其他重構手法。將所有對該變數的引用動作，替換為對它賦值的那個表示式自身

double basePrice = anOrder.basePrice();

return (base > 10000 );

替換為：

return (anOrder.basePrice > 1000);

4、以查詢取代臨時變數

你的程式以一個臨時變數儲存某一表達式的運算結果。將這個表示式提煉到一個獨立的函式中。將這個臨時變數的所有引用點替換為對新函式的呼叫。此後，新函式就可被其他函式使用。

double basePrice = quantity * timePrice;

if(basePrice > 1000){

  return basePrice * 09.5;

} else {

  return basePrice * 0.98;

}

替換為：

if(basePrice() > 1000){

  return basePrice * 09.5;

} else {

  return basePrice * 0.98;

}

double basePrice(){

  return quantity * timePrice;

}

臨時變數只在所屬的函式中可見，如果把臨時變數替換為一個查詢，那麼同一個類中的所有函式都將可以獲得這個份資訊，這將帶給你極大的幫助，使你能夠為這個類編寫更清晰的程式碼。

5、引入註釋性變數

你有一個複雜的表示式。將該複雜表示式（或其中一部分）的結果放進一個臨時變數，以此變數名稱來解釋表示式用途。

if ((platform.toUpperCase().indexOf("MAC") > -1) && (browser.toUpperCase().indexOf("IE") > -1) && wasInitialized() && resize >0){

//do smothing

}

替換為：

final boolean isMacOs = platform.toUpperCase().indexOf("MAC") > -1;

final boolean isIEBrowser = browser.toUpperCase().indexOf("IE") > -1;

final boolean wasResized = resize >0;

if(isMacOs && isIEBrowser && wasInitialized() && wasResized){

  //do smothing

}

表示式有可能非常複雜難以理解。這種情況下，臨時變數可以幫助你將表示式分解為比較容易管理的形式。

在條件邏輯中，你可以用這項重構將每個條件子句提煉出來，以一個良好命名的臨時變數來解釋對應條件子句的意義。另一種情況是：在較長的演算法中，可以運用臨時變數來解釋每一步運算的意義。

6、分解臨時變數

你的程式有某個臨時變數被賦值超過一次，它既不是迴圈變數，也不被用於收集計算結果。 針對每次賦值，創造一個獨立、對應的臨時變數。

double temp = 2 * (height + width);

System.out.println(temp);

temp = height * width;

System.out.println(temp);

替換為：

double perimeter = 2 * (height + width);

System.out.println(perimeter);

double area = height * width;

System.out.println(area);

如果臨時變數被賦值超過一次，就意味著它們在函式中承擔了一個以上的責任。如果臨時變數承擔多個責任，它就應該被替換為多個臨時變數。每個變數只承擔一個責任，同一個臨時變數承擔兩件不同的事情會令程式碼閱讀者糊塗

7、移除對引數的賦值

程式碼對一個引數進行復制。以一個臨時變數取代該引數的位置。

int discount (int inputVal, int quantity, int yearToData){

  if(inputVal > 50) inputVal -= 2;

}

替換為：

int discount (int inputVal, int quantity, int yearToData){

  int result = inputVal;

  if(inputVal > 50) result -= 2;

}

如果程式碼的語義是按引用傳遞的，請在呼叫段檢查呼叫後是否還使用了這個引數。

8、替換演算法

想要把某個演算法替換為另一個更清晰的演算法。將函式本體替換成為另一個演算法。

String foundPerson(String[] people){

  for(int i = 0;i < people.length; i++){

      if(people[i].equals("Don")){

          return "Don";

      }

      if(people[i].equals("John")){

          return "John";

      }

      if(people[i].equals("Kent")){

          return "Kent";

      }

  }

  return "";

}

替換為：

String foundPerson(String[] people){

  List candidates = Arrays.asList(new String[]{"Don", "John", "Kent"});

  for(int i = 0;i < people.length; i++){

      if(candidates.contains(people[i])){

          return prople[i];

      }

  }

  return "";

}

（未完）

分享免費學習資料

針對於還會準備免費的Java架構學習資料（裡面有高可用、高併發、高效能及分散式、Jvm效能調優、MyBatis，Netty,Redis,Kafka,Mysql,Zookeeper,Tomcat,Docker,Dubbo,Nginx等多個知識點的架構資料）

為什麼某些人會一直比你優秀，是因為他本身就很優秀還一直在持續努力變得更優秀，而你是不是還在滿足於現狀內心在竊喜！希望讀到這的您能點個小贊和關注下我，以後還會更新技術乾貨，謝謝您的支援！

有需要獲取免費資料可點喜歡關注我，加秋秋群：856443934獲取！

（部分資料如下）​

相關推薦