2

我试图编写以下代码以允许在输入 E 时连续抛硬币并退出。不确定 do-while 循环是否是连续执行此操作的正确方法,或者我应该使用另一种方法。

do {
    guess = sc.next();
    tossGenerator(guess);
    }while(!guess.equals("E")||!guess.equals("e"));

那么,我是不是因为无法跳出 do 循环或应该使用不同的方法而错误地对代码进行了表述。请帮忙。谢谢。

4

4 回答 4

8

更改&&||

} while (!guess.equals("E") && !guess.equals("e"));

或者像这样重新排列它:

} while (!(guess.equals("E") || guess.equals("e")));

或者,您可以使用String.equalsIgnoreCase()和消除连词

} while (!guess.equalsIgnoreCase("e"));
于 2012-11-21T17:15:10.277 回答
3

将其更改为

while(!guess.equalsIgnoreCase("E") );
于 2012-11-21T17:16:04.223 回答
2

退出条件应使用 AND 运算符:

!guess.equals("E") && !guess.equals("e")

否则任何"E"or"e"都会使至少其中一个变得微不足道,因为如果它是“e”,那么它就不是“E”,反之亦然。

于 2012-11-21T17:15:42.617 回答
1

您的代码的一个问题是,tossGenerator(guess)即使guess是“e”,它也会调用。另一个是它guess总是不是“e”或不是“E”(不能同时是两者)。我会这样写:

guess = sc.next();
while (!"e".equalsIgnoreCase(guess)) {
    tossGenerator(guess);
    guess = sc.next();
}

或者,使用for循环:

for (guess = sc.next(); !"e".equalsIgnoreCase(guess); guess = sc.next()) {
    tossGenerator(guess);
}
于 2012-11-21T17:17:06.343 回答