Resultset ist nie geschlossen - Sonarqube Analyse

Question

In unserem Code haben wir die Ergebnissätze in Endblock geschlossen noch zeigt das Sonar, dass es nie geschlossen ist. Bitte helfen Sie. Hier haben wir eine Verbindung mit Spring Datenquellen utils erstellt und die gleichen haben die Verbindungen zum Pool freigegeben.

try { 
    con = DataSourceUtils.getConnection(dataSource); // connection to database using spring 
    stmt = con.createStatement(); 
    rs = stmt.executeQuery("<>"); 
    . 
    . 
} 
catch (Exception e) { 
} 
finally { 
    if (stmt != null && !stmt.isClosed()) { 
    stmt.close(); 
    } 
    if (rs != null && !rs.isClosed()) { 
    rs.close(); 
    } 
    if (con != null) { 
    DataSourceUtils.releaseConnection(con, dataSource); 
    } 
} 

Answer 1

sollten Sie verwenden die Try-with-Ressource-Anweisung, um Ihren Code Cleanup und richtigen Umgang mit Ressourcen sicherzustellen:

try (final Connection con = DataSourceUtils.getConnection(dataSource); // connection to database using spring 
    final Statement stmt = con.createStatement(); 
    final ResultSet rs = stmt.executeQuery("<>");) { 
    ... 
} catch (Exception e) { 
    // handle Exceptions here 
} 

Answer 2

Es ist möglich, für

stmt.close(); 

eine SQLException zu werfen. Wenn das passiert, wird

rs.close(); 

wird nie ausgeführt werden. Wie von anderen vorgeschlagen, sollten Sie die Verwendung von try with resource in Betracht ziehen.

Answer 3

Endlich fand ich den Grund, warum Sonar den Fehler wirft, obwohl wir die Ressourcen geschlossen haben.

Das Sonar erwartet, dass jede Ressource separat in einem separaten try catch geschlossen wird. Der Grund für sis ist, dass wenn andere Ressourcen geschlossen werden, andere möglicherweise offen bleiben.

Wie diese

finally{ 
try{ 
if(resultset!=null){ 
resultset.close(); 
} 
catch(SQLException e){ 
--- 
--- 
} 
if(connection!=null){ 
connection.close(); 
} 
catch(SQLException e){ 
--- 
--- 
} 
} 

Auch in dem obigen Code, fügen Sie bitte logger.error in Ausnahme eines weiteren Fehler von Sonar zu vermeiden !! ! :) logger.error("", e);