我已经用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(正确)怎么了?
当前回答
在我的很多函数中，我使用了类似的东西，但逻辑相反。
DWORD dwError = ERROR_SUCCESS;
do
{
if ( (dwError = SomeFunction()) != ERROR_SUCCESS )
{
/* handle error */
continue;
}
if ( (dwError = SomeOtherFunction()) != ERROR_SUCCESS )
{
/* handle error */
continue;
}
}
while ( 0 );
if ( dwError != ERROR_SUCCESS )
{
/* resource cleanup */
}
其他回答
我想没什么，真的。老师们对goto过敏，因为他们听说那里很糟糕。否则你可以这样写:
bool guard = true;
do
{
getInput();
if (something)
guard = false;
} while (guard)
这几乎是一样的。
也许这样更简洁(因为所有循环信息都包含在块的顶部):
for (bool endLoop = false; !endLoop;)
{
}
从某种意义上说，结构化编程构造优先于(有些非结构化的)中断和继续语句，这是不好的。相比之下，根据这一原则，他们更喜欢“去”。
我总是建议让你的代码尽可能结构化……不过，正如乔恩·斯基特指出的那样，不要让它更有条理!
这更像是一个美观的东西，更容易阅读代码，你明确地知道为什么循环会在循环的声明中停止。
如果您的循环在后台线程上运行，这可能会很糟糕，因此当您通过终止一个UI线程来关闭应用程序时，这段代码将继续执行。正如其他人已经说过的，你应该总是使用某种支票来提供一种取消的方式。
在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);
同样，通过一个简单的例子，两者都很容易理解;但是如果循环变得非常大或嵌套很深(这意味着您可能已经重构过了)，那么第一种样式可能会更清晰一些。
