使用 try-with-resources 或在“finally”子句中关闭此“PreparedStatement”。在 Spring JDBC [重复]

Posted 2023-03-27

【中文标题】使用 try-with-resources 或在“finally”子句中关闭此“PreparedStatement”。在 Spring JDBC [重复]

【英文标题】：Use try-with-resources or close this "PreparedStatement" in a "finally" clause. In Spring JDBC [duplicate]

【发布时间】：2018-01-30 22:30:45

【问题描述】：

我从 Sonar 收到以下问题： SampleDao.java null 使用 try-with-resources 或在“finally”子句中关闭此“PreparedStatement”。

我正在使用带有以下代码的 Spring JDBC：

ResponseObject user = jdbcTemplate.query(new PreparedStatementCreator() 
        @Override
        public PreparedStatement createPreparedStatement(Connection con) throws SQLException 
            PreparedStatement ps = con.prepareStatement(MY_QUERY);
            ps.setLong(1, parameter);
            return ps;
        
    , new MyResultSetExtractor());

我的问题是我认为我不需要关闭连接，因为我有一个连接池，所以你认为我必须做什么，只是忽略声纳问题？或者还有什么我可以做的吗？

谢谢

【问题讨论】：

连接池通常会为您提供一个实现，在关闭时将连接释放回池中。遵循 Liskov 替换规则，您不应假设您的连接已实现，并且您应始终在其合同规定的情况下关闭它。

Spring JdbcTemplate 将确保 PreparedStatement 对象根据需要关闭，因此您需要告诉 SonarQube 不要警告您有关此实例的信息。见Turning Sonar off for certain code。

@sturcotte06 如果您的代码没有分配连接，那么您不应该关闭它，除非明确记录。由于这里 Spring 是分配连接的那个，因此应该关闭它的是 Spring（除非 spring 明确记录您需要关闭连接）。

@MarkRotteveel 是的，你是对的，我阅读了问题但没有检查代码。

【参考方案1】：

这似乎是一个问题，因为您正在创建一个匿名内部类，并且 sonar 期望在常规方法中创建的 prepare 语句被关闭。

如果您使用的是 java 8，请尝试用 labmda 表达式替换匿名内部类。

ResponseObject user = jdbcTemplate.query(coneection -> 
        PreparedStatement ps = con.prepareStatement(MY_QUERY);
        ps.setLong(1, parameter);
        return ps;
    
, new MyResultSetExtractor());

【讨论】：

为什么这有助于声纳代码分析？代码仍在创建PreparedStatement 对象而不关闭它。

另外，代码错误。太多 和 coneection != con。