关闭 ZipOutputStream

Question

我有点困惑。我知道空拉链是不合法的。但是这个示例片段怎么样：

ZipOutputStream zos = null; 
try
{
    zos = new ZipOutputStream(new FileOutputStream("..."));
    //
    //..
    //
}
finally
{
    zos.close();
}

如果由于某种原因（可能是特殊情况）没有添加 zip 条目，那么在关闭尝试时将引发以下异常：

Exception in thread "main" java.util.zip.ZipException: ZIP file must have at least one entry
    at java.util.zip.ZipOutputStream.finish(ZipOutputStream.java:304)
    at java.util.zip.DeflaterOutputStream.close(DeflaterOutputStream.java:146)
    at java.util.zip.ZipOutputStream.close(ZipOutputStream.java:321)

在这种情况下，关闭流的最干净的方法是什么？

谢谢...

Answer 1

您应该关闭FileOutputStream，而不是ZipOutputStream，因为前者实际上消耗了系统资源。

File zipFile = new File("/tmp/example.zip");
FileOutputStream fos = null;
try
{
   fos = new FileOutputStream(zipFile);
   ZipOutputStream zos = new ZipOutputStream(fos);

   // ...

   zos.close();
}
catch (IOException ex)
{
   // log/report exception, then delete the invalid file
   IOUtils.closeQuietly(fos);
   zipFile.delete();
}
finally
{
   IOUtils.closeQuietly(fos);
}

IOUtils 类位于 Jakarta Commons IO 中。使用它意味着您不必处理 close() 可能抛出的可能但很少有用的 IOException。

Answer 2

您应该跟踪是否向 zip 流添加了内容，并仅在添加内容时关闭它：

ZipOutputStream zos = null; 
OutputStream file = new FileOutputStream("...")
int itemsAdded=0;
try
{
    zos = new ZipOutputStream(file);
    //
    //..
    // itemsAdded++; 
}
finally
{
    if ( itemsAdded > 0 ) {
         zos.close();
    } else {
         file.close();
    }
}

或者如果您不需要计数，只需使用 boolean 标志。

Answer 3

我所做的是在运行邮政编码之前进行条件检查以查看是否有任何内容要压缩，而不是仅在添加内容时关闭流。这帮助我简化了流程，我认为通常可以用来处理“ZIP 文件必须至少有一个条目”问题。确实，关闭 zos 可能会引发其他异常，但这种情况很少见。

我认为这是 Java 的问题，当没有文件可压缩时，Java 无法处理这种情况。

IE：

int itemsToAdd=0;
//....

if ( itemsToAdd > 0 ) {

    ZipOutputStream zos = new ZipOutputStream(file);
    try {
        //add files to zip
    }
    finally {
        zos.close();
    }
}

Answer 4

如果您使用 Java 7 或更高版本，可以使用 try-with-resources 以获得更好的可读性

例如：

try (ZipOutputStream zos = new ZipOutputStream(new FileOutputStream("..."))) {
    // do something with zos
}

Answer 5

使用 Java 7 或更高版本，只需 try-with-resource 就足以关闭 ZipOutputStream，无需调用 close() 或 closeEntry() 方法来刷新缓冲区数据并关闭流。

检查了ZipOutputStream::close()的代码，它会调用finish()，并且在finish方法中，有如下检查，这意味着当流关闭时会自动调用closeEntry。

public void finish() throws IOException {
ensureOpen();
if (finished) {
    return;
}
if (current != null) {
    closeEntry();
}
// write central directory
long off = written;
for (XEntry xentry : xentries)
    writeCEN(xentry);
writeEND(off, written - off);
finished = true;

}