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对象或类似的东西。