Java ZipInputStream提取错误

Question

下面是一些从仅包含单个文件的zip文件中提取文件的代码。但是，提取的文件与通过WinZip或其他zip实用程序提取的同一文件不匹配。我希望如果文件包含奇数个字节，它可能会被一个字节关闭（因为我的缓冲区大小为2，而我只是在读取失败时中止）。但是，在分析（使用WinMerge或Diff）使用下面的代码提取的文件与通过Winzip提取的文件时，有几个区域的Java提取缺少字节。有谁知道为什么或如何解决这个问题？

package zipinputtest;

import java.io.BufferedOutputStream;
import java.io.File;
import java.io.FileInputStream;
import java.io.FileNotFoundException;
import java.io.FileOutputStream;
import java.io.IOException;
import java.io.OutputStream;
import java.util.zip.ZipInputStream;

public class test2 {
    public static void main(String[] args) {
        try {
            ZipInputStream zis = new ZipInputStream(new FileInputStream("C:\\temp\\sample3.zip"));
            File outputfile = new File("C:\\temp\\sample3.bin");
            OutputStream os = new BufferedOutputStream(new FileOutputStream(outputfile));
            byte[] buffer2 = new byte[2];
            zis.getNextEntry();
            while(true) {
                if(zis.read(buffer2) != -1) {
                    os.write(buffer2);
                }
                else break;
            }
            os.flush();
            os.close();
            zis.close();
        } catch (FileNotFoundException e) {
            e.printStackTrace();
        } catch (IOException e) {
            e.printStackTrace();
        }
    }
}

我能够使用此图像生成错误（将其保存并压缩为sample3.zip并在其上运行代码），但任何足够大小的二进制文件都应显示差异。

Answer 1

您可以使用更加逐字的方式来检查是否所有字节都被读取和写入，例如像

这样的方法

  public int extract(ZipInputStream in, OutputStream out) throws IOException {
    byte[] buffer = new byte[BUFFER_SIZE];
    int total = 0;
    int read;
    while ((read = in.read(buffer)) != -1) {
      total += read;
      out.write(buffer, 0, read);
    }
    return total;
  }

如果read中未使用write()参数，则该方法假定buffer的全部内容将被写出，如果{buffer可能不正确1}}没有完全填满。

可以在OutputStream方法的内部或外部刷新并关闭extract()。调用close()就足够了，因为它也会调用flush()。

在任何情况下，Java的“标准”I / O代码（如java.util.zip包）已经过广泛测试和使用，因此它不太可能有一个如此根本的错误导致字节很容易错过。

Answer 2

while (true) {
    if(zis.read(buffer2) != -1) {
        os.write(buffer2);
    }
    else break;
}

常见问题。你忽略了计数。应该是：

int count;
while ((count = zis.read(buffer2)) != -1)
{
    os.write(buffer2, 0, count);
}

NB：

1. 缓冲区大小为2是荒谬的。使用8192或更多。
2. flush()之前close()是多余的。