Question

我有以下代码，但我不确定我在效率/刷新/关闭流方面做得正确。一些建议会有很大帮助，谢谢

    OutputStream out = null;
    try {
        out = new BufferedOutputStream(new FileOutputStream(file, true));
        byte[] buf = new byte[32 * 1024]; // should this be 32KB?
        while ((in.read(buf)) > 0) {
            out.write(buf);
        }
        out.flush();
    } catch (Exception e) {
        e.printStackTrace();
    } finally {
        try {
            if (out != null)
                out.close();
            if (in != null)
                in.close();
        } catch (IOException e) {
            e.printStackTrace();
        }
    }

Answer 1

您遇到的最重要的问题是您忽略了您阅读的字节数。

for(int len; (len = in.read(buf)) > 0;) 
        out.write(buf, 0, len);

如果您不使用您假设的长度，您将始终读取32 KB，这是一个很大的假设。

当您有大量小写时，缓冲区非常有用。

BufferedOutputStream的默认缓冲大小为8 KB，如果您的写入比此小得多，即＆lt; 512字节，他们真的可以帮助。

但是，如果你写的是32 KB，他们可能什么都不做，或者没有帮助。我会带他们出去。

BTW，没有缓冲区，你不需要调用flush（）;

BTW2

KB = 1024 bytes
kB = 1000 bytes
Kb = 1024 bits
kb = 1000 bits.

Answer 2

你的代码似乎可以从“它的工作”角度来看......但是你可以通过使用资源尝试看起来更“漂亮”。 Try with Resources 您提供的代码基本上会变成以下内容：

try(OutputStream out = new BufferedOutputStream(new FileOutputStream(file, true)) {
    byte[] buf = new byte[1024];
    while ((in.read(buf)) > 0) {
        out.write(buf);
    }
    out.flush();
}

这是一个Java7功能，如果流资源实现了java.lang.AutoCloseable，那么它将自动关闭。

根据您的尝试，以下内容可能是一个更简单的解决方案？

PrintStream p = new PrintStream(new BufferedOutputStream(new FileOutputStream(aFile, true)));

如何使用缓冲流附加到Java中的文件？

2 个答案: