0

我正在尝试将n个文件合并为单个文件。但是我的功能出现了奇怪的行为。该函数在n秒内被调用x次。假设我有 100 个文件要合并,每秒我调用 5 个文件并合并它。在下一秒,数量翻倍为 10,但从 1-5 是与之前相同的文件,其余的是新文件。它工作正常,但在某些时候,它给出零字节或有时给出正确的大小。

您能帮我找出下面函数中的错误吗?

public void mergeFile(list<String> fileList, int x) {
    int count = 0;
    BufferedOutputStream out = null;
    try {
        out = new BufferedOutputStream(new FileOutputStream("Test.doc"));
        for (String file : fileList) {
            InputStream in = new BufferedInputStream(new FileInputStream(file));
            byte[] buff = new byte[1024];
            in.read(buff);
            out.write(buff);
            in.close();
            count++;
            if (count == x) {
                break;
            }
        }
        out.flush();
        out.close();
    } catch (IOException e) {
        e.printStackTrace();
    }
}

*对不起我的英语不好

4

2 回答 2

1

in.read(buff);

检查Javadoc。该方法不能保证填充缓冲区。它返回一个值,告诉您它读取了多少字节。您应该使用它,在这种情况下,您应该在决定要写入多少字节(如果有的话)时使用它。

于 2012-07-13T10:29:06.490 回答
0

您没有读取完整的文件,您从每个文件中最多只能读取1024 个字节。您需要循环读取,只要它返回数据(或使用类似Files.copy()的东西。

顺便说一句:如果您使用大缓冲区进行复制,则不需要 BufferedOutputStream。

public void mergeFile(list<String> fileList, int x) throws IOException {
    try (OutputStream out = new FileOutputStream("Test.doc");) {
        int count=0;
        for (String file : fileList) {
            Files.copy(new File(file).toPath(), out);
            count++;
            if (count == x) {
                break;
            }
        }
    }
}

如果您关闭,您也不需要 flush()。我在这里使用“try-with-resource”,所以我不需要明确地关闭它。最好传播异常。

于 2014-10-08T23:51:40.513 回答