我不会说这很糟糕 -但同样地,我通常至少会寻找替代方案。
在我写第一件事的情况下,我几乎总是至少尝试将其重构为更清晰的内容。有时它是
bool无能为力的(或者替代方法是拥有一个变量,该变量除了指示循环的结束之外,没有其他意义,而不是一条break语句),但这至少值得一试。
作为使用
break标志比使用标志更清晰的示例,请考虑:
while (true){ doStuffNeededAtStartOfLoop(); int input = getSomeInput(); if (testCondition(input)) { break; } actonInput(input);}现在,让我们强制它使用标志:
boolean running = true;while (running){ doStuffNeededAtStartOfLoop(); int input = getSomeInput(); if (testCondition(input)) { running = false; } else { actonInput(input); }}我认为后者阅读起来更复杂:它有一个额外的
else块,
actOnInput它更加缩进,并且,如果你想弄清楚
testCondition返回时会发生什么
true,则需要仔细查看块的其余部分以检查是否存在是不是经过了else可能会出现的块是否
running已经被设置为
false与否。
该
break语句可以更清楚地传达意图,并让块的其余部分继续进行所需的操作,而不必担心较早的条件。
请注意,这与人们在一个方法中拥有多个return语句的说法完全一样。例如,如果我可以在前几行中得出方法的结果(例如,因为某些输入为空,为空或为零),那么发现直接返回该答案要比使用变量存储结果更清楚,然后是一整段其他代码,最后是一条
return语句。



