不关闭我的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
的内存并释放内存,因为它没有被引用但是,在执行您的方法之后,语句保留的资源应该更早地释放,而不仅仅是垃圾收集器运行一次。
您可以做的是编写一个包含Statement
和ResultSet
作为成员的类并返回此类,如下所示:
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块中。