Sonarqube squid: S2095 fałszywie dodatni

Question

W naszej bazie kodu otrzymujemy Sonar informuje o naruszenie przepisów kalmarów: S2095 na kodzie jak poniżej:

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); 
    } 

z Utilities.close realizowane jako

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

Czy istnieje sposób na uniknięcie fałszywych alarmów?

Answer 1

Krótka odpowiedź, nie ma sposobu, aby ich uniknąć na chwilę.

Dłuższa odpowiedź: Zwykle przekazanie otwartej wartości do metody powinno oznaczać ją jako zamkniętą, aby uniknąć fałszywych trafień. Powinieneś sprecyzować wersję sonaru java, której używasz.

Ta reguła opiera się na symbolicznym mechanizmie wykonawczym i jest ograniczona do granic metody i jako taka nie ma możliwości określenia na razie, że wywołanie tej metody narzędzia z pewnością zamknie otwarty zasób.

Pamiętaj jednak, że zespół sonar java pracuje nad tym, aby ten limit zniknął.

Answer 2

Jeśli używasz środowiska Java 7+, istnieje prosty sposób użycia funkcji "wypróbuj za pomocą zasobów", która może zamknąć sam zasób i nie musisz już o niego dbać. Zobacz spróbować (PreparedStatement ps = connection.prepareStatement (DML)), 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; 
}