Haruskah Hasil dan Pernyataan JDBC ditutup secara terpisah meskipun Koneksi ditutup sesudahnya?

Dikatakan sebagai kebiasaan yang baik untuk menutup semua sumber daya JDBC setelah penggunaan. Tetapi jika saya memiliki kode berikut, apakah perlu untuk menutup Resultset dan Pernyataan?

Connection conn = null;
PreparedStatement stmt = null;
ResultSet rs = null;
try {
    conn = // Retrieve connection
    stmt = conn.prepareStatement(// Some SQL);
    rs = stmt.executeQuery();
} catch(Exception e) {
    // Error Handling
} finally {
    try { if (rs != null) rs.close(); } catch (Exception e) {};
    try { if (stmt != null) stmt.close(); } catch (Exception e) {};
    try { if (conn != null) conn.close(); } catch (Exception e) {};
}

Pertanyaannya adalah apakah penutupan koneksi berfungsi atau apakah ada sumber daya yang digunakan.

Apa yang telah Anda lakukan adalah latihan yang sempurna dan sangat baik.

Alasan saya mengatakan ini adalah praktik yang baik ... Misalnya, jika karena alasan tertentu Anda menggunakan tipe penyatuan basis data "primitif" dan Anda menelepon connection.close(), koneksi akan dikembalikan ke kolam dan ResultSet/ Statementtidak akan pernah ditutup dan kemudian Anda akan mengalami banyak masalah baru yang berbeda!

Jadi Anda tidak bisa selalu mengandalkan connection.close()pembersihan.

Saya harap ini membantu :)

Java 1.7 membuat hidup kita lebih mudah berkat pernyataan coba-dengan-sumber daya .

try (Connection connection = dataSource.getConnection();
    Statement statement = connection.createStatement()) {
    try (ResultSet resultSet = statement.executeQuery("some query")) {
        // Do stuff with the result set.
    }
    try (ResultSet resultSet = statement.executeQuery("some query")) {
        // Do more stuff with the second result set.
    }
}

Sintaks ini cukup singkat dan elegan. Dan connectionmemang akan ditutup bahkan ketika yang statementtidak bisa dibuat.

Dari javadocs :

Ketika suatu Statementobjek ditutup, ResultSetobjek saat ini , jika ada, juga ditutup.

Namun, javadocs tidak begitu jelas tentang apakah Statementdan ResultSetditutup ketika Anda menutup yang mendasarinya Connection. Mereka hanya menyatakan bahwa menutup Koneksi:

Merilis Connectionbasis data objek dan sumber daya JDBC ini segera alih-alih menunggu mereka dirilis secara otomatis.

Menurut pendapat saya, selalu ditutup secara eksplisit ResultSets, Statementsdan Connectionsketika Anda selesai dengan mereka sebagai implementasi closedapat bervariasi antara driver database.

Anda dapat menghemat banyak kode boiler-plate dengan menggunakan metode seperti closeQuietlypada DBUtils dari Apache.

Saya sekarang menggunakan Oracle dengan Java. Di sini sudut pandang saya:

Anda harus menutup ResultSetdan Statementsecara eksplisit karena Oracle memiliki masalah sebelumnya dengan menjaga kursor tetap terbuka bahkan setelah menutup koneksi. Jika Anda tidak menutup ResultSet(kursor) itu akan menimbulkan kesalahan seperti kursor terbuka maksimum terlampaui .

Saya pikir Anda mungkin mengalami masalah yang sama dengan database lain yang Anda gunakan.

Ini adalah tutorial Tutup ResultSet ketika selesai :

Tutup ResultSet ketika selesai

Tutup ResultSetobjek segera setelah Anda selesai bekerja dengan ResultSetobjek meskipun Statementobjek menutup ResultSetobjek secara implisit ketika menutup, menutup ResultSetsecara eksplisit memberi kesempatan kepada pengumpul sampah untuk mengingat memori sedini mungkin karena ResultSetobjek dapat menempati banyak memori tergantung pada permintaan.

ResultSet.close();

Jika Anda menginginkan kode yang lebih ringkas, saya sarankan menggunakan Apache Commons DbUtils . Pada kasus ini:

