return語句做提取方法

Question

時

比方說，你有一個很長的方法，像這樣：

int monster() 
{ 
    int rc = 0; 

    // some statements ... 

    if (dragonSlayer.on_vacation()) { 
     cout << "We are screwed!\n"; 
     if (callTheKing() == true) 
      return 1; 
     else 
      return 2; 
    } else { 
     cout << "We are saved!\n"; 
     slayTheDragon(); 
    } 

    // rest of long method... 

    return rc; 
} 

和我工作skeletonizing的代碼。我想提取龍殺死部分到

int handleDragon() { 
    if (dragonSlayer.on_vacation()) { 
     cout << "We are screwed!\n"; 
     if (callTheKing() == true) 
      return 1; 
     else 
      return 2; 
    } else { 
     cout << "We are saved!\n"; 
     slayTheDragon(); 
    } 

    return 0; // ? 
} 

並用handleDragon（）調用替換怪物（）中的代碼。

但有一個問題。該部分中間有一個返回聲明。如果我保留處理handleDragon（）的返回代碼的部分，它將保持垃圾大方法。

除了使用異常，有沒有一種優雅和安全的方式來重構這段代碼從怪物方法？應如何處理這些類型的情況？

Answer 1

返回0從handleDragon方法，如果龍者可用：

int handleDragon() { 
    if (dragonSlayer.on_vacation()) { 
     cout << "We are screwed!\n"; 
     if (callTheKing() == true) 
      return 1; 
     else 
      return 2; 
    } else { 
     cout << "We are saved!\n"; 
     slayTheDragon(); 
     return 0; 
    } 
} 

然後回到monster方法，如果返回值是大於零，返回該值，否則進行：

// some statements ... 

int handleDragonResult = handleDragon(); 
if (handleDragonResult > 0) { 
    return handleDragonResult; 
} 

// rest of long method... 

您還應該記錄handleDragon方法，以解釋返回的值。

Answer 2

你不能做到這一點：

int handleDragon() { 
    int rc = 0; 

    if (dragonSlayer.on_vacation()) { 
     cout << "We are screwed!\n"; 
     if (callTheKing() == true) 
      rc = 1; 
     else 
      rc = 2; 
    } else { 
     cout << "We are saved!\n"; 
     slayTheDragon(); 
    } 

    return rc; 
} 

然後：

int monster() 
{ 
    int rc = 0; 

    // some statements ... 

    rc = handleDragon(); 

    // rest of long method... 

    return rc; 
} 

，或者如果你想要做的事與返回代碼：

int monster() 
{ 
    int rc = 0; 

    // some statements ... 

    int handleDragonReturnCode = handleDragon(); 

    if(handleDragonReturnCode == 0) { 
     // do something 
    } 

    else { 
     // do something else 
    } 

    // rest of long method... 

    return rc; 
} 

這是什麼你要？在一般說明中，請避免使用幻數，如1和2作爲您的返回碼。使用常量，#define或enum。

關於return，請嘗試讓一個退出您的功能。正如你發現的那樣，有多個return語句可以使重構變得困難（除非它真的很簡單，否則就理解邏輯）。

Answer 3

enum DragonHandled { DHSuccess, DHKing, DHNoKing }; 

inline DragonHandled askForKing() 
{ 
    if (callTheKing()) 
     return DHKing; 
    else 
     return DHNoKing; 
} 

DragonHandled handleDragon() 
{ 
    if (dragonSlayer.on_vacation()) { 
     cout << "We are screwed!\n"; 
     return askForKing(); 
    } 
    cout << "We are saved!\n"; 
    slayTheDragon(); 
    return DHSuccess; 
} 

int monster() 
{ 
    some_statements(...); 

    DragonHandled handled = handleDragon(); 
    if(handled != DHSuccess) 
     return handled; // enum to int is an implicit cast 

    return more_statements(...); 
} 

• 除了返回實際簽署數一個功能，我不會再回到int。如果結果有意義，請正確定義該含義（即：enum）。
• 函數確實是東西，無論它做什麼，都應該在它的名字中可見。所以應該有一個動詞在一個函數的名字（handledragon()，callTheKing()）。 monsters不是動詞，它不是你可以做的事情。如果我看到一個標識符monsters，我認爲它是一個怪物容器。
• 檢查if(x == true)只是無用的噪音，因爲if(x)更加簡潔，也是如此。

Answer 4

問題是關於戰略，所以我認爲理查德費恩的答案是一個很好的答案。

，使之成爲一個重構模式它看起來是這樣的：

語境：在一個更大的方法的中間段是被提取。

問題：該部分包含返回語句。

解：

1. 代碼提取到一個新的方法將在返回類型相同更大方法。
2. 找到一個不代表任何意義的值。調用該值繼續。
3. 在返回CONTINUE的新方法末尾添加一條語句。
4. 在較大的方法中，測試CONTINUE的新方法的返回值。如果不是，則返回該值。

這將是主要方法。作爲下一步，您可以將新方法的返回值重構爲更有意義的內容（例如在sbi的答案中）。你必須找到一種方法來處理返回類型不是標量類型或簡單類型的情況，返回一個NULL對象或類似的東西。