Co powinienem najpierw zamknąć, PreparedStatement lub połączenie?

Question

Podczas korzystania z PreparedStatement w JDBC, należy najpierw zamknąć PreparedStatement lub Connection? Właśnie zobaczyłem próbkę kodu, w której Connection jest zamknięty, ale wydaje mi się bardziej logiczne, aby najpierw zamknąć PreparedStatement.

Czy istnieje standardowy, akceptowany sposób na zrobienie tego? Czy to ma znaczenie? Czy zamknięcie Connection powoduje również zamknięcie PreparedStatement, ponieważ PreparedStatement jest bezpośrednio związane z obiektem Connection?

Answer 1

Oświadczenie. Spodziewam się, aby zamknąć (w kolejności)

1. zestaw wyników
2. oświadczenie
3. połączenie

(i sprawdzić null po drodze!)

tj zamknij w w odwrotnej kolejności kolejność otwierania.

Jeśli użyjesz Spring JdbcTemplate (lub podobnej), to zaopiekuje się nim dla ciebie. Alternatywnie możesz użyć Apache Commons DbUtils i DbUtils.close() lub DbUtils.closeQuietly().

Answer 2

Następujące procedury należy zrobić (w kolejności)

• ResultSet
• PreparedStatement
• Connection.

Zaleca się również zamknięcie wszystkich obiektów związanych z JDBC w pobliżu, aby zagwarantować zamknięcie.

//Do the following when dealing with JDBC. This is how I've implemented my JDBC transactions through DAO.... 

Connection conn = null; 
PreparedStatement ps = null; 
ResultSet rs = null; 

try { 
    conn = .... 
    ps = conn.prepareStatement(...); 

    //Populate PreparedStatement 
    rs = ps.executeQuery(); 

} catch (/*All relevant exceptions such as SQLException*/Exception e) { 
    logger.error("Damn, stupid exception: " , e); 
} finally { 
if (rs != null) { 
      try { 
       rs.close(); 
       rs = null; 
      } catch (SQLException e) { 
       logger.error(e.getMessage(), e.fillInStackTrace()); 
      } 
     } 

     if (ps != null) { 
      try { 
       ps.close(); 
       ps = null; 
      } catch (SQLException e) { 
       logger.error(e.getMessage(), e.fillInStackTrace()); 
      } 
     } 

     try { 
      if (conn!= null && !conn.isClosed()){ 
       if (!conn.getAutoCommit()) { 
        conn.commit(); 
        conn.setAutoCommit(true); 
       } 
       conn.close(); 
       conn= null; 
      } 
     } catch (SQLException sqle) { 
      logger.error(sqle.getMessage(), sqle.fillInStackTrace()); 
     } 
} 

Widać mam sprawdzić czy moje przedmioty tracą połączenie, sprawdź pierwszy jeśli połączenie nie jest autocommited. Wiele osób nie sprawdza tego i zdaje sobie sprawę, że transakcja nie została przekazana do DB.