开关内部中断无法终止 FOR 循环

Question

我有一个代码片段：

int n = 0;
for (int i = 0; i < 50;i++)
{
    n = checkStatus();
    switch (n)
    {
       case 1:
          break;
          break;//This is unreachable and so i cannot Terminate the For Loop within the SWITCH
    }
}

如评论中所述，只有当我声明一个布尔值并在 Switch 测试结束时，我才能直接从 Switch 终止 For 循环

if(LoopShouldTerminate)
    break;

PS：或者也许我很困惑！

[发布] 我收到消息，问题已解决，但我想假设在 for 循环中使用 Switch 不是一个好主意，因为我从很多开发人员那里听说，当我得到想要的结果，所以使用 switch 需要额外的布尔值或将 Int i 值直接推到 50，但是如果我们使用 while 循环会发生什么？

Answer 1

解决方案 1：将循环和 switch 移至不同的方法：

for(int i = 0; i < 50; ++i)
{
    if (DoCheckStatus(i)) break;
}

...

bool DoCheckStatus(int i)
{
    switch(CheckStatus(i))
    {
        case 1 : return true;
        default: return false;
    }
}

解决方案 2：采用上述方法来消除使用急切扩展方法的循环：

static void DoWhile<T>(this IEnumerable<T> sequence, Func<T, bool> predicate)
{
    foreach(T item in sequence) 
        if (!predicate(item)) return;
}

...

Enumerable.Range(0, 50).DoWhile(DoCheckStatus)

解决方案 3：采用上述方法来消除循环和辅助方法：

Enumerable.Range(0, 50).DoWhile(i=> 
{ 
    switch(CheckStatus(i))
    {
        case 1 : return true;
        default: return false;
    }
});

Answer 2

您可以使用 goto 来中断跳出 switch 内的循环。

        int n = 0;
        for (int i = 0; i < 50;i++)
        {
            n = checkStatus();
            switch (n)
            {
                case 1:
                    goto outofloop;

            }
        }

    :outofloop
        // more code

goto 为数不多的好用处之一...

Answer 3

只需更改 i 的值即可：

int n = 0;
        for (int i = 0; i < 50;i++)
        {
            n = checkStatus();
            switch (n)
            {
                case 1:
                    i += 50;
                    break;

            }
        }

Answer 4

如果 switch 之后没有其他代码，您可以在 for 循环本身中检查是否继续循环：

bool doContinue = true;
for (int i = 0; i < 50 && doContinue; i++)
{
    n = checkStatus();
    switch (n)
    {
        case 1:
            doContinue = false;

    }
}

Answer 5

您可以将循环放在方法内部并只使用 return 吗？

示例：

myLoopingMethod()
{
    int n = 0;
    for (int i = 0; i < 50;i++)
    {
        n = checkStatus();
        switch (n)
        {
            case 1:
                return;
        }
    }
}

另一种选择是使用传统的 if/else 而不是 switch/case。然后你可以使用break，它会跳出你的for循环

Answer 6

您可以使用布尔值。

                int n = 0;
            for (int i = 0; i < 50; i++)
            {
                bool shouldBreak = false;
                n = checkStatus();
                switch (n)
                {
                    case 1:
                        shouldBreak = true;
                        break;
                }

                if (shouldBreak)
                    break;
            }

Answer 7

我不知道循环完成后是否需要执行其他逻辑，但您可以尝试使用 return 代替。另一个选项是设置一个布尔标志，并在切换后退出。

switch (n)
{
    case 1:
        return;
}

Answer 8

这听起来有点糟糕的设计；有几种方法可以解决这个问题：

如果只有一个 switch 条件会破坏外部循环，那么只需在输入 switch 之前检查一下：

 if(n == 1)
    break;

 switch(n) { }

如果多个条件可以破坏循环，请重构为更 linq 风格的查询：

Enumerable.Range(0, 50).FirstOrDefault(x => listOfBreakCodes.Contains(checkStatus());

这应该调用 checkStatus 最多 50 次，直到遇到 1（或其他中断代码），然后不再继续评估元素。