Sonar жалуется на регистрацию и перестройку исключения

Question

У меня есть следующий фрагмент кода в моей программе, и я запускаю SonarQube 5 для проверки качества кода после его интеграции с Maven.

Однако, Сонар жалуется, что я должен Либо регистрировать или отменять это исключение.

Что мне здесь не хватает? Я уже не регистрирую исключение?

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

Вы должны сделать это таким образом:

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 не должно беспокоить вас больше

Answer 2

Если вы считаете, что SQLException может быть проигнорировано, то вы можете добавить его в список исключений для кальмара: правило S1166.

1. Идём в Правило-> Поиск кальмара: S1166.
2. Редактировать исключения в профиле качества.
3. Добавить SQLException в список.

Answer 3

Я наткнулся на эту же проблему. Я не уверен на 100%, если я полностью прав в данный момент, но в основном вы должны реконструировать или зарегистрировать полное исключение. В то время как e.getMessage() просто дает вам подробное сообщение, но не моментальный снимок стека выполнения.

От Oracle docs (Throwable):

Throwable содержит снимки стеки выполнения своей нити в то время он был создан. Он также может содержать строку сообщения, которая дает больше информации об ошибке. Со временем броски могут сдерживать распространение других комбайнов. Наконец, вбрасываемый может также содержать причину: еще один бросок, который заставлял эту брошюру быть сконструированной. Запись этой причинной информации называется связующим средством исключения, поскольку сама причина может иметь причину и т. Д., Что приводит к «цепочке» исключений, каждая из которых вызвана другой.

Это означает, что решение, предоставляемое abarre works, потому что весь объект исключения (sqle) передается в регистратор.

Надеюсь, это поможет. Cheers.

Answer 4

Что сонар просит вас сделать, это сохранить весь объект исключения. Вы можете использовать что-то вроде:

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