重構手法04：Replace Temp with Query （以查詢取代臨時變數）

 

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

double basePrice = _quantity*_itemPrice;             if (basePrice > 1000)             {                 return basePrice * 0.95;             }             else             {                 return basePrice * 0.98;             }

 

if (BasePrice() > 1000)             {                 return BasePrice() * 0.95;             }             else             {                 return BasePrice() * 0.98;             } ……… private int BasePrice()         {             return _quantity* _itemPrice;         }

動機：臨時變數的問題在於：它們是暫時的，而且只能在所屬函數內使用。由於臨時變數只是在所屬函數內可見，所以它們會驅使你寫出更長的函數，因為只有這樣你才能訪問到需要的臨時變數。如果把臨時變數替換為一個查詢，那麼同一個類中的所有函數都可以獲得這份資訊。這將帶給你極大協助，使你能夠為這個類編寫更清晰地代碼。

         Replace Temp with Query （以查詢取代臨時變數）往往是你運用Extract Method （提煉函數）之前必不可少的一個步驟。局部變數會使代碼難以被提煉，所以你應該儘可能把它們替換為查詢式。

         這個重構手法較為簡單的情況是：臨時變數只被賦值一次，或者賦值給臨時變數的運算式不受其他條件影響。其他情況比較棘手，但也可能發生。你可能需要先運用Split Temporary Variable （分解臨時變數）或Separate Query form Modifier （將查詢函數和修改函數分離）使情況變得簡單一些，然後再替換臨時變數。如果你想替換的臨時變數是用來收集結果的）例如迴圈中的累加值），就需要將某些程式邏輯（例如迴圈）複製到查詢函數去。

做法：1、找出只被賦值一次的臨時變數。如果某個臨時變數被賦值超過一次，考慮使用Split Temporary Variable （分解臨時變數）將它們分解成多個變數。

         2、將該變數聲明為const。

         3、編譯。這可確保臨時變數的確只被賦值一次。

         4、將“對該臨時變數賦值”之語句的等號右側部分提煉到一個獨立函數中。首先將函式宣告為private。日後你可能會發現有更多的類需要使用它。那是放鬆對它的保護也很容易。確保提煉出來的函數無任何副作用，也就是說該函數並不修改任何對象內容。如果它有副作用，就對它進行Separate Query form Modifier （將查詢函數和修改函數分離）.

         5、編譯，測試。

         6、在該變數身上實施 Inline Temp （內聯臨時變數）。

         我們常常使用臨時變數儲存迴圈中的累加資訊。在這種情況下，這個迴圈都可以被提煉為一個獨立函數，這也使原本的函數可以少掉幾行擾人的迴圈邏輯。有時候，你可能會在一個迴圈中累加好幾個值。這種情況下你應該針對每個累加值重複一遍迴圈，這樣就可以將所有臨時變數都替換為查詢。當然，迴圈應該很簡單，複製這些代碼才不會帶來危險。

         運用此手法，你可能會擔心效能問題。和其他問題一樣，我們現在不管它，因為它十有八九根本不會造成任何影響。若是效能真的出了問題，你也可以在最佳化時期解決它。程式碼群組織良好，你往往能發現更有效最佳化方案。如果沒用進行重構，好的最佳化方案就可能與你失之交臂。如果效能實在太糟糕，要把臨時變數放回去也很容易。