基于SQLite的Java多角色登录功能开发问题求助（NetBeans环境）

咱们先梳理你代码里的几个关键问题，这些就是导致登录判断逻辑失效的核心原因：

1. PreparedStatement 使用错误（SQL注入风险+参数未绑定） #

你写了带?的PreparedStatement，但完全没给这个占位符赋值，还在手动拼接用户名和密码的字符串——这既浪费了PreparedStatement的安全优势（防SQL注入），还会导致SQL语句语法错误，因为Position=?里的?没有被替换，执行查询时必然出问题。

2. 登录判断逻辑的低级错误 #

• passwordCon1.getText().matches(Pass1.getText())：matches()是用来做正则表达式匹配的，你只是想判断密码是否一致，应该用equals()或者equalsIgnoreCase()（如果不需要区分大小写的话）。
• usernameCon1.getText().matches(regex)：你代码里根本没定义regex变量，这会直接触发编译或运行错误；而且完全没必要用正则匹配用户名，既然已经从数据库查到了匹配项，直接判断非空或者和输入一致就行。
• 用隐藏JTextField中转查询结果完全是多此一举，既增加复杂度，还可能因为控件初始化或赋值延迟导致判断失效，直接在查询后处理跳转逻辑或者返回一个封装的用户信息更靠谱。

修复后的完整代码示例 #

第一步：重构数据库查询方法，直接返回用户职位 #

我们让查询方法直接返回匹配到的职位，没匹配到就返回null，逻辑更清晰：

// 重构后的数据库查询方法
private String loginDatabase(String userName, String passWord) { 
    String userPosition = null;
    try { 
        // 正确使用PreparedStatement，所有参数用占位符，避免SQL注入
        String query = "select Position from LoginManager where Username=? and Password=?"; 
        PreparedStatement pst = conn.prepareStatement(query); 
        // 给占位符赋值，索引从1开始
        pst.setString(1, userName);
        pst.setString(2, passWord);
        ResultSet rs = pst.executeQuery(); 
        // 如果查询到结果，获取职位
        if (rs.next()){ 
            userPosition = rs.getString("Position"); 
        } 
        // 按顺序关闭资源
        rs.close();
        pst.close(); 
    } catch(Exception e){ 
        JOptionPane.showMessageDialog(null, "数据库查询错误：" + e.getMessage()); 
        e.printStackTrace();
    } 
    return userPosition;
} 

第二步：重构登录逻辑，简化判断流程 #

private void login(){ 
    String userNameInput = Usr1.getText().trim();
    String passWordInput = Pass1.getText().trim();
    
    // 先校验输入是否为空
    if (userNameInput.isEmpty() || passWordInput.isEmpty()) {
        JOptionPane.showMessageDialog(null, "请输入账号和密码", "输入错误", JOptionPane.WARNING_MESSAGE);
        return;
    }

    // 调用查询方法获取用户职位
    String position = loginDatabase(userNameInput, passWordInput);

    // 根据职位跳转对应界面
    if (position == null) {
        JOptionPane.showMessageDialog(null, "账号或密码错误", "登录失败", JOptionPane.ERROR_MESSAGE);
    } else {
        // 清空输入框
        Usr1.setText("");
        Pass1.setText("");
        // 用switch更清晰的处理多职位跳转
        switch(position) {
            case "Administrator":
                Admin(); // 你的管理员界面跳转方法
                break;
            case "Manager":
                // 这里写Manager界面的跳转逻辑，比如 new ManagerFrame().setVisible(true);
                JOptionPane.showMessageDialog(null, "欢迎经理登录");
                break;
            case "Teacher":
                // 这里写Teacher界面的跳转逻辑
                JOptionPane.showMessageDialog(null, "欢迎教师登录");
                break;
            default:
                JOptionPane.showMessageDialog(null, "未知职位，无法跳转", "登录错误", JOptionPane.ERROR_MESSAGE);
        }
    }
} 

额外优化建议 #

• 密码加密存储：现在你直接存明文密码，非常不安全，建议用BCrypt或者SHA-256对密码加密后再存入数据库，登录时对输入密码加密后再和数据库密文比对。
• 用try-with-resources自动关闭资源：Java 7及以上支持该特性，能自动关闭ResultSet、PreparedStatement等资源，避免资源泄漏，示例：

try (PreparedStatement pst = conn.prepareStatement(query);
     ResultSet rs = pst.executeQuery()) {
    // 查询逻辑写在这里
} catch (Exception e) {
    // 异常处理
}

• 避免硬编码字符串：把"Administrator"、"Manager"这些职位字符串定义成常量，比如private static final String ROLE_ADMIN = "Administrator";，后期修改更方便，还能避免拼写错误。

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