java - Findbugs错误 "Load of known null value"SQL连接

Question

我有 DAO 类，其中包含获取和发送数据的方法。

我正在捕获 SQL 请求中的异常，因此我需要在 try 括号之外声明连接变量。

每个方法看起来都是这样的:

public Role getRole(int roleId) {
    Connection connection = null;
    ResultSet rs = null;
    PreparedStatement statement = null;
    Role role = null;

    try {
        connection = dataSource.getConnection();
        statement = connection.prepareStatement("select ROLE_ID, ROLE_TEXT from ROLES WHERE ROLE_ID = :1");
        statement.setInt(1, roleId);
        rs = statement.executeQuery();
        rs.next();
        role = roleMapper.mapRow(rs, 1);
    } catch (SQLException e) {
    } finally {
        JdbcUtils.closeResultSet(rs);
        JdbcUtils.closeStatement(statement);
        JdbcUtils.closeConnection(connection);
        return role;
    }
}

但是有问题。 Finbugs 给我一个错误，说:

在 DAO.getRole 中加载已知空值

和

may fail to clean up java.sql.Statement

那么我应该怎样做才能避免这种情况呢？

Answer 1

getRole 可以返回 null。此外:

if (rs.next()) {
    role = roleMapper.mapRow(rs, 1);
}

我更喜欢另一种表示法。不幸的是，错误解决方案包括让 getRole 抛出异常(最好)或让返回 Optional<Role>

//public Role getRole(int roleId) throws SQLException {
public Optional<Role> getRole(int roleId) {
    try (Connection connection = dataSource.getConnection();
            PreparedStatement statement =
                connection.prepareStatement(
                "select ROLE_ID, ROLE_TEXT from ROLES WHERE ROLE_ID = :1")) {
        statement.setInt(1, roleId);
        try (ResultSet rs = statement.executeQuery()) {
            if (rs.next()) {
                return roleMapper.mapRow(rs, 1);
            }
        }
    } catch (SQLException e) { //
        Logger.getLogger(getClass().getName()).log(Level.SEVERE, "ID: " + roleId, e); //
    }
    return Optional.empty(); //
}