本文轉載，原作者：皮斯特勞沃

開門見山

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

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

  //重構前
    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();
}

本文轉載，原作者：皮斯特勞沃