多个返回语句或“goto end”；

Question

什么是更好的样式/可读性？

我有一个循环读取输入并根据输入做不同的事情，当发生错误时，我需要一个简单的return;。示例：

while( get_input() )
{
    if( input == "somethingcool" )
    {
        if( !process_somethingcool() )
            return; // <-- a couple of these
    }
    //...
    else // bad input, error handling is fancier than this, but irrelevant to the question
        return;
}
return;

那么我是否应该用goto end; 替换单个return;s 并在上面示例中的最后一个返回正上方放置一个标签end:？我不需要“使用 RAII”，因为在 if 块中没有分配任何内容。除了样式/可读性/性能之外，这两种方式在所有意义上都是相同的？

我认为性能是相同的，但只是为了确定：是吗？

Answer 1

对于 C，goto 是合理的（它在 Linux 内核中广泛使用），因为您可以通过单点返回来增强可读性。

对于 C++，因为您有可能引发异常，所以您隐含了多个返回点，因此您应该始终使用具有多个返回的 RAII 模型。

Answer 2

你想发动一场宗教战争吗？

说真的，有时候 goto 是最好的选择。二十年来，我见过大约 3 或 4 次。多次返回不一定是邪恶的，但如果你必须重复大量的清理代码，那么它们就会变得非常混乱。

很多时候，您可以重构代码以使这样的选择变得不必要。没有看到你的代码很难提出具体的建议，但可能是这样的：

void f()
{
   bool bDone=false;
   while (!bDone && get_input())
   {
      if (input == "cool")
      {
         process_cool();
         bDone = true;
      }
      else if (input == "unfinished")
      {
         process_something();
      }
      else
      {
          // error
          bDone = true;
      }
   }
}

重构的一大帮助是确保循环内没有几十行。如果您有大量工作要做，请将其分解为函数并从 while 循环中调用少量函数。

请记住，单个函数应该只做一件事。

另一种强烈推荐的方法是使用异常来处理错误情况，但如果您刚刚完成处理，使用异常是一种不好的方式，因此这可能无法完全解决您的问题。

如果您仍然感到困惑，请考虑发布一段更真实的代码，我们或许可以建议如何最好地处理这些问题。

祝你好运！

Answer 3

就我个人而言，我喜欢让任何块的内容尽可能小。一行，调用另一个函数是理想的（在 Bob Martin 的 Clean Code 之后）。

我会选择你没有提出的选项：

while(get_input()) {
    if(!process_input(input)) {
         break;
    }
}

其中 process_input 将选择适当的 process_... 函数，返回返回的任何内容，如果输入错误，则返回 false。

Answer 4

当你想回来的时候就回来。 C++ 语言可以处理这个问题，它是编写代码最直观的方式。

在某些语言中，清理工作必须在使用站点完成，因此将“清理”阶段集中在您“转到”的单个块中可能是个好主意。在 C 语言中，这种习语很常见。

在 C++ 中，当资源超出范围时，资源会在其析构函数中自行清理。所以在使用现场，什么都不需要，最简单、最干净、最不容易出错的解决方案就是多条return语句。

Answer 5

多个返回语句实际上是一种很好的风格，并且几乎总是比尝试使用单个返回点生成更简洁的代码。 gotos 在 C++ 中非常没用，因为（除了它们的其他问题）它们不能跳过初始化，这可能会迫使你在远离它们的使用点的地方进行初始化，这也是一种不好的风格。

Answer 6

样式和可读性将引导我使用异常来处理错误情况。您的调用者如何知道您的方法是否被正确调用？

也就是说：goto 不应该用于此； return 是更好的选择。如果您需要在例程结束时执行任何操作，无论它为何退出 - 那么您应该抛出异常并使用 catch 块。

Answer 7

return 一般优先于goto；通常，诋毁标签的人无法提出任何实际论据来支持他们的教条。¹

但是，您的情况更为明确：您为什么选择goto？两种解决方案是等效的，只是goto 要求您在函数结束之前写end:，占用空间并且无缘无故地看起来很丑。

因此，在提供的两个选项中，我推荐returns。也就是说，我还建议认真考虑在回答这个问题时给出的其他建议（例外情况，并限制条件句的“嵌套性”）。

---

¹ 或者，如果他们这样做，它通常类似于“goto 让你的对象挂起”，这是不正确的（根据 the C++0x FDIS 中的 6.6/2）。 goto 不会破坏 RAII 模型。

Answer 8

goto End; 很好，部分原因在于可读性，但也因为如果您稍后意识到在关闭函数之前需要发生一些事情，例如释放内存或释放资源，您可以集中该过程而不是复制粘贴它。

Answer 9

其实不建议使用goto。它会在代码的可读性方面产生问题。最好使用break; 语句，在您想离开循环并在代码末尾返回的地方。这是标准程序。

while( get_input() )
{
    if( input == "somethingcool" )
    {
        if( !process_somethingcool() )
            break; // <-- a couple of these
    }
    //...
    else // bad input, error handling is fancier than this, but irrelevant to the question
        break; //although its better if you handle it somehow here.
}
return;

Answer 10

为什么不使用“保护子句”（通过反转逻辑）而不是嵌套的 if 来构建您的代码。那么使用 goto 的整个问题就变得没有意义了。

while( get_input() )
{
    if( input != "somethingcool" )
    {
        return; //handle error
    }
    if( !process_somethingcool() )
        return; //handle error
    }
}
return; //success

Answer 11

两者都没有。您应该将代码重构为可读的 并且可维护。我见过的每个案例（而且我见过很多） 程序员需要goto，大多数情况下 在他需要多次退货的情况下（以及在所有情况下 return 已经嵌套在一个循环中）会更好地解决 将代码重构为单独的、更简单的函数，每个函数 它在函数末尾有一个返回。 （多 对于非常简单的函数，返回有时是合理的；例如 如果函数中唯一的语句是 switch，每个 case 都以 return 结尾。）

Answer 12

恕我直言，使用多个返回，不要使用 goto。使用 goto 的人，也会放火烧狗。