BufferedReader关闭流的问题

Question

线上代码，如何正确的关闭BufferedReader流。我用的JDK1.7
原来的代码如下：

public static String httpPostWithJson(String ecUrl, String params) {
        try {
            // 创建连接
            URL url = new URL(ecUrl);
            HttpURLConnection connection = (HttpURLConnection) url.openConnection();
            connection.setDoOutput(true);
            connection.setDoInput(true);
            connection.setRequestMethod("POST");
            connection.setUseCaches(false);
            connection.setInstanceFollowRedirects(true);
            connection.setRequestProperty("Content-Type", "application/x-www-form-urlencoded");
            connection.connect();

            // POST请求
            DataOutputStream out = new DataOutputStream(connection.getOutputStream());
            out.writeBytes(params);
            out.flush();
            out.close();

            // 读取响应
            BufferedReader reader = new BufferedReader(new InputStreamReader(connection.getInputStream()));
            String lines;
            StringBuffer sb = new StringBuffer("");
            while ((lines = reader.readLine()) != null) {
                lines = new String(lines.getBytes(), "utf-8");
                sb.append(lines);
            }
            // System.out.println(sb);
            reader.close();
            // 断开连接
            connection.disconnect();
            return sb.toString();
        } catch (MalformedURLException e) {
            logger.error("httpPostWithJsonMalformedURLException error", e);
            e.printStackTrace();
        } catch (UnsupportedEncodingException e) {
            logger.error("httpPostWithJsonUnsupportedEncodingException error", e);
            e.printStackTrace();
        } catch (IOException e) {
            logger.error("httpPostWithJsonIOException error", e);
            e.printStackTrace();
        }
        return null;
    }

在上述代码中，流的关闭是放在try中的，但是我们都知道流资源的关闭尽量要放在finally块中，因为如果try中代码执行失败，流没有被正确关闭就造成资源浪费。因为我将流的关闭移到finally块中如下：

    public static String httpPostWithJson(String ecUrl, String params) {
    BufferedReader reader = null;
    HttpURLConnection connection = null;
    try {
        URL url = new URL(ecUrl);
        connection = (HttpURLConnection) url.openConnection();
        // 创建连接
        connection.setDoOutput(true);
        connection.setDoInput(true);
        connection.setRequestMethod("POST");
        connection.setUseCaches(false);
        connection.setInstanceFollowRedirects(true);
        connection.setRequestProperty("Content-Type", "application/x-www-form-urlencoded");
        connection.connect();

        // POST请求
        DataOutputStream out = new DataOutputStream(connection.getOutputStream());
        out.writeBytes(params);
        out.flush();
        out.close();

        // 读取响应
        reader = new BufferedReader(new InputStreamReader(connection.getInputStream()));
        String lines;
        StringBuffer sb = new StringBuffer("");
        while ((lines = reader.readLine()) != null) {
            lines = new String(lines.getBytes(), "utf-8");
            sb.append(lines);
        }
        return sb.toString();
    } catch (MalformedURLException e) {
        logger.error("httpPostWithJsonMalformedURLException error", e);
        e.printStackTrace();
    } catch (UnsupportedEncodingException e) {
        logger.error("httpPostWithJsonUnsupportedEncodingException error", e);
        e.printStackTrace();
    } catch (IOException e) {
        logger.error("httpPostWithJsonIOException error", e);
        e.printStackTrace();
    } finally {
        try {
            if (null != reader) {
                reader.close();
            }
            if (null != connection) {
                connection.disconnect();
            }
        } catch (IOException e) {
            e.printStackTrace();
        }
    }
    return null;
}

这样虽然在finally中关闭了流，但是又要在finally中引入IOException，这样是不是很麻烦啊

关于关闭流，还有没有好的措施或者实践经验呢？

谢谢~~

补充：JDK 1.7但是不能用Try-with-resources机制

Answer 1

这个问题其实无须过多困扰。也没有必要往JDK1.7的try-with-resources上扯。
首先关闭资源放在try块里一定会有问题：资源可能不被关闭。
所以资源的关闭应该放在finally里，这没有什么疑问。
至于finally块里close资源会额外引入IOE,这也是无法避免的。
目前(就我见到过的)绝大多数代码里，捕获IOE后，最多打一条log，更多的是noop，即no operations，do nothing。
close的时候IOE发生的几率很小，它应该属于一种操作系统层面的error，选择忽略它是正确的选择，毕竟你的系统不能因为一个资源关闭错误而停止运行。况且，如果你硬要捕获这个IOE，那能做些什么呢。

如果不想在finally块里引入try-catch，我见过guava的一种关闭方式，写个工具方法叫做closeQuietly()，不吵不闹就挺好。

Answer 2

谢邀
可以考虑Java7的try with resource

try (BufferedReader br = ...) {
    //...
} catch (IOException ex) {
    //...
}

参考

Answer 3

来一发安利，Lombok，一个可以让你少些很多代码的增强库。

@CleanUp

Answer 4

同意1楼的答案，你补充的问题，也是由于IDE的问题。

另外对于实现了 AutoCloseable 和 Closeable 接口的资源，最好都使用 try-with-resources结构。

以你的代码作为例子,省略了一些代码

try{
    reader.readline();
}catch(IOException e){
    e.printStackTrace();
}finally {
    try {
       reader.close();
    } catch (IOException e) {
       e.printStackTrace();
    }
}

假设 readline 和 close 都发生io异常,使用 JDK1.7 前的异常捕捉结构，只能捕捉到 close 的异常，readline 的异常被抑制了。（因为 finally 的代码必须执行）

另外一个问题，在 try-with-resources 中资源的 AutoCloseable 的 close 方法什么时候执行？

在执行完 try 代码块的代码之后就会执行（这里不贴实验代码了）,所以使用 try-with-resources 的结构，catch 块和 finally 块都是在资源进行关闭之后才会执行的。