无间断切换

Question

我有一些 switch 语句，如下所示。请注意，没有休息。 Findbugs 仅报告第二个 case 语句的错误。 错误是：发现 Switch 语句，其中一个案例落入下一个案例。

switch(x) {

    case 0:
        // code

    case 1:
        // code

    case 2:
        // code
}

Answer 1

Findbugs 指出，如果第一个案例中有任何代码，那么从一个案例过渡到下一个案例通常不是一个好主意（尽管有时它可以起到很好的效果）。因此，当它看到第二个 case 并且没有 break 时，它会报告错误。

例如：

switch (foo) {
    case 0:
        doSomething();
    case 1:
        doSomethingElse();
    default:
        doSomeOtherThing();
}

这是完全有效的 Java，但它可能没有达到作者的预期：如果 foo 是 0，全部三个函数 doSomething、doSomethingElse 和 doSomeOtherThing 运行（按顺序）。如果 foo 为 1，则仅运行 doSomethingElse 和 doSomeOtherThing。如果 foo 是任何其他值，则仅运行 doSomeOtherThing。

相反：

switch (foo) {
    case 0:
        doSomething();
        break;
    case 1:
        doSomethingElse();
        break;
    default:
        doSomeOtherThing();
        break;
}

这里，只有一个函数会运行，具体取决于 foo 的值。

由于忘记 break 是一种常见的编码错误，Findbugs 等工具会为您标记它。

有一个常见的用例，您连续有多个 case 语句，并且没有中间代码：

switch (foo) {
    case 0:
    case 1:
        doSomething();
        break;
    case 2:
        doSomethingElse();
        break;
    default:
        doSomeOtherThing();
        break;
}

在那里，我们想要调用 doSomething if <代码>foo是0或1。大多数工具不会将此标记为可能的编码错误，因为在 case 1 之前的 case 0 中没有代码，而且这是一种相当常见的模式。

Answer 2

我把这些写成评论，但后来就看不到了。我正在把它们变成一个答案。这实际上是 TJCrowder 的答案的扩展。

您可以在此处找到导致 Findbugs 报告错误的相关规则。

您可以通过创建包含以下内容（例如 filter.xml）的 xml 文件并使用 -exclude filter.xml 选项运行该工具来防止 Findbugs 报告此类错误。请参阅Findbugs 上的过滤器。

<FindBugsFilter>
  <Match>
    <Bug category="PERFORMANCE" />
  </Match>
</FindBugsFilter>

Answer 3

Switch 失败属于 Findbugs 的“狡猾代码”类别。我认为它只是标记 switch 语句中第一次出现的失败，以减少错误消息的数量。

Answer 4

如果没有中断，它们将相互陷入，因此如果 x == 0 您将遍历每个 case 语句块中的所有代码。 Findbugs 可能对错误有误，也可能是没有中断的错误条件，即 case 0 中的某些内容导致 case 1 中的某些内容中断。

如果没有确切的代码和错误，我无法真正提供进一步的帮助。缺乏休息时间是故意的吗？

Answer 5

通常错误分析工具不喜欢代码中的失败，因为在大多数情况下，用户只是忘记编写中断。

我不知道是否有办法专门禁用 FindBugs 的警告，但 Checkstyle 工具可以识别特殊注释，例如 /* Fallthrough */ 假设用户确实希望执行以下代码。添加这种注释也可以提高可读性。
http://checkstyle.sourceforge.net/config_coding.html#FallThrough

Java 代码约定还提到了fallthrough注释的使用。
http://www.oracle.com/technetwork/java/ javase/documentation/codeconventions-142311.html

Answer 6

如果您的案例没有中断，一旦触发案例，开关将遍历所有案例，直到找到案例的中断或结束。如果您有默认情况，如果您在这种情况下 foo 的开关值与任何其他情况不匹配，它将触发。

Answer 7

是的，也可以使用不带中断的 switch 语句。仅有语法上的变化。例如：

String fruit="Apple";
switch (fruit)
{
    case "Orange": System.out.println ("Orange");
    
    case "Apple":  System.out.println ("Apple");
    
    default:  System.out.println ("Invalid");
    
}

像这样使用 switch 语句称为 enchaned switch