Sonarqube calamaro: S2095 falso positivo

Question

Nella nostra base di codice otteniamo Sonar riporta violazione per la regola del calamaro: S2095 sul codice come il seguente:

PreparedStatement ps = null; 
    try { 
     ps = connection.prepareStatement(DML); 
     ps.setString(1, externalDeviceId); 
     ps.setInt(2, internalDeviceId); 
     ps.execute(); 
     return ps.getUpdateCount() > 0; 
    } finally { 
     Utilities.close(ps); 
    } 

con Utilities.close implementato come

public static final void close(final AutoCloseable ac) { 
     if(ac != null) { 
      try { 
       ac.close(); 
       } catch(Exception e) { 
      } 
     } 
    } 

Is c'è un modo per evitare questi falsi positivi?

Answer 1

Risposta breve, non c'è modo di evitare quelli per il momento.

Risposta più lunga: Normalmente, passare un valore aperto a un metodo dovrebbe contrassegnarlo come chiuso per evitare falsi positivi. Dovresti specificare la versione del plug-in java di sonar che stai utilizzando.

Questa regola si basa sul motore di esecuzione simbolica ed è limitata ai limiti di un metodo e come tale, non è possibile determinare per il momento che una chiamata a questo metodo di utilità chiuderà sicuramente la risorsa aperta.

Si noti tuttavia che il team di java sonar sta lavorando per rendere questo limite andare via.

Answer 2

Se si utilizza Java 7+, c'è un modo molto semplice per utilizzare le risorse try-with che sono in grado di chiudere la risorsa stessa e non è più necessario occuparsene. See visita (PreparedStatement ps = connection.prepareStatement (DML)), un tutorial: https://docs.oracle.com/javase/tutorial/essential/exceptions/tryResourceClose.html

try (PreparedStatement ps = connection.prepareStatement(DML)) { 
    ps.setString(1, externalDeviceId); 
    ps.setInt(2, internalDeviceId); 
    ps.execute(); 
    return ps.getUpdateCount() > 0; 
}