SonarQube检测finally块中conn存空指针风险，求解决方案

仅做非空检查够不够？ #

答案是够的，这是最直接解决SonarQube提示的方式。

conn为空的情况只会出现在连接创建环节抛异常的时候——比如URL.openConnection()抛出MalformedURLException或者IOException，导致conn变量根本没被赋值。只要在finally块里先判断if (conn != null)，再调用disconnect()，就能彻底避免空指针异常。

举个标准写法：

HttpsURLConnection conn = null;
try {
    URL url = new URL("https://example.com");
    conn = (HttpsURLConnection) url.openConnection();
    // 配置请求方法、超时、请求头等参数
    conn.setRequestMethod("GET");
    conn.setConnectTimeout(5000);
    // 执行请求、处理响应...
} catch (IOException e) {
    // 这里处理异常，比如打日志、抛业务异常
} finally {
    if (conn != null) {
        conn.disconnect();
    }
}

这种写法下，SonarQube的空指针提示会直接消失，逻辑也完全没问题。

还有哪些更顺手的处理方式？ #

除了基础的非空检查，还有几种更省心的写法：

1. 用try-with-resources（强烈推荐） #

从Java 7开始，HttpsURLConnection间接支持AutoCloseable接口，用try-with-resources语法可以自动帮你关闭连接，根本不用写finally块：

try (HttpsURLConnection conn = (HttpsURLConnection) new URL("https://example.com").openConnection()) {
    // 配置参数、处理请求响应的逻辑和之前一样
    conn.setRequestMethod("GET");
    // ... 其他操作
} catch (IOException e) {
    // 异常处理
}

代码块结束后，JVM会自动调用conn.close()，而这个方法内部已经帮你处理了disconnect()的逻辑，既简洁又不会有空指针风险，是现在最规范的写法。

2. 封装通用工具方法 #

如果项目里很多地方都要处理HTTP连接关闭，可以写个工具方法统一处理空指针：

public static void safeDisconnect(HttpURLConnection conn) {
    if (conn != null) {
        conn.disconnect();
    }
}

之后在finally块里直接调用就行：

finally {
    safeDisconnect(conn);
}

这样能减少重复代码，也方便统一修改关闭逻辑。

3. 用Optional包装（有点过度，看场景） #

如果你的项目习惯用Optional处理空值，也可以这么写：

finally {
    Optional.ofNullable(conn).ifPresent(HttpsURLConnection::disconnect);
}

效果和非空检查一样，但对于这种简单场景来说，有点没必要，不如直接写if判断直观。

内容的提问来源于stack exchange，提问作者Mdhar9e