运营商|| 未定义参数类型boolean,String

Question

我继续使用以下IF语句获取上述错误消息.任何帮助表示赞赏.

public void sendMessage(View button) {
        String mName = Name.getText().toString();
        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();
        }

Answer 1

首先,您通常不应该使用!=比较字符串; 使用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查找;它将更快地执行查找.)

PS正如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) { . . .

Answer 2

您需要将每个条件设置为已明确说明.一种更紧凑的写作方式是:

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) {
}

Answer 3

编译器错误¹的原因是表达式

mGuess != "1" || "2" || ..

被解析等价于

((mGuess != "1") || "2") || ..

但是,类型myGuess != "1"是boolean,所以上面的表达式被输入为

((boolean) || String) || String) || ..

但是boolean || String根据编译器错误无效:

运营商|| 未定义参数类型boolean,String

¹请参阅解决方案的其他答案之一.