使用JDBC资源的正确方法

Question

即使在Java 7中，我也经常看到类似的代码

    Connection c = null;
    PreparedStatement s = null;
    try {
        c = dataSource.getConnection();

        String sql = "insert into table (column1, column2) values (?, ?);
        s = c.prepareStatement(sql);
        int i = 1;
        s.setString(i++, "111");
        s.setString(i++, "222");

        s.execute();

    } catch (SQLException e) {
        log.error("failed to insert", e);
    } finally {
        DbUtils.close(s, c); // closes s and c with no questions
    }

但是根据规范，当关闭连接时，将释放所有语句和结果集。

我听说JDBC驱动程序有可能不遵循JDBC API的规则，并且旧方法更好。 我的问题是听取对此的意见。 代码看起来更好，但是如果很危险怎么办？ 在我看来，最优选的方法是在此处使用try-with-resources 。 它足够安全吗？

Answer 1

资源是必须在程序完成后关闭的对象。因此，您不必在finally块中手动关闭该对象。

 String sql = "insert into table (column1, column2) values (?, ?);
    try(Connection  c = dataSource.getConnection();
    PreparedStatement s = c.prepareStatement(sql);) {


        int i = 1;
        s.setString(i++, "111");
        s.setString(i++, "222");

        s.execute();

    } catch (SQLException e) {
        log.error("Failed to insert transaction", e);
    } 

Answer 2

您应该始终释放/关闭您使用的资源，例如ResultSet，Statement，Connection。 那应该在finally块中完成。 您在此处粘贴的代码段看起来还可以（只要在QuietDb中关闭了s和c）。

Answer 3

虽然关闭Connection将关闭与之关联的所有Statement ，但是在大多数数据库中，对于一个Connection可以同时打开多少Statement ，存在一个上限。

如果不关闭Statement ，则会增加出现ORA-01000: maximum open cursors exceeded类的错误的可能性ORA-01000: maximum open cursors exceeded 。

如果关闭Statements ， 它们也将关闭其ResultSet ，因此您不必同时关闭两者。

Answer 4

使用try-with-resources可以将您的代码重写为：

try (
    Connection c = dataSource.getConnection();
    PreparedStatement s = c.prepareStatement("insert into table (column1, column2) values (?, ?)");
) {
    int i = 1;
    s.setString(i++, "111");
    s.setString(i++, "222");

    s.execute();
} catch (SQLException e) {
    log.error("failed to insert", e);
}

这样可以保证语句和连接都被关闭（即使关闭该语句也会引发Exception！）。 唯一的区别可能是关闭时可能会抛出异常，因为我不知道您的DbUtils.close吞下了异常。

通常，try-with-resources提供了更简洁的代码，并且可以更好地确保资源以正确的顺序正确关闭，而无需花费太多时间。