简化嵌套循环逻辑

Question

我在尝试简化嵌套循环中包含的一些逻辑时遇到了一些麻烦

，我在下面提供了一些伪代码，这是我正在处理的基础知识。实际代码比较复杂，包含多个嵌套循环结构。

我负责维护这段代码，我不是它的原始作者

这是一个 goto 有效的地方吗？

while(CONDITION)
{
  while(CONDITITION2)
  {
    if(CONDITION3)
      break CONDITION2

    if(CONDITION4)
      break CONDITION
  }
}

执行这些代码中断有很多复杂性。

我不允许使用 goto，但我认为这将是我目前正在尝试做的更简单的解决方案。我应该尝试证明在这种情况下使用 goto 的合理性吗？ 谢谢普拉

尚特:)

Answer 1

引入一个变量

bool breakOuter = false;
while(CONDITION)
{
  while(CONDITITION2)
  {
    if(CONDITION3)
      break;

    if(CONDITION4)
    {
      breakOuter = true;
      break;
    }
  }
  if(breakOuter)
    break;
}

Answer 2

您正在为自己设置混乱且复杂的代码。如果可能的话，请考虑 LINQ（循环本身实际上是一种代码味道，尤其是复杂的循环），否则请考虑将内部循环重构为单独的方法并使用 return;。

编辑：Eric Lippert 的精彩帖子批评了此问题的各种替代方案：http://blogs.msdn.com/b/ericlippert/archive/2010/01/11/continuing-to-an-outer-loop.aspx

Answer 3

使用变量作为条件 1：

bool condition1 = true;
while (CONDITION && condition1)
{
    while (CONDITION2)
    {
        if (CONDITION3)
            break;
        if (CONDITION4)
        {
            condition1 = false;
            break;
        }
    }
}

Answer 4

当 Java（不是 C#，但相关）决定禁止 goto 时，他们通过代码分析发现 90% 的 goto 用于循环，其余的则用于非常糟糕的事情。

Break 和 continue 仍然存在，因为它们是合法的、有效的本地化 goto。确保记录他们的去向和原因，但它们仍然非常有用。

您似乎正在以最有效的方式使用它们。

Answer 5

即使没有完全了解上下文，我也可以说我不想成为该代码的下一个程序员。

LINQ 或许可以简化一些。如果您使用 ReSharper，它会在可以的情况下为您推荐。

除此之外，我会更多地思考如何首先处理这些嵌套循环，并尝试摆脱尽可能多的循环。更好的对象模型或巧妙地使用模式可以为您带来更好且更易于维护的解决方案。

Answer 6

怎么样：

while(CONDITION)
{
  while(CONDITITION2)
  {
    if(CONDITION3)
      break CONDITION2

    if(CONDITION4)
      break CONDITION2
  }
  if(CONDITION4)
    break CONDITION
}

您还可以尝试将中断条件放入 while 条件中。