In base a quale circostanza potrebbe fuoriuscire questa risorsa?

Question

Eclipse 4 dà un avvertimento che dice il stmt non può potenzialmente essere chiusi e causare una perdita di risorse:

class Test { 
    public void test() { 
     PreparedStatement stmt = null; 
     try { 
      stmt = HibernateSession.instance().connection().prepareStatement(""); 
     } catch (final SQLException e) { 
      e.printStackTrace(); 
     } finally { 
      if (stmt != null) 
       try { 
        stmt.close(); 
       } catch (final SQLException e) { 
        e.printStackTrace(); 
       } 
     } 
    } 
} 

In quale circostanza potrebbe succedere?

Answer 1

Immagino che la conclusione sia questa: si tratta di un bug di Eclipse?

Answer 2

Devi essere utilizzando meta di Java 7 con le risorse o di un blocco try-finally:

try(stmt = HibernateSession.instance().connection().prepareStatement("")) { 


} 

Questo avviso verrà generato sui tipi che ereditano da AutoCloseable non sono garantiti per essere chiuso. (O possibile Closeable, non ricordo quale).

Ora che vedo quello che chiedi, scrivi solo codice meno complicato.

Foo f = null; // don't do this, but it's what you're doing 
f = new Foo(); 

è quello che stai facendo, e vi trovarono una delle diverse situazioni in cui in realtà si deve pagare una penale per questo lavoro estranea.

Inoltre il tuo try/finally deve essere pulito. .close() non puoi lanciare, perché stai prendendo?

try { // don't do this 
    stmt.close(); 
} 
catch(SQLException exc) { 

} 

Dovrebbe generare un avviso Eclipse che ti dice che stai prendendo qualcosa che non getta. Potrebbe anche essere un errore di compilazione, non è sicuro, ma sembra che tu possa trarre vantaggio dal giocare con Eclipse> Preferenze> Compilatore e rivedere quali avvisi sono intelligenti. Se non capisci un avvertimento, google e vedere se sarebbe utile a voi, non solo saltare su di esso. (un po 'come hai fatto con questo).

Answer 3

Una perdita è possibile se viene generata un'eccezione quando si chiama stmt.close() nel blocco finally.

Answer 4

Il problema era che nel blocco finally si poteva verificare un'eccezione che impediva la chiusura di stmt.

Una soluzione è che si può sostituire tutto nel blocco finally con:

JDBCUtilities.close(stmt); 

Vedere la docs per JDBCUtilities.close. Come puoi vedere, non verrà generata alcuna eccezione utilizzando questo metodo di utilità in modo da non doverti preoccupare della perdita di risorse. Un ulteriore vantaggio è che il metodo di utilità gestisce la custodia null per stmt e quindi non è necessario codificarla autonomamente.

In realtà, è good practice per utilizzare JDBCUtilities.