Android Studio中EditText输入验证问题：错误提示后代码仍执行

看起来你遇到的问题是验证逻辑明明触发了Toast，但后续代码还是执行了——这其实不是你的doSomeWork()方法逻辑错了，大概率是输入值的获取时机不对，或者验证后的处理逻辑没被正确拦截。我帮你梳理下问题和解决方案：

首先，修正验证逻辑的小漏洞 #

你当前的判断条件是tahunAngka < 0 || tahunAngka > 5，但你需要的是1-5的数字，所以应该改成tahunAngka < 1 || tahunAngka > 5（同理月份应该是bulanAngka <1 || bulanAngka>12），不然输入0的话不会触发错误提示。

核心问题：确保验证用的是最新的输入值 #

很多时候这种问题都是因为你用了全局变量存储tahunAngka，但没有在用户输入后实时更新这个变量就调用了doSomeWork()。比如你可能在Activity初始化时获取了一次值，之后用户输入6时，验证用的还是旧的合法值，而后续处理用了新的6。

解决方法是：每次触发操作（比如点击按钮）时，先从EditText获取最新的输入，再传入验证方法：

// 假设这是按钮点击的回调方法
public void onSubmitButtonClick(View view) {
    EditText tahunEt = findViewById(R.id.tahun_edittext);
    EditText bulanEt = findViewById(R.id.bulan_edittext);

    // 先处理空输入，避免NumberFormatException
    String tahunStr = tahunEt.getText().toString().trim();
    if (tahunStr.isEmpty()) {
        Toast.makeText(this, "请输入年份数字", Toast.LENGTH_SHORT).show();
        return;
    }
    String bulanStr = bulanEt.getText().toString().trim();
    if (bulanStr.isEmpty()) {
        Toast.makeText(this, "请输入月份数字", Toast.LENGTH_SHORT).show();
        return;
    }

    // 转成整数
    int tahunAngka = Integer.parseInt(tahunStr);
    int bulanAngka = Integer.parseInt(bulanStr);

    // 传入方法验证
    doSomeWork(tahunAngka, bulanAngka);
}

// 修改doSomeWork为接收参数，避免全局变量的坑
public void doSomeWork(int tahunAngka, int bulanAngka) {
    if (tahunAngka < 1 || tahunAngka > 5) {
        Toast.makeText(MainActivity.this, "You Must Enter a Number Between 1 - 5", Toast.LENGTH_SHORT).show();
        return; // 这里return后，下面的代码和后续逻辑都不会执行
    }
    if (bulanAngka < 1 || bulanAngka > 12) {
        Toast.makeText(MainActivity.this, "You Must Enter a Number Between 1 - 12", Toast.LENGTH_SHORT).show();
        return;
    }

    // 只有验证通过才会执行到这里，写你的业务逻辑
    // 比如：Log.d("TAG", "验证通过，年份：" + tahunAngka + "，月份：" + bulanAngka);
}

额外优化：从源头限制输入（可选） #

为了避免用户输入无效数字，你可以给EditText设置输入过滤器，直接让它只能输入1-5的数字：

EditText tahunEt = findViewById(R.id.tahun_edittext);
tahunEt.setFilters(new InputFilter[]{
    new InputFilter() {
        @Override
        public CharSequence filter(CharSequence source, int start, int end, Spanned dest, int dstart, int dend) {
            try {
                // 拼接当前输入后的完整内容
                String fullInput = dest.subSequence(0, dstart) + source + dest.subSequence(dend, dest.length());
                if (!fullInput.isEmpty()) {
                    int num = Integer.parseInt(fullInput);
                    // 不在1-5范围内就拦截输入
                    if (num < 1 || num > 5) {
                        return "";
                    }
                }
            } catch (NumberFormatException e) {
                // 输入非数字也拦截
                return "";
            }
            return null; // 允许输入
        }
    }
});

最后排查点 #

如果还是有问题，检查：

• 你的业务处理逻辑是不是写在了doSomeWork()外面？比如调用doSomeWork()之后还有代码执行，那即使doSomeWork()return了，外面的代码还是会跑。
• 有没有多次调用doSomeWork()的情况？比如按钮重复点击导致逻辑重复执行。

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