循环最佳实践

Question

我有一个非常大的循环，循环 1000 行。 如果找到幻值 1，我将退出循环。 如果未找到幻值 1 但找到幻值 2，则循环需要跳到开头。 现在我正在使用 switch、一些 if 和 goto。 我读到 goto 不是最好的方法。 有没有更好的方法来完成这项工作？

Answer 1

要退出循环，您可以使用 break 语句，要转到下一条记录，您可以使用继续声明。

for(int i = 0; i < 1000; i++)
{
    if(magicValue1)
       break;
    if(magicValue2)
       continue;
}

我不赞成使用 GOTO 语句，我只是指出一个可能的用例

您可以使用 goto 跳转语句来启动/退出循环，但我会保留除非您使用嵌套循环，否则请远离此选项。 我认为 goto 语句仍然有其优化、干净退出等用途。但总的来说，最好相当谨慎使用它。

for(int i = 0; i < 100; i++)
{ 
  start:

  for(int i = 0; i < 10; i++)
  {
     if(magicValue1)
       goto end;
    if(magicValue2)
       goto start;
  }
}
end : 

Answer 2

怎么样：

for(int i = 0; i < 1000; i++) {
    if(values[i] == MAGIC_VALUE_1) {
        break;
    } else if(values[i] == MAGIC_VALUE_2) {
        i = 0;
    }
}

如果“跳到开头”的意思是“跳过此记录并处理下一条记录”，请将 i = 0 替换为 继续。

Answer 3

没有 break 的 while 变体：

bool continue = true; int i = 0;
while (i < 1000 && continue){
    if(values[i] == MAGIC_VALUE_1) {
        continue=false;
    } else if(values[i] == MAGIC_VALUE_2) {
        i = 0;
    }
    i++;
}

Answer 4

我还不能发表评论（距离 1 个代表点），

但这不是更好吗：

for (int i = 0; i < 1000; i++)
{
    if (magicValue1)
    {
       break;
    }
    else if (magicValue2)
    {
       dosomething();
       i=0;
    }
}

而且我不确定“重新开始搜索”是什么意思。

Answer 5

我采用#2 情况意味着您不想执行（即跳过）#2 情况中的循环体，而不是您想要将循环重置为 0。（如果我知道，请参阅代码注释向后。）

这个建议可能会引起争议，因为 for 循环中不太传统的条件可以说是自记录规模较低，但如果这不打扰你，一种简洁的方式来写我认为你的想要的是：

        for (int i= 0; i<values.Length && values[i]!= MAGIC_1; i++)
        {
            if (values[i] == MAGIC_2)
            {
                // Don't do the loop body for this case but continue on looping
                continue;
                // If you want to reset the loop to zero instead of skip the 2 case,
                // comment-out the continue; and un-comment the line below:
                // i=0;
            }
            // Do long loop body here
        }

Answer 6

请注意，如果您在 MagicValue 为 2 时将计数器设置回 0，并且您的代码从不更改这些值，则您可能会陷入无限循环。

Answer 7

更复杂的可能是：

我们定义 2 个扩展方法。

public static class Extensions
{
   public static bool isMagic_1(this int i)
   {
         return i == 1;
   }

   public static bool isMagic_2(this int i)
   {
         return i == 2;
   }
}

现在您可以这样做：

  for(int i = 0; i < 1000; i++)
  {
     if(i.isMagic_1())
       break;
     if(i.isMagic_2())
       continue;
  }

希望这会有所帮助！