我应该使用 goto 语句吗？ [关闭]

Question

我有一段代码如下：

try
{
Work:

   while(true)
   {
      // Do some work repeatedly...
   }
}
catch(Exception)
{
   // Exception caught and now I can not continue 
   // to do my work properly

   // I have to reset the status before to continue to do my work
   ResetStatus();

   // Now I can return to do my work
   goto Work; 
}

与使用goto 相比，有没有更好的选择？或者这是一个好的解决方案？

Answer 1

听起来你真的想要一个循环。我会写成：

bool successful = false;
while (!successful)
{
    try
    {
        while(true)
        {
            // I hope you have a break in here somewhere...
        }
        successful = true;
    }
    catch (...) 
    {
        ...
    }
}

您可能想改用do/while 循环；我倾向于更喜欢直接的 while 循环，但这是个人喜好，我可以在这里看到它可能更合适。

我不会使用goto。它往往会使代码更难遵循。

当然，如果您真的想要无限循环，只需将try/catch 放入循环中即可：

while (true)
{
    try
    {
        ...
    }
    catch (Exception)
    {
        ...
    }
}

Answer 2

Goto 很少适合使用。使用会使 99% 的查看您的代码的人感到困惑，甚至在技术上正确使用它也会大大减慢对代码的理解。

在大多数情况下，重构代码将消除对goto 的需求（或使用欲望）。 IE。在您的特定情况下，您可以简单地将 try/catch 移动到 while(true) 内。将迭代的内部代码制作成单独的函数可能会更简洁。

while(true)
{
  try
  {
      // Do some work repeatedly...
  }
  catch(Exception)
  {
   // Exception caught and now I can not continue 
   // to do my work properly

   // I have to reset the status before to continue to do my work
   ResetStatus();
  }
}

Answer 3

将 try/catch 移到 while 循环中似乎更有意义。然后您可以处理错误，循环将正常继续，而无需使用标签和 goto 路由控制流。

Answer 4

在每次迭代中捕获并恢复状态，即使在外部捕获也一样，但您仍处于循环中，您可以决定是继续还是中断循环。

ALSO： 从一开始就捕获Exception 是错误的（如果您捕获StackOverflowException 或MemoryLeachException，您会怎么做 - 编辑：这只是举例，检查文档以了解您可以在现实中捕捉到什么）。捕获您希望抛出的具体类型的异常。

while(true)
{
    try
    {
        // Do some work repeatedly...
    }
    catch(FileNotFoundException) //or anything else which you REALLY expect.
    {
        // I have to reset the status before to continue to do my work
        ResetStatus();
        //decide here if this is till OK to continue loop. Otherwise break.
    }
}

对于那些在 cmets 中非常聪明的人：Why don't catch general exception