0

我有几个 zip 文件，每个文件也包含几个我想使用 ZipInputStream 类提取的文件。其中有一些图像。当我尝试使用 BufferedOutputStream 提取这些图像时，它们被部分解压缩并且图像不完整。 

private void extractArchives() {

    ZipInputStream zis;

    File archiveDir = new File(
             Environment.getExternalStorageDirectory().getAbsolutePath() +
     "/archives/");

    File[] files = archiveDir.listFiles();

    for (int i = 0; i < files.length; ++i)
    {
        File file = files[i];

        try
        {
            zis = new ZipInputStream(new FileInputStream(file));
            ZipEntry ze;

            while ((ze = zis.getNextEntry()) != null)
            {
                BufferedOutputStream bos;
                byte[] buffer = new byte[102400];
                int count;

                while ((count = zis.read(buffer)) != -1)
                {
                    String fileName = ze.getName();

                    if (fileName.endsWith(".jpg"))
                    {
                        path += File.separator + fileName;
                        bos = new BufferedOutputStream(new FileOutputStream(path));
                        bos.write(buffer, 0, count);
                        bos.close();
                    }
                }
            }

            zis.close();

        }

        catch(FileNotFoundException e) { continue; }

        //If the file is not a zip file or is a directory
        catch (IOException e) { continue; }

    }
}

上面的代码有什么问题吗？使用 BufferedOutputStream 会导致这个问题吗？我很欣赏任何想法。谢谢。

4

2 回答 2

1

我根据 Erwin 所说的修改了方法，现在它可以工作了：

private void extractArchives() {

    File archiveDir = new File(
                Environment.getExternalStorageDirectory().getAbsolutePath() +
                "/archives/");

    String archivePath = archiveDir.getAbsolutePath();

    File[] files = archiveDir.listFiles();

    for (int i = 0; i < files.length; ++i)
    {
        File file = files[i];

        if(!file.isDirectory())
        {
            try {

                ZipInputStream zis = new ZipInputStream(new FileInputStream(file));
                ZipEntry entry = zis.getNextEntry();

                while (entry != null)
                {
                    if(entry.getName().endsWith(".jpg"))
                    {
                        String imagePath = themePath + File.separator + entry.getName();

                        BufferedOutputStream bos = new BufferedOutputStream(
                                    new FileOutputStream(imagePath));

                        byte[] buffer = new byte[4096];

                        int read = 0;

                        while ((read = zis.read(buffer)) != -1) bos.write(buffer, 0, read);

                        imagePath = "";

                        bos.close();
                    }

                    zis.closeEntry();

                    entry = zis.getNextEntry();
                }

                zis.close();
            }

            catch (FileNotFoundException e) {}

            catch (IOException e) {}
        }
    }
}
于 2014-01-20T19:03:07.203 回答
0

专注于遍历 zip 条目的循环，问题在于您从 zip 条目中读取最多 102400 个字节，然后将其写入新文件。如果将相同的图像文件写入新文件，则下一个最大 102400 字节。如果路径名与旧文件相同，它将覆盖旧文件。但是由于您不断地附加到局部变量“路径”，我不确定数据最终在哪里。

将 FileOutputStream 的开头拉出内部循环，并继续写入相同的 OutputStream，直到您无法从当前 ZipEntry 读取更多字节。只有这样才能转到下一个 ZipEntry 和下一个 OutputStream。

这是在 Java 中的流之间读取和复制的标准模式，因此我可以files在path没有zis.

生产代码中需要的另一项改进是包装bos.close()在 try/finally 块中，以便在从输入读取时出现异常时也关闭输出文件。

        while ((ze = zis.getNextEntry()) != null) {
            String fileName = ze.getName();

            if (fileName.endsWith(".jpg")) {
                String filepath = path + File.separator + fileName;
                BufferedOutputStream bos = new BufferedOutputStream(new FileOutputStream(filepath));
                byte[] buffer = new byte[102400];
                int count;
                while ((count = zis.read(buffer)) != -1) {
                    bos.write(buffer, 0, count);
                }
                bos.close();
            }
        }
于 2014-01-20T16:06:15.033 回答