The operator || is undefined for the argument type(s) boolean, String
我不断收到上述错误消息,并带有以下
if
语句:
String mGuess = Guess.getText().toString();
if (mGuess != "1" || "2" || "3" || "4" || "5" || "6" || "7" || "8" || "9" || "10") {
Toast.makeText(MainActivity.this,
"The number you entered was invalid. Please try again.",
Toast.LENGTH_LONG).show();
}
首先,通常不应该使用
!=
来比较字符串;请使用 equals()
代替。 ==
和 !=
运算符只会测试字符串是否是相同的对象;他们不测试相同的值。其次,您需要像这样扩展表达式:
if (!mGuess.equals("1") || !mGuess.equals("2") || /* etc */) { . . .
最后,这个逻辑实际上没有任何意义。条件将始终为真(
mGuess
始终“不等于”至少除一个测试字符串之外的所有字符串)。您可能想要:
if (!mGuess.equals("1") && !mGuess.equals("2") && /* etc */) { . . .
更简洁的方法是:
List<String> validStrings = Arrays.asList("1", "2", ...);
if (!validStrings.contains(mGuess)) { ...
(您可以将
validStrings
声明为 static
类成员,以节省每次通过该方法创建一个成员。另外,请参阅 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) { . . .
您需要按照已经解释的方式明确每个条件。更紧凑的编写方式是:
Set<String> oneToTen = new HashSet<String> (Arrays.asList("1", "2", "3", "4", "5", "6", "7", "8", "9", "10");
if (!oneToTen.contains(mGuess)) {
或者,如果您知道 mGuess 是一个数字,您可以先将其解析为整数:
int guess = Integer.parseInt(mGuess);
if (guess < 0 || guess > 10) {
}
编译器错误1的原因是表达式
mGuess != "1" || "2" || ..
等同于
进行解析((mGuess != "1") || "2") || ..
但是,
myGuess != "1"
的类型是boolean
,所以上面的表达式输入为
((boolean) || String) || String) || ..
但是
boolean || String
是无效的,根据编译器错误:
运算符||参数类型布尔值、字符串未定义
1 请参阅其他答案之一以获取解决方案。
您需要使用
&&
来评估否定表达式,使用 .equals
进行 String
比较,并在 if
语句中使用语法正确的表达式:
if (!mGuess.equals("1") && !mGuess.equals("2") && ...