0

我是编程新手，在使用带有 resultSet next() 方法的 executeUpdate 时遇到了问题。它仅通过结果集迭代一次，然后执行更新关闭结果集。我收到错误：结果集未打开。不允许操作“下一步”。确认自动提交已关闭。我添加了 con.setAutoCommit(false) 语句，但问题仍然存在。我需要使用不同的变量值多次运行更新。这是我的代码：

    try {
        String eidQuery = "SELECT EID FROM EMPLOYEE_DATA WHERE ACTIVE = TRUE ORDER BY EID";
        int nextEID;
        Statement st = con.createStatement();
        con.setAutoCommit(false);
        rs = st.executeQuery(eidQuery);
        while (rs.next()){
            nextEID = rs.getInt(1);
            String getDailyTotals = "SELECT DATE, SUM(TOTAL), MAX(OUT_1) FROM PUNCHES WHERE EID = " + nextEID + " AND DATE >= '" + fd + "' "
                    + "AND DATE <= '" + td + "' GROUP BY DATE";

            ResultSet rs2 = st.executeQuery(getDailyTotals);
            while (rs2.next()){
                double dailyTotal = rs2.getDouble(2);
                if (dailyTotal > 8){
                    double dailyOT = dailyTotal-8;
                String dailyDate = rs2.getDate(1).toString();
                Timestamp maxTime = rs2.getTimestamp(3);
                String updateOT = "UPDATE PUNCHES SET OT = " + dailyOT + " WHERE EID = " + nextEID + " AND DATE = '" + dailyDate + "' AND OUT_1 = '" + maxTime + "'";

                st.executeUpdate(updateOT);

            }
            }
        }

        rs = st.executeQuery("SELECT PUNCHES.EID, EMPLOYEE_DATA.FIRST_NAME, EMPLOYEE_DATA.LAST_NAME, SUM(PUNCHES.OT) FROM PUNCHES "
                + "JOIN EMPLOYEE_DATA ON PUNCHES.EID = EMPLOYEE_DATA.EID WHERE PUNCHES.DATE >= '" + fd + "' AND PUNCHES.DATE <= '" + td + "' GROUP BY EMPLOYEE_DATA.FIRST_NAME, EMPLOYEE_DATA.LAST_NAME, PUNCHES.EID");

        Reports.setModel(DbUtils.resultSetToTableModel(rs));

    } catch (SQLException ex) {
        Logger.getLogger(GUI.class.getName()).log(Level.SEVERE, null, ex);
        JOptionPane.showMessageDialog(null, ex);
    }
4

3 回答 3

2

您是编程新手和（显然）Java 新手。以下是我可以为您提供的一些建议：

  1. 帮自己一个忙，了解一下PreparedStatement。您不应该通过连接来创建 SQL Strings。
  2. 您正在将混合数据库和 UI Swing 代码的经典新手罪提交到一个单一的、难以调试的块中。最好将您的应用程序分解为层。从封装所有数据库代码的数据访问接口开始。进行测试并为您的 UI 提供一个可以使用的实例。
  3. 不要在 ResultSet 的循环内交错更新查询。最好将两者完全分开。
  4. 阅读有关 MVC 的信息。你会希望你的 Swing 视图与应用控制器分开。让Controller与数据访问接口交互，获取结果，并将结果交给View进行展示。保持它们解耦和分离。
  5. 学习JUnit。它将帮助您进行测试。
于 2018-04-06T15:54:38.947 回答
2

从 java.sql.ResultSet javadoc：

当生成它的 Statement 对象关闭、重新执行或用于从多个结果序列中检索下一个结果时，ResultSet 对象将自动关闭。

执行更新后，之前的 ResultSet 将关闭。您需要重新编写代码以解决此问题。 https://docs.oracle.com/javase/7/docs/api/java/sql/ResultSet.html

最简单的返工方法可能是使用两个语句，一个用于查询，一个用于更新，但正如 duffymo 的回答中所指出的，您可以做更多的事情来改进事情。

于 2018-04-06T15:57:54.673 回答
0

来自 API 的语句文档“默认情况下，每个 Statement 对象只能同时打开一个 ResultSet 对象。因此，如果一个 ResultSet 对象的读取与另一个 ResultSet 对象的读取交错，则每个对象必须由不同的 Statement 对象生成”

如果要在同一个嵌套循环中读取两个不同的 ResultSet，则需要两个不同的语句。

于 2018-04-06T16:04:13.357 回答