在上一篇文章中介紹了“ 內聯臨時變數“。本文將介紹“以查詢取代臨時變數”這種重構手法。

  下面讓我們來學習這種重構手法吧。

開門見山

發現：你的程式以一個臨時變數儲存某一表達式的運算結果。

        解決：將這個表示式提煉到一個獨立函式中。將這個臨時變數的所有引用點替換為對新函式的呼叫。

	//重構前
	double basePrice = _quantity * _itemPrice;
	if(basePrice > 5000)
		return basePrice * 0.95;
	else
		return basePrice * 0.98;

	//重構後
	if(basePrice() > 5000)
		return basePrice() * 0.95;
	else
		return basePrice() * 0.98;
	...
	
	double basePrice(){
		 return  _quantity * _itemPrice;
	}
 

動機

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

        該重構方法往往是你運用提煉函式之前必不可少的一個步驟。區域性變數會使得程式碼難以提煉，所以應該儘可能把它們替換為查詢式。比較簡單的情況是：臨時變數只被賦值一次，或者賦值給臨時變數的表示式不受其它條件影響。

做法

        我們常常使用臨時變數儲存迴圈中的累加資訊。在這種情況下，整個迴圈都可以被提煉為一個獨立的函式，這可以減少原函式中幾行迴圈邏輯程式碼。該手法的使用可能會讓你擔心效能上的問題。就像和其它效能問題一樣，我們現在不用管它，因為十有八九根本沒有造成任何影響。就算真的出問題了，你也可以在優化時期解決它。程式碼組織良好，你也會發現更有效的優化方案；如果沒有進行重構，好的優化方案就可能與你失之交臂。

示例

//重構前
double getPrice() {
	int basePrice = _quantity * _itemPrice;
	double discountFactor;
	if (basePrice > 5000)
		discountFactor = 0.95;
	else
		discountFactor = 0.98;
	return basePrice * discountFactor;
}

double getPrice() {
	final int basePrice = _quantity * _itemPrice;
	final double discountFactor;
	if (basePrice > 5000)
		discountFactor = 0.95;
	else
		discountFactor = 0.98;
	return basePrice * discountFactor;
}

double getPrice() {
	final int basePrice = basePrice();
	final double discountFactor;
	if (basePrice > 5000)
		discountFactor = 0.95;
	else
		discountFactor = 0.98;
	return basePrice * discountFactor;
}

private double basePrice() {
	return _quantity * _itemPrice;
}

double getPrice() {
	final int basePrice = basePrice();
	final double discountFactor;
	if (basePrice() > 5000)
		discountFactor = 0.95;
	else
		discountFactor = 0.98;
	return basePrice * discountFactor;
}

double getPrice() {
	final double discountFactor;
	if (basePrice() > 5000)
		discountFactor = 0.95;
	else
		discountFactor = 0.98;
	return basePrice()  * discountFactor;
}

double getPrice() {
	final double discountFactor = discountFactor() ;
	return basePrice()  * discountFactor;
}

private double discountFactor(){
	if (basePrice() > 5000)
		return 0.95;
	else
		return 0.98;
}

//重構後
double getPrice() {
	return basePrice()  * discountFactor();
}

重構筆記文章如下