数据库用户存在性检查异常排查：小型银行应用无法插入新用户

核心问题 #

你的checkIfExist方法存在两个关键错误：

1. 结果集判断逻辑错误：ResultSet对象即使没有查询到任何匹配数据，executeQuery()也会返回一个非null的实例，所以if(rs != null)永远为true，导致方法始终返回true，无法插入新用户。正确的判断方式是调用rs.next()，该方法会尝试移动到下一条记录，返回true表示存在匹配数据。
2. 变量误用：方法参数是id，但代码中却使用了外部变量customerID，这会导致传入的参数未被正确使用，查询逻辑完全失效。

修正后的checkIfExist方法 #

public boolean checkIfExist(Long id) throws SQLException { 
    String sql = "select * from customers where id=?";
    // 使用try-with-resources自动关闭资源，无需手动在finally中处理
    try (Connection conn = connect();
         PreparedStatement ps = conn.prepareStatement(sql)) {
             
        ps.setLong(1, id); // 使用方法参数id，修正变量误用问题
        try (ResultSet rs = ps.executeQuery()) {
            return rs.next(); // 直接返回是否存在匹配记录
        }
    } catch (Exception e) { 
        System.out.println(e);
        return false;
    }
}

额外优化建议 #

1. 更高效的查询语句：可以改用select count(*) from customers where id=?，避免查询全量数据，性能更优：

public boolean checkIfExist(Long id) throws SQLException { 
    String sql = "select count(*) from customers where id=?";
    try (Connection conn = connect();
         PreparedStatement ps = conn.prepareStatement(sql)) {
             
        ps.setLong(1, id);
        try (ResultSet rs = ps.executeQuery()) {
            rs.next();
            return rs.getInt(1) > 0;
        }
    } catch (Exception e) { 
        System.out.println(e);
        return false;
    }
}

2. 数据库层面的双重保障：在customers表的id字段添加唯一约束，即使代码逻辑出现问题，数据库也会拒绝重复插入，避免数据不一致：

ALTER TABLE customers ADD CONSTRAINT unique_customer_id UNIQUE (id);

3. 简化insert方法的资源管理：同样使用try-with-resources简化代码，减少手动关闭资源的冗余：

public void insert() throws SQLException {
    String sql = "INSERT INTO customers(id, name) VALUES(?, ?)";

    boolean ifExist = checkIfExist(customerID);
    if(!ifExist) { 
        try (Connection conn = connect();
             PreparedStatement ps = conn.prepareStatement(sql)) {
                 
            ps.setLong(1, customerID);
            ps.setString(2, customerName);
            ps.executeUpdate();
        } catch (Exception e) {
            e.printStackTrace();
        }
    }
}

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