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类成员，以便每次通过该方法保存创建一个。另请参阅the answer by assylias了解如何使用HashSet代替ArrayList用于查找;它将更快地执行查找。）

P.S。正如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

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

Answer 4

您需要使用&&评估否定表达式，使用.equals进行String比较，并在if语句中使用语法正确的表达式：

if (!mGuess.equals("1") && !mGuess.equals("2") && ...

另见：Java String.equals versus ==

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

4 个答案: