我已经用Java编程好几年了,但我最近才回到学校获得一个正式学位。我很惊讶地发现,在我的上一个作业中,我因为使用下面这样的循环而失分。

do{
     //get some input.
     //if the input meets my conditions, break;
     //Otherwise ask again.
} while(true)

现在对于我的测试,我只是扫描一些控制台输入,但我被告知这种循环是不鼓励的,因为使用break类似于goto,我们只是不这样做。

我完全理解goto和它的Java兄弟break:label的陷阱,而且我很明智地不使用它们。我也意识到,一个更完整的程序会提供一些其他的逃避方式,比如结束程序,但我的教授并没有提到这一点,所以……

do-while(正确)怎么了?


当前回答

如果您的循环在后台线程上运行,这可能会很糟糕,因此当您通过终止一个UI线程来关闭应用程序时,这段代码将继续执行。正如其他人已经说过的,你应该总是使用某种支票来提供一种取消的方式。

其他回答

根据我的经验,在大多数情况下循环有“主要”条件继续。这是应该写入while()操作符本身的条件。所有其他可能打破循环的条件都是次要的,不那么重要。它们可以被写成额外的if() {break}语句。

而(正确)往往令人困惑,可读性较差。

我认为这些规则并不能覆盖100%的情况,而可能只有98%。

对我来说,问题在于可读性。

带有true条件的while语句不会告诉您任何关于循环的信息。这使得理解它的工作更加困难。

从这两个片段中更容易理解什么?

do {
  // Imagine a nice chunk of code here
} while(true);

do {
  // Imagine a nice chunk of code here
} while(price < priceAllowedForDiscount);

也许是我运气不好。或许我只是缺乏经验。但是每次我回忆起处理while(true)内部有break时,可以改进将Extract Method应用到while-block的代码,它保留了while(true),但(巧合吗?)将所有的break转换为返回。

根据我的经验,没有休息(即返回或抛出)是相当舒服和容易理解的。


  void handleInput() {
      while (true) {
          final Input input = getSomeInput();
          if (input == null) {
              throw new BadInputException("can't handle null input");
          }
          if (input.isPoisonPill()) {
              return;
          }
          doSomething(input);
      }
  }

我猜用break对老师来说就像折断树枝来得到果实,用一些其他的技巧(向树枝鞠躬),这样你得到了果实,而树枝仍然活着。

虽然不一定是为什么不使用While的答案(正确),但我一直觉得这个漫画和伴随作者的声明简明地解释了为什么用While而不是do- While。

关于你的问题:没有内在的问题

while(true) {
   do_stuff();
   if(exit_time) {
      break;
   }
}

... 如果您知道自己在做什么,并确保exit_time在某个时刻的值为true。

老师不鼓励你使用while(正确),因为除非你知道自己在做什么,否则很容易犯重大错误。