我已经用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(正确)怎么了?


当前回答

1) do -while没有错(正确)

2)你的老师错了。

NSFS ! !:

3)大多数老师是老师而不是程序员。

其他回答

在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);

同样,通过一个简单的例子,两者都很容易理解;但是如果循环变得非常大或嵌套很深(这意味着您可能已经重构过了),那么第一种样式可能会更清晰一些。

这并不是一件可怕的事情,但是在编码时您需要考虑其他开发人员。甚至在学校。

其他开发人员应该能够在循环声明中看到循环的退出子句。你没有那样做。您将退出子句隐藏在循环的中间,为其他试图理解您的代码的人增加了更多的工作。这也是避免使用“break”这类词汇的原因。

话虽如此,在现实世界中,您仍然会在大量代码中看到类似的东西。

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

这更像是一个美观的东西,更容易阅读代码,你明确地知道为什么循环会在循环的声明中停止。

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

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

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