数组未填充

Question

当我稍后查看它时，数组中应该至少有 3 个条目，但它只显示一个。我相信这是你有问题的方法，有什么建议吗？

    String[] getKidsNamebyCid(int cid) {
        String[] out = new String[20];
        try {
            String qry = "SELECT KIDSNAME FROM TBLKIDS WHERE CID = ?";//setting query command
            ps = connect.prepareStatement(qry);//preparing statement
            ps.setInt(1, cid);//setting CID
            ps.executeQuery();//running command
            int i = 0;
            while (ps.getResultSet().next()) {
                out[i] = ps.getResultSet().getString("KIDSNAME");
                i++;
            }
        } catch (SQLException se) {
            se.printStackTrace();
        } catch (Exception e) {
            e.printStackTrace();
        }
        return out;
    }

Answer 1

getResultSet() 调用不是吸气剂。该方法对数据库起作用，您不能只是重复调用它；第一次获得 ResultSet 对象（需要关闭），第二次重置一切。所以不要；您需要调用 getResultSet() 一次。

我怎么知道？通过阅读。直接来自getResultSet() 文档：

每个结果只应调用此方法一次。

此外，这段代码也充斥着严重的代码问题，这些问题更普遍地集中在资源上。资源是不只是内存中的一堆位的对象，它们代表（并保持打开）“资源”。对于数据库，它是与数据库引擎的连接。您不能只制造资源，您必须始终将它们放在“保护”块中，以确保资源被清理干净。因此，除非没有其他方法，否则您永远不会希望它们成为一个字段（然后它们作为一个字段内部的东西就变成了一种资源）。

那么，您的 PreparedStatement 是一个字段这一事实？不好。你这样调用 .getResource 的事实，无人看管？不好。

最后，您的异常处理很愚蠢。面对检查异常时的默认行为是将它们添加到您的 throws 子句中。如果你做不到，正确的做法是throw new RuntimeException("uncaught", e);，而不是你所做的。

executeQuery 已经返回一个结果集。一般来说，永远不要打电话给getResultSet*。

最后，数组或多或少已经过时了；你想要收藏。

把它们放在一起：

    // delete your 'ps' field!

    List<String> getKidsNamebyCid(int cid) throws SQLException {
        var out = new ArrayList<String>();
        String qry = "SELECT KIDSNAME FROM TBLKIDS WHERE CID = ?";
        try (PreparedStatement ps = connect.prepareStatement(qry)) {
            ps.setInt(1, cid);

            try (ResultSet rs = ps.executeQuery()) {
                while (rs.next()) out.add(rs.getString("KIDSNAME"));
            }
        }
        return out;
    }

*) PreparedStatement API 非常不幸。您与 PS 交互的方式与 Statement（您应该很少使用；您不能将用户输入添加到普通的简语句中）完全不同，但是因为原因和历史 PreparedStatement 扩展了 Statement。这意味着一整套方法都在 PreparedStatements 中，你不应该调用。那真不幸。这里有两件事要学习：[1] Java 不会破坏向后兼容性，即使这意味着某些 API 很糟糕，并且 [2] JDBC 并不是真正用于“人类消费”的。我们也不用机器代码对 CPU 进行编程，但机器代码存在并将继续存在。我们使用“机器码”作为胶水；库和语言开发人员用作通用语言的东西。 JDBC 也是如此：它不适合您。使用具有良好 API 的库，例如 JDBI。这可能超出了您高中课程的要求，但是，嘿。没什么可说的，除了：在你的课程和老师中，他们让你使用“真正的”**开发人员不使用的过时工具。

**) “真实”的含义是：正在编写为获得大量资金和/或眼球的应用程序提供动力的代码。

Answer 2

您需要了解PreparedStatement 的实际工作原理。我强烈建议您按照教程来学习如何使用它，然后按照您自己的代码的模式。

但它也都在文档中，所以让我们引用各种相关的部分。

executeQuery() 的 javadoc 说：

在此PreparedStatement 对象中执行SQL 查询并返回查询生成的ResultSet 对象。

此时问题中的代码已经错了，因为它**忽略了executeQuery()调用的返回值。

另外，getResultSet() 的 javadoc 说：

以ResultSet 对象的形式检索当前结果。此方法应该每个结果只调用一次。

问题中的代码在这一点上更加错误，因为它在循环中重复调用getResultSet()。

---

如果您已经阅读了您正在使用的方法的 javadoc，那么很明显有些地方是错误的。如前所述，通过教程将展示如何正确执行此操作。实际上，任何使用 JDBC 执行查询的示例的网络搜索都会显示这一点。

有关其工作原理的更多背景信息，execute() 的 javadoc 说：

执行此PreparedStatement 对象中的SQL 语句，它可以是任何类型的SQL 语句。 一些准备好的语句返回多个结果； execute 方法处理这些复杂的语句以及由方法executeQuery 和executeUpdate 处理的更简单的语句形式。

execute 方法返回一个布尔值来指示第一个结果的形式。您必须调用方法getResultSet 或getUpdateCount 来检索结果；您必须调用 getMoreResults 才能转到任何后续结果。

getMoreResults() 的 javadoc 说：

移动到此Statement 对象的下一个结果，如果它是ResultSet 对象，则返回true，并隐式关闭使用getResultSet 方法获得的任何当前ResultSet 对象.

“返回多个结果”不是指来自单个查询的多行，而是来自多个查询的多个结果。它通常需要执行存储过程或 SQL 代码块才能发生。

---

这是从执行单个SELECT 语句中正确获取多行的方法：

String qry = "SELECT KIDSNAME FROM TBLKIDS WHERE CID = ?";
try (PreparedStatement ps = connect.prepareStatement(qry)) {
    ps.setInt(1, cid);//setting CID
    try (ResultSet rs = ps.executeQuery()) {
        int i = 0;
        while (rs.next()) {
            out[i] = rs.getString("KIDSNAME");
            i++;
        }
    }
}

如果有问题的 SQL 代码有返回了多个结果集，你可以这样做：

try (PreparedStatement ps = connect.prepareStatement(qry)) {
    // call ps.setXxx() methods here
    boolean isResultSet = ps.execute();
    while (isResultSet) {
        try (ResultSet rs = ps.getResultSet()) {
            int i = 0;
            while (rs.next()) {
                // call rs.getXxx() methods here
                i++;
            }
        }
        isResultSet = ps.getMoreResults();
    }
}

最好使用for 循环编写，以将循环逻辑保持在一起：

try (PreparedStatement ps = connect.prepareStatement(qry)) {
    // call ps.setXxx() methods here
    for (boolean isResultSet = ps.execute(); isResultSet; isResultSet = ps.getMoreResults()) {
        try (ResultSet rs = ps.getResultSet()) {
            for (int i = 0; rs.next(); i++) {
                // call rs.getXxx() methods here
            }
        }
    }
}