Java 参数类型为布尔字符串的运算符| |未定义

我通过下面的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();
        }

---

首先，您通常不应该使用

=以比较字符串；改用
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)) { ...

Set<String> oneToTen = new HashSet<String> (Arrays.asList("1", "2", "3", "4", "5", "6", "7", "8", "9", "10");

if (!oneToTen.contains(mGuess)) {

您需要使用
&&
来计算否定表达式，使用
.equals
进行
字符串比较，并在
if
语句中使用语法正确的表达式：

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

另请参见：
您需要按照前面的说明明确每个条件。更简洁的编写方法是：

List<String> validStrings = Arrays.asList("1", "2", ...);
if (!validStrings.contains(mGuess)) { ...

Set<String> oneToTen = new HashSet<String> (Arrays.asList("1", "2", "3", "4", "5", "6", "7", "8", "9", "10");

if (!oneToTen.contains(mGuess)) {

Set-oneToTen=新的HashSet（Arrays.asList（“1”、“2”、“3”、“4”、“5”、“6”、“7”、“8”、“9”、“10”）；
如果（！oneToTen.包含（mGuess））{

或者，如果您知道mGuess是一个数字，可以先将其解析为整数：

int guess = Integer.parseInt(mGuess);
if (guess < 0 || guess > 10) {
}

int guess=Integer.parseInt（mGuess）；
如果（猜测<0 | |猜测>10）{
}
编译器错误1的原因是表达式

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

被等效地解析为

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

但是，
myGuess！=“1”
的类型是
boolean
，因此上面的表达式类型为

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

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

参数类型为布尔字符串的运算符| |未定义


我可以从其他答案中找到解决方案。
答案非常好且彻底。顺便说一句，在有大量字符串的情况下，
HashSet
或类似的方法可能是更有效的选择。@JeremyRoman-很好。我想我确实提出了这个建议。事实上。更进一步说，这个数据结构永远不会改变，而且应该是类的
静态final
成员。此外，如果它始终是数值的，您可以解析输入（当然是针对NumberFormatExceptions进行保护），然后打开该结果。这样会更清楚。编辑：Oops@assylias已经建议过了。：）@kcoppock-您必须将数字转换包装在一个try/catch中，以处理空白字段或非数字输入。对我来说，这是否更清晰是个问题。此外，OP可能实际上希望排除诸如“07”之类的输入。另一方面，如果OP希望将“07”与“7”一样处理，然后将其解析为int肯定是一个不错的选择。+1令人惊讶的是，我们中的其他人都没有想到解释错误消息本身。值得指出的是，
|
也不是为两个
字符串
参数定义的。@TedHopp没有必要与您的答案竞争，您已经确定了这一点。