Java员工登录表单验证问题：如何阻止重复弹出消息框？

核心问题1：空输入判断逻辑完全错误 #

你现在用username.contains("")来判断用户名是否为空，这是个典型误区——任何字符串都包含空字符串，所以这个条件永远返回true，哪怕用户输入了内容，这个判断也会误触发弹窗。正确的做法应该是用isEmpty()判断纯空，或者trim().isEmpty()判断空/全空格的情况（后者更严谨，能过滤用户输入的无效空格）。

核心问题2：独立的判断块导致重复弹窗 #

你代码里的空输入检查和登录验证是两个完全独立的if块：

• 第一个if-else块弹出空输入提示后，程序不会停止执行，会直接进入后面的登录验证逻辑
• 空输入肯定不满足username.contains("staff") && password.contains("pass")的条件，所以会立刻触发else分支的“Invalid Login Details”弹窗，这就导致了两个弹窗先后出现。

修复后的代码示例 #

String username = usernameTxt.getText().trim(); // 先去除前后空格，过滤无效的全空格输入
String password = passwordTxt.getText().trim();

// 第一步：先做空输入校验，校验不通过就直接终止后续逻辑
if (username.isEmpty()) {
    JOptionPane.showMessageDialog(null,"Please Enter Your Username Credentials.");
} else if (password.isEmpty()) {
    JOptionPane.showMessageDialog(null,"Please Enter Your Password Credentials.");
} else {
    // 只有输入都有效时，才执行登录验证逻辑
    if (username.equals("staff") && password.equals("pass")) { // 用equals做精确匹配，比contains更严谨
        JOptionPane.showMessageDialog(null,"Login Successfull","Success",JOptionPane.INFORMATION_MESSAGE);
        passwordTxt.setText(null);
        usernameTxt.setText(null);
        staffdashboard sd = new staffdashboard();
        sd.setVisible(true);
        this.setVisible(false);
    } else {
        JOptionPane.showMessageDialog(null,"Invalid Login Details","Login Error",JOptionPane.ERROR_MESSAGE);
        passwordTxt.setText(null);
        usernameTxt.setText(null);
    }
}

额外优化说明 #

• 把contains("staff")改成equals("staff")：contains会匹配任何包含目标字符串的内容（比如"staff123"也会通过），而登录验证通常需要精确匹配账号密码，equals更符合业务逻辑。
• 提前对输入做trim()处理：避免用户输入全空格的无效内容被当成有效输入，提升验证的严谨性。

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