我有一个非常大的循环，可以循环 1000 行。如果找到魔法值 1，我将退出循环。如果没有找到魔法值 1 但找到了魔法值 2，则循环需要跳到开头。现在我正在使用一个开关，一些ifs和一个goto。我读过 goto 不是最好的方法。有没有更好的方法来完成这项工作？
【问题讨论】：
要退出循环，您可以使用break 语句，要进入下一条记录，您可以使用continue 语句。
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 :
【讨论】：
goto 并没有什么坏处——它与break 和continue 没有根本的不同（后者只是碰巧使用了隐式标签）。如果你反对在这种情况下使用goto，你也应该反对break和continue。
这个怎么样：
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 替换为continue。
【讨论】：
没有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++;
}
【讨论】：
我还不能评论（1个代表点）
但这不是更好吗：
for (int i = 0; i < 1000; i++)
{
if (magicValue1)
{
break;
}
else if (magicValue2)
{
dosomething();
i=0;
}
}
我不确定“重新开始搜索”是什么意思。
【讨论】：
我将 #2 案例表示您不想在 #2 案例中执行（即跳过）循环体，而不是将循环重置为 0。（请参阅代码 cmets if I'倒过来了。）
这个建议可能会引起争议，因为 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
}
【讨论】：
请注意，如果您在 MagicValue 为 2 时将计数器设置回 0，并且您的代码永远不会更改值，那么您可能会陷入无限循环。
【讨论】：
更复杂的可能是：
我们定义了 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;
}
希望这会有所帮助！
【讨论】：