标签：sonarqube code-analysis findbugs java

下面是我得到声纳违反的代码

public static Connection getConnection(
        String db)
        throws SQLException {

        Connection connection = null;

        try {
            Driver dbRriver = (Driver) Class.forName(driver).newInstance();
            Properties props = new Properties();
            props.put("user", PropertyLoader.get("user123"));
            props.put("password", PropertyLoader.get("pass"));
            connection = dbRriver
                    .connect(PropertyLoader.get("URL"), props);
        } catch (Exception e) {
            **if(connection!= null){
            connection.close();**
            }
            LOGGER.error(IN_EXCEPTION, e);
        }
        return connection;
    }

在关闭连接之前,需要先检查连接是否为空,但是声纳给出的冲突为“ Dodgy-值已知为冗余的冗余nullcheck”.请提出建议,因为我可以在不删除null检查的情况下避免这种违反情况.

解决方法:

因为无论哪种方式,如果最终陷入困境,连接将始终为空.有几个地方可以有例外,但是所有这些都在分配连接之前= dbRriver.connect(PropertyLoader.get(“ URL”),props);
因此,catch永远为空.
是的,您需要按照@Stultuske的建议将这些代码最终放入

标签：sonarqube,code-analysis,findbugs,java
来源： https://codeday.me/bug/20191027/1947792.html

本站声明： 1. iCode9 技术分享网（下文简称本站）提供的所有内容，仅供技术学习、探讨和分享；
2. 关于本站的所有留言、评论、转载及引用，纯属内容发起人的个人观点，与本站观点和立场无关；
3. 关于本站的所有言论和文字，纯属内容发起人的个人观点，与本站观点和立场无关；
4. 本站文章均是网友提供，不完全保证技术分享内容的完整性、准确性、时效性、风险性和版权归属；如您发现该文章侵犯了您的权益，可联系我们第一时间进行删除；
5. 本站为非盈利性的个人网站，所有内容不会用来进行牟利，也不会利用任何形式的广告来间接获益，纯粹是为了广大技术爱好者提供技术内容和技术思想的分享性交流网站。