首先,你通常不应该使用!=来比较字符串;使用equals()代替. ==和!=运算符只会测试字符串是否是相同的对象;他们不测试相同的价值.其次,你需要像这样扩展表达式:
if (!mGuess.equals("1") || !mGuess.equals("2") || /* etc */) { . . .
最后,这个逻辑实际上没有任何意义.条件将始终为真(mGuess将始终与至少除一个测试字符串之外的所有字符串“不相等”).你可能想要:
if (!mGuess.equals("1") && !mGuess.equals("2") && /* etc */) { . . .
更简洁的方法是:
List validStrings = Arrays.asList("1", "2", ...);
if (!validStrings.contains(mGuess)) { ...
(您可以将validStrings声明为静态类成员,以便每次通过该方法保存创建一个.另外,请参阅the answer by assylias,了解如何使用HashSet而不是ArrayList进行查找;它将更快地执行查找.)
附:正如assylias和kcoppock在评论中所提到的,您应该考虑将输入解析为int值,然后进行数值测试.不同的是,解析为int会将“07”视为与“7”相同.如果你想允许,那么这段代码将完成这项工作:
boolean ok = false;
try {
int guess = Integer.parseInt(mGuess);
ok = guess >= 1 && guess <= 10;
} catch (NumberFormatException ignored) {
}
if (!ok) { . . .