遇到ResultSet定位在首行前问题：Java数据库查询代码调试求助

问题根源分析 #

你的代码里有两个关键错误导致了这个报错：

1. 变量名错误导致行数统计失效：统计总行数的循环中，你写了while(resultSet.next())，但实际存储统计结果的ResultSet是temp2，这会导致numRows始终为0，后续循环逻辑完全错误。
2. 游标调用顺序错误：ResultSet初始化后，游标默认停在第一行数据之前，必须先调用.next()移动到有效行，才能读取该行数据。你现在的逻辑是先读取数据再移动游标，第一次读取时游标还处于无效位置，直接触发ResultSet is positioned before first row错误。

修正后的代码示例 #

直接重构代码，同时解决SQL注入、重复查询、资源泄漏等问题：

public List<Integer> getCars(int uID) throws SQLException {
    List<Integer> carIdList = new ArrayList<>();
    // 使用PreparedStatement避免SQL注入，同时无需额外查询行数
    String sql = "SELECT CarID FROM Car WHERE UserID = ?";
    
    // try-with-resources自动关闭资源，避免内存泄漏
    try (PreparedStatement stmt = yourConnectionObject.prepareStatement(sql)) {
        stmt.setInt(1, uID);
        try (ResultSet rs = stmt.executeQuery()) {
            // 遍历ResultSet：先移动游标到有效行，再读取数据
            while (rs.next()) {
                carIdList.add(rs.getInt("CarID"));
            }
        }
    }
    
    // 如果确实需要返回数组，可转成int数组
    // return carIdList.stream().mapToInt(Integer::intValue).toArray();
    return carIdList;
}

关键优化点说明 #

• 游标调用顺序：永远先调用rs.next()判断是否有有效行，再调用getXXX()读取数据，这是ResultSet的基本使用规则。
• 避免重复查询：不需要单独查询总行数，直接遍历ResultSet即可，减少一次数据库请求，提升性能。
• SQL注入防护：用PreparedStatement代替字符串拼接的查询方式，避免恶意SQL注入攻击。
• 自动资源管理：使用try-with-resources语法，确保Statement和ResultSet自动关闭，避免资源泄漏。
• 替代数组：用List代替固定长度数组，无需提前知道行数，避免数组越界或空间浪费。

原代码的其他问题提示 #

• 原代码中cArr未初始化，即使解决了游标问题，也会触发空指针异常。
• 两次查询相同条件，数据库会重复执行过滤逻辑，完全没必要。

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