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

但是有问题。蠢货给我一个错误，说：

Load of known null value in 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(); //
}

Findbugs错误＆＃34;已知空值的加载＆＃34; SQL连接

1 个答案: