Android Java开发问题：Toast始终执行else分支

嘿，我一眼就看到问题出在哪啦！你的代码里有几个关键小错误，导致不管输入什么分数都只会触发else分支，咱们一步步来解决：

核心问题 #

• 硬编码的固定分数：你一开始把全局变量score设成了100，而且在按钮点击逻辑里从来没更新过这个变量的值——不管用户在输入框填了什么，程序判断的永远是100，自然会走到最后那个else分支。
• 没读取用户输入内容：你虽然找到了editTextNumber这个输入控件，但完全没用到它来获取用户输入的分数，这才是最关键的遗漏点！
• 重复设置点击监听（额外坑）：你应该是在布局里用android:onClick把按钮绑定到了checkResults方法，但又在这个方法里嵌套了一层setOnClickListener。这会导致第一次点击按钮时，只执行了Log和设置监听的操作，第二次点击才会触发分数判断——这也是逻辑混乱的原因之一！

修复后的完整代码 #

// 删掉原来的全局score变量
public void checkResults(View view) {
    Log.i("Results", "Button Tapped");
    TextView editTextNumber = findViewById(R.id.editTextNumber);
    
    // 先获取用户输入的文本，处理空输入的情况
    String scoreInput = editTextNumber.getText().toString().trim();
    if (scoreInput.isEmpty()) {
        Toast.makeText(getApplicationContext(), "请输入分数哦", Toast.LENGTH_SHORT).show();
        return;
    }
    
    int score;
    // 处理非数字输入，避免程序崩溃
    try {
        score = Integer.parseInt(scoreInput);
    } catch (NumberFormatException e) {
        Toast.makeText(getApplicationContext(), "请输入有效的数字呀", Toast.LENGTH_SHORT).show();
        return;
    }
    
    // 成绩判断逻辑
    if (score < 29) {
        Toast.makeText(getApplicationContext(), "You got an F", Toast.LENGTH_SHORT).show();
    } else if (score >= 30 && score <= 39) {
        Toast.makeText(getApplicationContext(), "You got E", Toast.LENGTH_SHORT).show();
    } else if (score >= 40) { // 把else改成明确的条件，逻辑更清晰
        Toast.makeText(getApplicationContext(), "You passed!", Toast.LENGTH_SHORT).show();
    }
}

关键修复点说明 #

1. 移除全局固定score：每次点击按钮时，都从输入框读取最新的用户输入，而不是用一开始硬编码的100。
2. 处理输入异常：加入了空输入和非数字输入的判断，避免用户输入无效内容导致程序崩溃。
3. 去掉重复的点击监听：直接在绑定的checkResults方法里处理所有逻辑，点击按钮就会立即执行分数判断，不用点两次。
4. 明确else分支的条件：把原来的else改成score >=40，让逻辑更清晰，也能避免歧义。

内容的提问来源于stack exchange，提问作者Bisonalizer