0

我有一个名为数据库的课程。

    public class Database {
    public Connection connect = null;
    public Statement st = null;
    public PreparedStatement ps = null;
    public ResultSet rs = null;

    public boolean connectDB() throws Exception {
      try {

      Class.forName("com.mysql.jdbc.Driver");

      connect = DriverManager
        .getConnection("jdbc:mysql://localhost/ots?"
              + "user=root&password=mongolia");
    } catch (Exception e) {
      System.out.println(e);
    }
    return true;
   }

    public void disconnectDB() {
    try {
      if (rs != null) {
        rs.close();
      }

      if (st != null) {
        st.close();
      }

      if (connect != null) {
        connect.close();
      }
    } catch (Exception e) {

    }
    }

     } 

和名为用户的类,它正在扩展数据库类

public class User extends Database {
    public ResultSet fetchTable(){
        try{
            connectDB();            
            st = connect.createStatement();
            rs = st.executeQuery("SELECT * FROM user");         
        }catch(Exception e){
            System.out.println(e);
        }finally{
            disconnectDB();
        }
        return rs;
    }
  }
//Inside JSP page
  User user = new User();
  ResultSet data = user.fetchTable();

  //getting exception in the data.next() function
  //java.sql.SQLException: Operation not allowed after ResultSet closed

  while(data.next()){
        out.println("<p>"+data.getInt(0)+"</p>");
   }

//getting exception in the data.next() function
//java.sql.SQLException: Operation not allowed after ResultSet closed
4

1 回答 1

6

例外是完全可以预料的。您正在连接数据库、获取结果集、关闭数据库和结果集,然后尝试访问已关闭的结果集。

这不是 JDBC 应该如何工作的。

您需要List<User>在检索结果集后直接将其映射到 a ,然后关闭结果集并返回List<User>

对于一些具体的例子,前往这个问题的答案:JDBC driver throws "ResultSet Closed" exception on empty ResultSet


与具体问题无关,代码中还有其他严重问题。其中,您已将 和 声明为实例变量Connection,而不是方法局部变量。当多个线程之间共享同一个实例时,这将很难失败(当两个或多个用户同时访问您的 Web 应用程序时可能会发生这种情况)。我也会解决这个问题。StatementResultSet


更新:到目前为止发布的其他答案建议删除disconnectDB()调用或仅在迭代其他方法中的结果集后调用它。这是错误的。你不应该方法传出ResultSet去。您的代码仍将是线程不安全的,并且在发生异常时您仍会冒资源泄漏的风险。您应该在同一个方法块中创建、使用和关闭它。这是从上述问题复制粘贴的正确方法:

public List<User> list() throws SQLException {
    Connection connection = null;
    PreparedStatement statement = null;
    ResultSet resultSet = null;
    List<User> users = new ArrayList<User>();

    try {
        connection = database.getConnection();
        statement = connection.prepareStatement("SELECT id, username, email, age FROM user");
        resultSet = statement.executeQuery();
        while (resultSet.next()) {
            users.add(new User(
                resultSet.getLong("id"),
                resultSet.getString("username"),
                resultSet.getString("email"),
                resultSet.getInteger("age")));
        }
    } finally {
        close(resultSet, statement, connection);
    }

    return users;
}
于 2012-12-15T20:57:56.700 回答