我已经用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(正确)怎么了?
当前回答
根据我的经验，在大多数情况下循环有“主要”条件继续。这是应该写入while()操作符本身的条件。所有其他可能打破循环的条件都是次要的，不那么重要。它们可以被写成额外的if() {break}语句。
而(正确)往往令人困惑，可读性较差。
我认为这些规则并不能覆盖100%的情况，而可能只有98%。
其他回答
这是你的枪，你的子弹和你的脚…
说它不好是因为你在自找麻烦。你或本页上的任何其他海报都不会有简短的while循环的例子。
麻烦将在未来某个非常随机的时间开始。可能是其他程序员造成的。可能是安装软件的人。可能是最终用户。
为什么?我必须找出为什么700K的LOC应用程序会逐渐开始消耗100%的CPU时间，直到每个CPU都饱和。这是一个神奇的while (true)循环。这件事又大又恶心，但归结起来就是:
x = read_value_from_database()
while (true)
if (x == 1)
...
break;
else if (x ==2)
...
break;
and lots more else if conditions
}
没有最后的其他分支。如果值不匹配If条件，循环将一直运行到时间结束。
当然，程序员指责最终用户没有选择程序员期望的值。(然后我在代码中删除了while(true)的所有实例。)
恕我直言，使用while(true)这样的结构并不是很好的防御性编程。它会回来困扰你的。
(但我确实记得，如果我们不评论每一行，教授就会扣分，即使是i++;)
在break语句中使用while(正确)没有大问题，但是有些人可能会认为它会略微降低代码的可读性。试着给变量起个有意义的名字，在合适的位置求值表达式。
对于你的例子，这样做似乎更清楚:
do {
input = get_input();
valid = check_input_validity(input);
} while(! valid)
如果do while循环变长，尤其如此——您确切地知道在哪里检查是否发生了额外的迭代。在抽象级别上，所有变量/函数都有适当的名称。while(true)语句的作用是告诉你，处理不在你认为的地方。
也许你想在第二次循环时得到不同的输出。类似的
input = get_input();
while(input_is_not_valid(input)) {
disp_msg_invalid_input();
input = get_input();
}
这样我就更容易理解了
do {
input = get_input();
if (input_is_valid(input)) {
break;
}
disp_msg_invalid_input();
} while(true);
同样，通过一个简单的例子，两者都很容易理解;但是如果循环变得非常大或嵌套很深(这意味着您可能已经重构过了)，那么第一种样式可能会更清晰一些。
Douglas Crockford说过，他多么希望JavaScript包含一个循环结构:
loop
{
...code...
}
我也不认为Java会因为拥有循环结构而变得更糟糕。
while(true)循环本身并没有什么错，但是老师们倾向于不鼓励它们。从教学的角度来看，很容易让学生创造出无穷无尽的循环，而不理解为什么循环没有被转义。
但是他们很少提到的是，所有的循环机制都可以用while(true)循环复制。
while( a() )
{
fn();
}
和
loop
{
if ( !a() ) break;
fn();
}
and
do
{
fn();
} while( a() );
等于:
loop
{
fn();
if ( !a() ) break;
}
and
for ( a(); b(); c() )
{
fn();
}
等于:
a();
loop
{
if ( !b() ) break;
fn();
c();
}
只要你能以一种工作的方式设置你的循环，你选择使用的结构是不重要的。如果它恰好适合for循环，则使用for循环。
最后一点:保持循环的简单性。如果在每次迭代中都需要执行许多功能，那么就将其放入一个函数中。在你让它工作之后，你总是可以优化它。
1) do -while没有错(正确)
2)你的老师错了。
NSFS ! !：
3)大多数老师是老师而不是程序员。
我想没什么，真的。老师们对goto过敏，因为他们听说那里很糟糕。否则你可以这样写:
bool guard = true;
do
{
getInput();
if (something)
guard = false;
} while (guard)
这几乎是一样的。
也许这样更简洁(因为所有循环信息都包含在块的顶部):
for (bool endLoop = false; !endLoop;)
{
}
