在读取所有字符时是否有理由在InputStreamReader上使用BufferedReader？

Question

我目前使用以下函数来执行简单的HTTP GET.

public static String download(String url) throws java.io.IOException {
    java.io.InputStream s = null;
    java.io.InputStreamReader r = null;
    //java.io.BufferedReader b = null;
    StringBuilder content = new StringBuilder();
    try {
        s = (java.io.InputStream)new URL(url).getContent();

        r = new java.io.InputStreamReader(s);
        //b = new java.io.BufferedReader(r);

        char[] buffer = new char[4*1024];
        int n = 0;
        while (n >= 0) {
            n = r.read(buffer, 0, buffer.length);
            if (n > 0) {
                content.append(buffer, 0, n);
            }
        }
    }
    finally {
        //if (b != null) b.close();
        if (r != null) r.close();
        if (s != null) s.close();
    }
    return content.toString();
}

我认为没有理由使用,BufferedReader因为我只是按顺序下载所有内容.我是否认为BufferedReader在这种情况下没有用处？

Answer 1

在这种情况下，我会像您一样做（使用字节数组进行缓冲而不是流缓冲区之一）。

不过也有例外。您看到缓冲区（这次是输出）的一个地方是在 servlet API 中。在调用flush()之前，数据不会写入底层流，允许您缓冲输出，但如果发生错误，则转储缓冲区并改为写入错误页面。如果您需要使用mark(int)和reset()重置流以重新读取，则可能会缓冲输入。例如，您可能会在决定将流传递给哪个内容处理程序之前检查文件头。

无关，但我认为你应该重写你的流处理。这种模式最能避免资源泄漏：

    InputStream stream = new FileInputStream("in");
    try { //no operations between open stream and try block
        //work
    } finally { //do nothing but close this one stream in the finally
        stream.close();
    }

如果您要打开多个流，请嵌套 try/finally 块。

您的代码正在做的另一件事是假设返回的内容是用您的 VM 的默认字符集编码的（尽管这可能就足够了，具体取决于用例）。