Swing登录GUI无法正常登录，代码无报错求助（需匹配时显示成功提示）

嘿，我帮你找到了代码里的核心问题，还有几个可以优化的点，一起来看看：

首先，你的登录逻辑里有个致命的小bug：遍历所有数据库记录时，只要遇到一条不匹配的记录，就会把Message的文本改成"Failed"，哪怕之前已经找到匹配的记录并设置了"Login Success"。比如数据库里有正确的用户名密码，但后面还有其他员工记录，循环到后面的时候就会覆盖掉成功的提示，最终显示的是最后一条记录的匹配结果，这就导致即使有正确的账号也可能显示失败。

下面是具体的修复方案和优化建议：

1. 最优方案：精准查询匹配用户（推荐） #

不要遍历所有记录，直接用SQL语句根据用户名查询对应的密码，既高效又避免循环覆盖的问题，还能防止SQL注入：

public Login() {
    submitButton.addActionListener(new ActionListener() {
        @Override
        public void actionPerformed(ActionEvent e) {
            // 先处理空输入情况
            String username = nameTextField.getText().trim();
            String password = passwordTextField.getText().trim();
            if (username.isEmpty() || password.isEmpty()) {
                Message.setText("请输入用户名和密码");
                return;
            }

            // 使用try-with-resources自动关闭数据库资源，避免资源泄漏
            try (Connection con = DriverManager.getConnection("jdbc:mysql://localhost:3306/java_db", "root", "")) {
                String sql = "SELECT password FROM employee WHERE name = ?";
                try (PreparedStatement pst = con.prepareStatement(sql)) {
                    pst.setString(1, username);
                    try (ResultSet rs = pst.executeQuery()) {
                        if (rs.next()) {
                            // 找到对应用户，对比密码
                            if (rs.getString("password").equals(password)) {
                                Message.setText("Login Success");
                            } else {
                                Message.setText("密码错误");
                            }
                        } else {
                            // 没有找到该用户
                            Message.setText("用户名不存在");
                        }
                    }
                }
            } catch (SQLException e1) {
                e1.printStackTrace();
                Message.setText("数据库连接异常");
            }
            // 注意：MySQL 8.0+ 不需要手动加载Driver类，DriverManager会自动发现
        }
    });
}

2. 修复原循环逻辑的方案（如果坚持遍历所有记录） #

如果你一定要保留遍历所有记录的写法，那需要用一个标记记录是否找到匹配项，并且找到后立即退出循环，最后统一设置提示文本：

public Login() {
    submitButton.addActionListener(new ActionListener() {
        @Override
        public void actionPerformed(ActionEvent e) {
            Connection con = null;
            Statement st = null;
            ResultSet rs = null;
            boolean isLoginSuccess = false;

            try {
                Class.forName("com.mysql.cj.jdbc.Driver");
                con = DriverManager.getConnection("jdbc:mysql://localhost:3306/java_db", "root", "");
                st = con.createStatement();
                rs = st.executeQuery("select * from employee");
                
                while (rs.next()) {
                    if (rs.getString("name").equals(nameTextField.getText()) && 
                        rs.getString("password").equals(passwordTextField.getText())) {
                        isLoginSuccess = true;
                        break; // 找到匹配项后立即退出循环，避免覆盖结果
                    }
                }

                // 循环结束后再统一设置提示
                if (isLoginSuccess) {
                    Message.setText("Login Success");
                } else {
                    Message.setText("Failed");
                }
            } catch (SQLException e1) {
                e1.printStackTrace();
                Message.setText("数据库错误");
            } catch (ClassNotFoundException e1) {
                e1.printStackTrace();
                Message.setText("驱动加载失败");
            } finally {
                // 确保所有资源都被关闭，避免泄漏
                try {
                    if (rs != null) rs.close();
                    if (st != null) st.close();
                    if (con != null) con.close();
                } catch (SQLException e2) {
                    e2.printStackTrace();
                }
            }
        }
    });
}

3. 额外优化建议 #

• 把密码输入框改成JPasswordField，这样输入的密码会以圆点显示，更安全：

  private JPasswordField passwordField; // 替换原来的JTextField passwordTextField
  

  获取密码时用new String(passwordField.getPassword())。
• 不要硬编码数据库连接信息（用户名、密码、URL），可以放在配置文件里，方便后续维护。
• 对用户输入做trim()处理，避免前后空格导致的不必要匹配失败。

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