在Java中使用带标签的中断是一种好习惯吗？

我盯着2001年的一些旧代码，发现以下语句：

   else {
     do {
       int c = XMLDocumentFragmentScannerImpl.this.scanContent();
       if (c == 60) {
         XMLDocumentFragmentScannerImpl.this.fEntityScanner.scanChar();
         XMLDocumentFragmentScannerImpl.this.setScannerState(1);
         break label913;
       }

我以前从未见过，并在这里发现标记为的中断：

http://docs.oracle.com/javase/tutorial/java/nutsandbolts/branch.html

这本质上不像goto吗？使用它甚至是一种好习惯吗？这让我不安。

不，这不像goto，因为您不能“转到”控制流的另一部分。

在您链接的页面上：

break语句终止带标签的语句；它不会将控制流转移到标签上。控制流紧接在标记（终止）的语句之后转移到该语句。

这意味着您只能中断当前正在执行的循环。

考虑以下示例：

first:
for( int i = 0; i < 10; i++) {
  second:
  for(int j = 0; j < 5; j ++ )
  {
    break xxx;
  }
}

third:
for( int a = 0; a < 10; a++) {

}

你可以替换xxx使用first或second（打破外或内循环），因为这两个循环正在执行，当你打的break说法，但将xxx与third将无法编译。

它并不那么可怕，goto因为它只将控制发送到带标签的语句（通常是循环构造）的末尾。使得goto它与众不同的是，它是一个任意分支，可以遍及任何地方，包括在方法源中的较高位置找到的标签，这样您就可以拥有自行生成的循环行为。Java中的标签破坏功能不允许这种疯狂，因为控制只能向前进行。

在大约12年前，我只使用过一次，在这种情况下，我需要打破嵌套循环，那么结构更复杂的替代方案将在循环中进行更复杂的测试。我不建议经常使用它，但我不会将其标记为自动错误代码气味。

总是有可能break用新方法代替。

考虑一下代码检查两个列表中的任何公共元素：

List list1, list2;

boolean foundCommonElement = false;
for (Object el : list1) {
    if (list2.contains(el)) {
        foundCommonElement = true;
        break;
    }
}

您可以这样重写它：

boolean haveCommonElement(List list1, List list2) {
    for (Object el : list1) {
        if (list2.contains(el)) {
            return true;
        }
    }
    return false;
}

当然，要检查两个列表这是更好地利用之间的共同元素list1.retainAll(new HashSet<>(list2))的方法来做到这一点，在O(n)与O(n)额外的内存，或排序两个列表O(n * log n)，然后找到共同的元素O(n)。

在阅读本答案的其余部分之前，请阅读认为有害的声明。如果您不想全部阅读，这里是我认为的重点：

毫无限制地使用go to语句会立即导致结果，即很难找到有意义的坐标集来描述过程进度，这非常困难。

或者换句话说，问题goto在于程序可以到达代码块的中间，而程序员在此时不了解程序状态。面向标准块的标准构造旨在清楚地描述状态转换，带标签break的目的是将程序带入特定的已知状态（包含标签的块之外）。

在现实世界中的命令性程序中，状态并没有明确地由块边界来描绘，因此，标记是否break是一个好主意还值得怀疑。如果该块更改了从该块外部可见的状态，并且有多个点要退出该块，则带标签break的等价于图元goto。唯一的区别是，您可以使用不确定状态开始新的块，而不是降落在不确定状态块中间的机会。

因此，总的来说，我认为标记break为危险的。在我看来，这是一个迹象，表明该块应被转换为功能，并且对封闭范围的访问受到限制。

但是，此示例代码显然是解析器生成器的产品（OP评论说它是Xerces源代码）。解析器生成器（或一般而言，代码生成器）通常会对其生成的代码放任自流，因为它们对状态有完全的了解，而人类无需了解状态。

这不像是goto在其中向后跳转流控制的语句。标签仅向您（程序员）显示中断发生的位置。同样，流控制将转移到break。之后的下一个语句。

关于使用它，我个人认为它没有太大用处，因为一旦您开始编写价值数千行的代码，它就变得微不足道了。但同样，这取决于用例。

至少在我看来，一种表达意图的更简洁的方法可以是将包含循环的代码片段放入一个单独的方法中，并简单地 return从中进行处理。

例如，更改此：

someLabel:
for (int j = 0; j < 5; j++)
{
    // do something
    if ...
        break someLabel;
}

到这个：

private void Foo() {
    for (int j = 0; j < 5; j++)
    {
        // do something
        if ...
            return;
    }
}

这也是开发其他语言的精通更地道，与您在未来的代码（或者工作以后你）。