使用ZipOutputStream损坏的zip文件

Question

我正在尝试创建一个zip文件，以便能够通过http发送多个文件。

我的问题是，生成的Zip文件在发送之前和之后都“损坏”。问题是我无法找到我做错了什么，因为我在控制台中没有收到任何错误。

那么，有人有一个想法文件我生成的zip文件损坏？

这是我的代码：

  OutputStream responseBody = t.getResponseBody();
            ByteArrayOutputStream baos = new ByteArrayOutputStream();
            ZipOutputStream zos = new ZipOutputStream(baos);


            int counter = 1;
            for (PDDocument doc : documents)
            {
                ZipEntry zipEntry = new ZipEntry("document" + counter);
                zos.putNextEntry(zipEntry);
                ByteArrayOutputStream docOs = new ByteArrayOutputStream();
                doc.save(docOs);
                docOs.close();
                zos.write(docOs.toByteArray());
                zos.closeEntry();
                zos.finish();
                zos.flush();

                counter++;
            }
            zos.close();
            baos.close();


            responseBody.write(baos.toByteArray());
            responseBody.flush();

谢谢你的帮助！

Answer 1

当循环终止ZIP条目时，您需要从循环内部删除zos.finish（），因为它由流末尾的zos.close（）处理。

有了非常大的流，你最好直接发送ZIP到响应体，绕过字节数组输出流内存缓冲区。

如果仍然存在问题，请检查输出的内容类型是否已设置。通过临时将字节[]写入文件以检查发送的ZIP格式，调试可能会更容易：

Files.write(Path.of("temp.zip"), baos.toByteArray());

下面的大纲显示了通过http发送一个简单的ZIP（从servlet，将前两行调整为适当的“t”调用）。如果在循环中添加自己的文档对象，这可能会帮助您检查代码的哪个步骤导致损坏：

// MUST set response content type:
// resp.setContentType("application/zip");
OutputStream out = resp.getOutputStream(); // or t.getResponseBody();
try(ZipOutputStream zos = new ZipOutputStream(out))
{
    while (counter-- > 0)
    {
        ZipEntry zipEntry = new ZipEntry("document" + counter+".txt");
        zos.putNextEntry(zipEntry);
        zos.write(("This is ZipEntry: "+zipEntry.getName()+"\r\n").getBytes());
    }
}