使用IOUtils读取HttpServletRequest输入流报潜在资源泄露如何修复？

错误产生原因 #

1. 核心是Eclipse静态代码分析的局限性：你现有代码中两次调用了request.getInputStream()，Eclipse无法识别两次调用返回的是同一个Closeable流实例，会判定第一次调用获取的、传入IOUtils.toString()的流没有被显式关闭，因此触发资源泄露警告。
2. 补充场景的触发逻辑：当data变量在try块内声明且未被使用时，Eclipse会判定IOUtils.toString()这行是无效代码，直接跳过资源泄露检测，因此不会报错；一旦你使用了data，该行代码被判定为有效，检测逻辑就会触发警告。
3. 额外隐患：多次调用request.getInputStream()不符合Servlet规范要求，流只能被读取一次，部分容器实现中第二次调用会返回空流，可能导致业务逻辑异常。

修复方案 #

方案1：显式声明流变量统一管理（兼容低版本Java） #

仅调用一次request.getInputStream()，将返回的流赋值给局部变量，后续读取、关闭都操作同一个变量，Eclipse就能正确识别资源的关闭逻辑：

String data = "";
InputStream inputStream = null;
try {
    inputStream = request.getInputStream();
    data = IOUtils.toString(inputStream, "UTF-8");
    // 此处可正常使用data变量
} catch (IOException e) {
    // 自定义异常处理逻辑
} finally {
    IOUtils.closeQuietly(inputStream);
}

方案2：使用try-with-resources语法（推荐，Java 7+支持） #

Java 7及以上版本可以使用官方标准的自动资源管理语法，无需手动写finally关闭流，代码更简洁，也不会触发Eclipse警告：

String data = "";
try (InputStream inputStream = request.getInputStream()) {
    data = IOUtils.toString(inputStream, "UTF-8");
    // 此处可正常使用data变量
} catch (IOException e) {
    // 自定义异常处理逻辑
}

方案3：压制警告（不推荐） #

如果确认代码逻辑无问题，不想调整代码结构，可以在对应方法上添加@SuppressWarnings("resource")注解，直接压制Eclipse的资源泄露警告，该方案无法解决多次调用getInputStream()的业务隐患，仅适合临时处理。

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