Connection conn = null;
PreparedStatement stmt = null;
ResultSet rs = null;
try {
    conn = // Retrieve connection
    stmt = conn.prepareStatement(// Some SQL);
    rs = stmt.executeQuery();
} catch(Exception e) {
    // Error Handling
} finally {
    DbUtils.closeQuietly(rs);
    DbUtils.closeQuietly(stmt);
    DbUtils.closeQuietly(conn);
}

Metode yang benar dan aman untuk menutup sumber daya yang terkait dengan JDBC ini (diambil dari Cara Menutup Sumber Daya JDBC dengan Benar - Setiap Saat ):

Connection connection = dataSource.getConnection();
try {
    Statement statement = connection.createStatement();

    try {
        ResultSet resultSet = statement.executeQuery("some query");

        try {
            // Do stuff with the result set.
        } finally {
            resultSet.close();
        }
    } finally {
        statement.close();
    }
} finally {
    connection.close();
}

Tidak masalah apakah Connectiondapat dikumpulkan atau tidak. Bahkan koneksi yang dapat dikumpulkan harus dibersihkan sebelum kembali ke kolam.

"Bersihkan" biasanya berarti menutup hasil & mengembalikan semua transaksi yang tertunda tetapi tidak menutup koneksi. Jika tidak, pooling kehilangan akal sehatnya.

Tidak, Anda tidak perlu menutup apa pun TETAPI koneksi. Per spesifikasi JDBC yang menutup objek yang lebih tinggi akan secara otomatis menutup objek yang lebih rendah. Penutup Connectionakan menutup Statementkoneksi apa pun yang telah dibuat. Menutup apa pun Statementakan menutup semua ResultSetyang dibuat oleh itu Statement. Tidak masalah apakah Connectiondapat dikumpulkan atau tidak. Bahkan koneksi yang dapat dikumpulkan harus dibersihkan sebelum kembali ke kolam.

Tentu saja Anda mungkin memiliki loop panjang bersarang pada Connectionpembuatan banyak pernyataan, lalu menutupnya adalah tepat. Saya hampir tidak pernah menutup ResultSet, sepertinya berlebihan ketika menutup Statementatau ConnectionAKAN menutupnya.

Saya membuat Metode berikut untuk membuat One Liner yang dapat digunakan kembali:

public void oneMethodToCloseThemAll(ResultSet resultSet, Statement statement, Connection connection) {
    if (resultSet != null) {
        try {
            if (!resultSet.isClosed()) {
                resultSet.close();
            }
        } catch (SQLException e) {
            e.printStackTrace();
        }
    }
    if (statement != null) {
        try {
            if (!statement.isClosed()) {
                statement.close();
            }
        } catch (SQLException e) {
            e.printStackTrace();
        }
    }

    if (connection != null) {
        try {
            if (!connection.isClosed()) {
                connection.close();
            }
        } catch (SQLException e) {
            e.printStackTrace();
        }
    }
}

Saya menggunakan Kode ini dalam Kelas induk yang diwarisi ke semua kelas saya yang mengirim DB Queries. Saya dapat menggunakan Oneliner pada semua Pertanyaan, bahkan jika saya tidak memiliki resultSet. Metode ini menangani penutupan ResultSet, Pernyataan, Koneksi dalam urutan yang benar. Inilah yang akhirnya tampak seperti blok saya.

finally {
    oneMethodToCloseThemAll(resultSet, preStatement, sqlConnection);
}

Sejauh yang saya ingat, di JDBC saat ini, Resultset dan pernyataan mengimplementasikan antarmuka AutoCloseable. Itu berarti mereka ditutup secara otomatis saat dihancurkan atau keluar dari ruang lingkup.

Beberapa fungsi kenyamanan:

public static void silentCloseResultSets(Statement st) {
    try {
        while (!(!st.getMoreResults() && (st.getUpdateCount() == -1))) {}
    } catch (SQLException ignore) {}
}
public static void silentCloseResultSets(Statement ...statements) {
    for (Statement st: statements) silentCloseResultSets(st);
}

Dengan Java 6 form saya pikir lebih baik untuk memeriksa apakah sudah ditutup atau tidak sebelum ditutup (misalnya jika beberapa pooler koneksi membatalkan koneksi di utas lainnya) - misalnya beberapa masalah jaringan - pernyataan dan status resultset dapat ditutup. (tidak sering terjadi, tapi saya punya masalah dengan Oracle dan DBCP). Pola saya untuk itu (dalam sintaks Java yang lebih lama) adalah:

try {
    //...   
    return resp;
} finally {
    if (rs != null && !rs.isClosed()) {
        try {
            rs.close();
        } catch (Exception e2) { 
            log.warn("Cannot close resultset: " + e2.getMessage());
        }
    }
    if (stmt != null && !stmt.isClosed()) {
        try {
            stmt.close();
        } catch (Exception e2) {
            log.warn("Cannot close statement " + e2.getMessage()); 
        }
    }
    if (con != null && !conn.isClosed()) {
        try {
            con.close();
        } catch (Exception e2) {
            log.warn("Cannot close connection: " + e2.getMessage());
        }
    }
}

Secara teori itu tidak 100% sempurna karena antara memeriksa keadaan dekat dan penutupan itu sendiri ada sedikit ruang untuk perubahan keadaan. Dalam kasus terburuk Anda akan mendapatkan peringatan lama. - tetapi lebih kecil dari kemungkinan perubahan negara dalam permintaan jangka panjang. Kami menggunakan pola ini dalam produksi dengan muatan "rata-rata" (150 pengguna simultan) dan kami tidak punya masalah dengan itu - jadi jangan pernah melihat pesan peringatan itu.