Cerrar correctamente las instancias de Connection
, Statement
, ResultSet
y manejar bien todos los errores que pueden resultar usando el patrón típico del try-catch-finally
no es fácil. Y si lo logras, el código no es elegante.
Por ejemplo, aún cuando corrijas tu problema immediato de mover la definición de las variables afuera del try
, el código aún tiene por lo menos un defecto de diseño.
Digamos que el código lo corriges de esta manera (y de paso, leyendo los comentarios bajo la respuesta de @Dev Joel, hay que aclarar que sí es necesario inicializar las variables a algún valor como null
. Sin eso, el compilador Java no puede garantizar que las variables tengan algún valor asignado cuando son usadas en el bloque finally
, y por eso se queja. Y de hecho, este punto está ligado al defecto que voy a demostrar):
String showTableSQL = "SELECT * FROM llamadas";
// inicializar las variables sí es necesario aquí.
Connection conn = null;
Statement st = null;
ResultSet rs = null;
try {
conn = conexion.getConnection(); // <-- ¿qué pasa si un error sucede aquí?
st = conn.createStatement();
rs = st.executeQuery(showTableSQL);
// ...
} catch (SQLException errorSQL) {
errorSQL.printStackTrace();
}
finally { // Cerramos las conexiones, en orden inverso a su apertura
try { rs.close(); } catch (Exception errorRS) { errorRS.printStackTrace(); }
try { st.close(); } catch (Exception errorST) { errorST.printStackTrace(); }
try { conn.close(); } catch (Exception errorCONN) { errorCONN.printStackTrace(); }
}
Ahora imaginemos que algún error sucede en este sentencia:
conn = conexion.getConnection();
Eso causa que st
y rs
queden con valores null
, y ahora, cuando el bloque finally
se ejecuta, tanto rs.close();
como st.close()
van a lanzar un NullPointerException
que se pudiera haber evitado.
De modo que la forma correcta de manejar el bloque finally
es verificando si las variables están a null
antes de intentar cerrarlas:
finally { // Cerramos las conexiones, en orden inverso a su apertura
try { if (rs != null) rs.close(); } catch (Exception errorRS) { errorRS.printStackTrace(); }
try { if (st != null) st.close(); } catch (Exception errorST) { errorST.printStackTrace(); }
try { if (conn != null) conn.close(); } catch (Exception errorCONN) { errorCONN.printStackTrace(); }
}
Una mejor alternativa
Lo anterior simplemente ilustra que es fácil cometer errores con este tipo de código debido a su complejidad. Y aún si logras hacerlo bien, la complejidad oscurece el verdadero propósito del código.
Por esta razón, a partir de Java 7, hay una mejor manera de manejar este tipo de situaciones, usando try-with-resources
. Usando esta técnica, nota cómo se puede simplificar tu código y cuán claro queda:
String showTableSQL = "SELECT * FROM llamadas";
try (Connection conn = conexion.getConnection();
Statement st = conn.createStatement();
ResultSet rs = st.executeQuery(showTableSQL)) {
// ...
} catch (SQLException e) {
e.printStackTrace();
}
Si lees la documentación, verás que esta estructura de código simplificada automáticamente maneja los aspectos siguientes:
- Todos los
.close()
se ejecutan automáticamente al final del bloque try
y en el orden inverso a la apertura de las diferentes instancias.
- Si ocurre algún error con uno o más de los
.close()
, esas excepciones no reemplazan la excepción principal, si la hay.
- Relacionado al punto anterior, si ocurre más de un error, sea en el cuerpo del
try
o con los .close()
, e.printStackTrace()
incluye la información de todos los errores automáticamente.
try { //todo} catch (Exception e) { e.printStackTrace(); }