声纳抱怨日志记录和重新抛出异常

Question

我的程序中有下面的代码，在与Maven集成后，我正在运行SonarQube 5进行代码质量检查。

然而，Sonar抱怨说，我应该记录或者重播这个异常。

我错过了什么？我还没有记录异常吗？

 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

如果您认为可以安全地忽略SQLExcture，那么您可以将其添加到squid： S1166规则的异常列表中。

1. 去统治-

Answer 2

sonar要求您做的是持久化整个异常对象。您可以使用以下内容：

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

Answer 3

你应该这样做：

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

声纳应该不会再打扰你了