使用“goto”在开关中？

Question

我看到了一个建议的编码标准，内容为除非在 switch 语句中失败，否则永远不要使用 goto。

我不跟。这个“异常”情况到底是什么样子，才能证明 goto 是合理的？

Answer 1

此构造在 C# 中是非法的：

switch (variable) {
   case 2: 
       Console.WriteLine("variable is >= 2");
   case 1:
       Console.WriteLine("variable is >= 1");
}

在 C++ 中，如果 variable = 2，它将运行两行。这可能是故意的，但很容易忘记第一个 case 标签末尾的 break; 。因此，他们在 C# 中将其定为非法。要模仿跌倒行为，您必须明确使用 goto 来表达您的意图：

switch (variable) {
   case 2: 
       Console.WriteLine("variable is >= 2");
       goto case 1;
   case 1:
       Console.WriteLine("variable is >= 1");
       break;
}

_{也就是说，有 一些情况，其中 goto 实际上是解决该问题的一个很好的解决方案。 永远不要用“永远不要使用某些东西”的规则来关闭你的大脑 。如果它 100% 无用，那么它从一开始就不会存在于语言中。不要使用goto是指南；这不是法律。}

Answer 2

C# 拒绝让 case 隐式失败（除非 case 中没有代码），就像 C++ 中一样：您需要包含 break。要明确失败（或跳转到任何其他情况），您可以使用goto case。由于没有其他方法可以实现这种行为，因此大多数（合理的）编码标准都允许这样做。

switch(variable)
{
case 1:
case 2:
    // do something for 1 and 2
    goto case 3;
case 3:
case 4:
    // do something for 1, 2, 3 and 4
    break;
}

---

一个现实的例子（根据要求）：

switch(typeOfPathName)
{
case "relative":
    pathName = Path.Combine(currentPath, pathName);
    goto case "absolute";

case "expand":
    pathName = Environment.ExpandEnvironmentVariables(pathName);
    goto case "absolute";

case "absolute":
    using (var file = new FileStream(pathName))
    { ... }
    break;

case "registry":
    ...
    break;
}

Answer 3

   public enum ExitAction {
        Cancel,
        LogAndExit,
        Exit
    }

这

ExitAction action = ExitAction.LogAndExit;
switch (action) {
    case ExitAction.Cancel:
        break;
    case ExitAction.LogAndExit:
        Log("Exiting");
        goto case ExitAction.Exit;
    case ExitAction.Exit:
        Quit();
        break;
}

比这更简洁（特别是如果您在 Quit() 中做了更多工作）

ExitAction action = ExitAction.LogAndExit;
switch (action) {
    case ExitAction.Cancel:
        break;
    case ExitAction.LogAndExit:
        Log("Exiting");
        Quit();
        break;
    case ExitAction.Exit:
        Quit();
        break;
}

Answer 4

除了使用 goto case 之外，您还可以 goto 另一个 case 子句中的标签：

    switch(i) {
    case "0":
        // do some stuff
        break;
    case "1":
        // other stuff, then "fall through" to next case clause
        goto Case2;
    case "2":
    Case2:
        break;
    }

这样，您就可以跳转到另一个 case 子句，而不必担心值或类型的表达式。

不过，某种可以替代 break 的显式“fallthrough”关键字会很好......

Answer 5

这是 C# 允许 switch case “失败”的唯一方法。在 C# 中（与 C、C++ 或 Java 不同），switch 语句中的 case 块必须以 break 或其他显式跳转语句结束。

Answer 6

通过对 Mehrdad Afshari 上述建议的扩展，我绝不会主张简单地将某个构造作为“糟糕的代码”或“糟糕的编码实践”而放逐。甚至“goto”语句在宏伟的计划中也占有一席之地。认为它们是邪恶的教条之所以成为现实，并不是因为结构中存在任何固有的缺陷——而是因为它们被严重（且糟糕地）过度使用。

无论如何，克尼根和里奇认为，让案件败诉才是正确的做法。坦率地说，我更倾向于相信他们的推理，而不是整个华盛顿雷德蒙德任何人的想法。或者任何基于雷德蒙德任何人的智慧的教条。

如果您听到“永远不要使用 xxx”，请在心里加上“无原因”。教条地抛弃任何东西都是荒谬的。设备的存在是因为有制造它们的理由。事后看来，它们通常被称为“坏”，不是因为设备本身有任何故障，而是因为不完全理解它们的人对它们的使用不当。因此，该设备几乎不会“坏”。几乎总是不好的就是用户的理解能力。即使对于原子裂变和聚变也是如此。

我见过一些极其怪异的代码结构，其唯一的功能就是避免使用“goto”语句。更糟糕的是什么呢？ “goto [label]”，还是 30 行恶心的代码，其功能是避免键入“goto [label]”？

在教条之前寻求知识。行动前三思。这些都是有用的建议。

Answer 7

我知道这是老话题，但问题仍然是现实的。我们可以使用下一个代码而不是带有 goto 语句的丑陋版本吗？

var variable = 2;
switch (variable)
{
case 2:
Console.WriteLine("variable is >= 2");
goto case 1;
case 1:
Console.WriteLine("variable is >= 1");
break;


}

可能会被替换为下一个更简洁的代码：

if (variable >= 2)
{
Console.WriteLine("variable is >= 2");
}
if (variable >= 1)
{
Console.WriteLine("variable is >= 1");
}