Sonar che si lamenta di registrare e rilanciare l'eccezione

Question

Ho il seguente codice nel mio programma e sto eseguendo SonarQube 5 per la verifica della qualità del codice su di esso dopo averlo integrato con Maven.

Tuttavia, Sonar si lamenta che io dovrei o registrare o riscrivere questa eccezione.

Cosa mi manca qui? Non sto già registrando l'eccezione?

private boolean authenticate(User user) { 
     boolean validUser = false; 
     int validUserCount = 0; 
     try { 
      DataSource dataSource = (DataSource) getServletContext().getAttribute("dataSource"); 
      validUserCount = new MasterDao(dataSource).getValidUserCount(user); 
     } catch (SQLException sqle) { 
      LOG.error("Exception while validating user credentials for user with username: " + user.getUsername() + " and pwd:" + user.getPwd()); 
      LOG.error(sqle.getMessage()); 
     } 
     if (validUserCount == 1) { 
      validUser = true; 
     } 
     return validUser; 
    } 

Answer 1

si dovrebbe fare in questo modo:

try { 
    DataSource dataSource = (DataSource) getServletContext().getAttribute("dataSource"); 
    validUserCount = new MasterDao(dataSource).getValidUserCount(user); 
} catch (SQLException sqle) { 
    LOG.error("Exception while validating user credentials for user with username: " + 
      user.getUsername() + " and pwd:" + user.getPwd(), sqle); 
} 

Sonar non dovrebbe disturbare più

Answer 2

Se credi che SQLException può essere ignorato, quindi è possibile aggiungere alla lista di eccezioni per calamaro: regola S1166.

1. Vai a Regola-> Cerca calamaro: S1166.
2. Modificare le eccezioni nel profilo di qualità.
3. Aggiungi SQLException all'elenco.

Answer 3

Mi sono imbattuto nello stesso problema. Non sono sicuro al 100% se ho completamente ragione a questo punto, ma in pratica dovresti rilanciare o registrare l'Eccezione completa. Mentre lo e.getMessage() ti fornisce solo il messaggio dettagliato ma non l'istantanea dello stack di esecuzione.

Dal Oracle docs (Throwable):

Un throwable contiene un'istantanea della pila esecuzione del filo al momento della sua creazione. Può anche contenere una stringa di messaggi che fornisce ulteriori informazioni sull'errore. Col passare del tempo, un lancio può sopprimere altri oggetti da lanciare da propagare. Infine, il lancio può contenere anche una causa: un altro lancio che ha causato la costruzione di questo proiettile. La registrazione di questa informazione causale viene definita come la funzione di eccezione concatenata, poiché la causa può, a sua volta, avere una causa e così via, portando a una "catena" di eccezioni, ciascuna causata da un'altra.

Ciò significa che la soluzione fornita da abarre funziona, poiché l'intero oggetto di eccezione (sqle) viene passato al registratore.

Spero che aiuti. Saluti.

Answer 4

Quello che l'ecoscandaglio ti chiede di fare è di mantenere l'intero oggetto di eccezione. Si può usare qualcosa di simile a:

try { 
     ...   
    } catch (Exception e) { 
     logger.error("Error", e); 
    }