不关闭我的JDBC PreparedStatements会导致内存泄漏吗?

我正在使用java.sql PreparedStatements,我想知道以下内容:

在Java中是Pass-by-Value,Dammit! ,以下是Java的Pass-By约定的示例:

public void foo(Dog d) { d = new Dog("Fifi"); // creating the "Fifi" dog } Dog aDog = new Dog("Max"); // creating the "Max" dog // at this point, aDog points to the "Max" dog foo(aDog); // aDog still points to the "Max" dog 

在我的代码中,这出现如下(半Java伪代码):

 public void method() { PreparedStatement pstmt = null; ResultSet rs = null; try { rs = executeStatement(sql-string, pstmt, conn, vars...); } catch (....) { /* error-handling */ } /// do stuff with the data rs.close(); } 

其中executeStatement是(类似于)以下内容:

 ResultSet executeStatement(String sql, PreparedStatement pstmt, Connection conn, Object[] vars...) { pstmt = conn.prepareStatement(sql); /// set pstmt variables... ResultSet rs = pstmt.execute(); return rs; } 

根据我对Java的传递约定的理解,在主代码中对pstmt做任何事都没用,因为即使在调用executeStatement之后它仍然是null。 但是,因为关闭PreparedStatement也会关闭ResultSet,所以我知道在处理ResultSet时, executeStatement创建的PreparedStatement没有关闭。

这是否意味着此处存在内存泄漏? (我对内存泄漏的理解以及它们如何被诊断/修复是充其量的)。 有没有什么方法可以以不同的方式构造它以避免泄漏,但是继续使用可以执行SQL字符串并以抽象方式返回ResultSet的方法?

我没有看到内存泄漏。 我发现的唯一问题是你没有在finally块中关闭resultSet。 因此,如果抛出exception,则rs.close()将不会被执行。

正如Andrei评论的那样,关闭结果集也将关闭基础声明。 我不确定你在哪里关闭连接,但这也应该发生在finally块中。

首先,传统意义上的内存泄漏,你已经分配了不能被引用的数据,在Java中不存在,因为只有未被引用的东西被收集。 但是,在这种情况下,你不是在寻找内存泄漏,而是资源泄漏(这在Java中更是一个问题):最终将收集PreparedStatement的内存并释放内存,因为它没有被引用但是,在执行您的方法之后,语句保留的资源应该更早地释放,而不仅仅是垃圾收集器运行一次。

您可以做的是编写一个包含StatementResultSet作为成员的类并返回此类,如下所示:

 class ResultSetStatementPair { ResultSetStatementPair(ResultSet rs, Statement stmt) { this.rs = rs; this.stmt = stmt; } ResultSet rs; Statement stmt; } ResultSetStatementPair executeStatement(String sql, Connection conn, Object[] vars...) { PreparedStatement pstmt = conn.prepareStatement(sql); ResultSet rs = pstmt.execute(); return new ResultSetStatementPair(rs, pstmt); } public void method() { Statement pstmt = null; ResultSet rs = null; try { ResultSetStatementPair pair = executeStatement(sql-string, pstmt, conn, vars...); rs = pair.rs; stmt = pair.stmt; // do stuff with the data } catch (....) { /* error-handling */ } finally { if(rs != null) rs.close(); if(stmt != null) stmt.close(); } } 

还要注意我添加了一个finally,并将do stuff移动到try块中。