SonarLint仍然显示阻止程序错误，即使我们处理了该语句，连接在finally block中关闭

Question

SonarLint显示以下错误：

1. '使用try with resources或在“finally”子句中关闭此“语句”。'
2. '使用try with resources或在“finally”子句中关闭此“连接”。'<即使我们关闭了finally block中的语句stmt，Connection con，阻塞器也会出错

请查找示例代码。

public String getProductNumber() throws BusinessDelegateException {

        String productNo = null;
        Connection con = null;
        Statement stmt = null;
        ResultSet   rs  = null;
        String query  = //some query
        try {
            DataSource ds = getDataSource();
            con = ds.getConnection();
            stmt = con.createStatement();
            rs   = stmt.executeQuery(query);
            productNo =.......
            ....................
        }catch (Exception e) {
            String errorMsg = "Error occured in getProductNumber()";
            throw new BusinessDelegateException(errorMsg, e);
        }finally{
            try {
                if(rs != null)
                    rs.close();
                if (stmt != null)
                    stmt.close();
                if (con != null)
                    con.close();
            } catch (SQLException e) {
                e.printStackTrace();
            }
        }

        return productNo;
    }

我们可以通过以下方式修改finally块来解决这个问题。但它似乎仍然像是接球拦网的重复。我们还有别的办法解决这个问题吗？

finally{
        try {
            if(rs != null)
                rs.close();
        } catch (SQLException e) {
            e.printStackTrace();
        }
        try {
            if (stmt != null)
                stmt.close();
        } catch (SQLException e) {
            e.printStackTrace();
        }
        try {
            if (con != null)
                con.close();
        } catch (SQLException e) {
            e.printStackTrace();
        }
    }

Answer 1

使用Try with资源。这是最好的选择。

Answer 2

如果不尝试使用资源，您只能通过使用可重用的方法来改进代码，请调用main方法：

closeResources(rs, stmt, con);

它将为每个资源调用不同的方法，例如语句：

 public void closeResource(Statement stmt) {
    if (stmt != null) {
        stmt.close();
    } catch (SQLException e) {
        e.printStackTrace();
    }

顺便说一句，你最好使用logger而不是e.printStackTrace（）

对于完整的解决方案，您可以检查在数组中添加资源并在循环中关闭它们的广泛示例：

for (Closeable resource : resources) {
   try {
     resource.close();