Java正确实现输入循环校验 解决JOptionPane弹窗重复问题

你的代码出现重复弹窗是两个逻辑冲突叠加导致的：

• 外层do-while循环每轮会调用两次checkTaskDescription()：do代码块里调用一次，while条件判断时又调用一次，相当于每轮循环触发两次方法执行，自然弹出两次输入框。
• checkTaskDescription()方法内部存在冗余逻辑：检测到输入长度超限、弹出错误提示后，又额外弹出了一次输入框，和外层循环的重试逻辑重复。

第一步：修正checkTaskDescription方法逻辑 #

去掉方法内部多余的重复输入弹窗，只保留「单次获取输入→校验长度→返回校验结果」的逻辑，同时补上空指针处理（避免用户点击输入框取消按钮时报错）：

public boolean checkTaskDescription() {
    // 单次弹出输入框获取用户输入
    taskDesc = JOptionPane.showInputDialog(null, "Please enter a short description of the task");
    // 处理用户点击取消按钮的场景，避免空指针异常
    if (taskDesc == null) {
        taskDesc = "";
    }
    if (taskDesc.length() > 50) {
        // 输入不合法仅弹出错误提示，不在方法内做重复输入请求
        JOptionPane.showMessageDialog(null, "Please enter a task description of less than 50 characters.", "ERROR", 3);
        return false;
    }
    JOptionPane.showMessageDialog(null, "Task successfully captured.");
    return true;
}

第二步：修正外层循环逻辑 #

核心原则是每轮循环只调用一次checkTaskDescription()，不要在循环条件里重复调用方法，两种写法都可以实现需求：

写法1：直接用while循环（更简洁） #

// 只要校验不通过，就自动进入下一轮重试
while (!checkTaskDescription()) {
    // 循环体内不需要写额外逻辑
}

写法2：用do-while循环（和你原来的代码结构更接近） #

boolean isInputValid;
do {
    isInputValid = checkTaskDescription();
} while (!isInputValid);

改完后的运行逻辑：程序弹出输入框→用户输入内容长度超过50时弹出错误提示→自动重新弹出输入框→重复直到输入长度符合要求→弹出「Task successfully captured」提示结束流程，全程不会出现重复弹出输入框的问题。

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