使用ZipInputStream解压缩永远不会完成

时间:2016-04-15 09:48:39

标签: java android zip zipfile zipinputstream

我正在使用AsyncTask来解压缩文件,而且一切似乎都很顺利(ZIP存档中的所有文件都被解压缩),但我的解压缩方法永远不会完成。

这是我的解压缩类的来源:

public class MyUnzipper {


    public static boolean unzipFileIntoDirectory(String inputFilename, String outputPath) throws Exception {
        ZipInputStream zis = null;
        BufferedOutputStream dest = null;

        try {
            File archive = new File(inputFilename);
            File destinationDir = new File(outputPath);

            final int BUFFER_SIZE = 1024;

            zis = new ZipInputStream(new BufferedInputStream(new FileInputStream(archive), BUFFER_SIZE));
            ZipEntry entry = null;
            File destFile;
            while( (entry = zis.getNextEntry()) != null ){

                destFile = new File(destinationDir, entry.getName());

                if( entry.isDirectory() ){
                    destFile.mkdirs();
                }
                else {
                    // check for and create parent directories if they don't exist
                    File parentDir = destFile.getParentFile();
                    if ( null != parentDir ) {
                        if ( !parentDir.isDirectory() ) {
                            parentDir.mkdirs();
                        }
                    }

                    int count;
                    byte data[] = new byte[BUFFER_SIZE];
                    dest = new BufferedOutputStream(new FileOutputStream(destFile), BUFFER_SIZE);
                    Log.i("MyUnzipper", "Beginning unzip of " + destFile);

                    while( (count = zis.read(data, 0, BUFFER_SIZE)) != -1 ){ 
                        dest.write(data, 0, count);
                        Log.v("MyUnzipper", "Count = " + count);
                    }
                    dest.flush();
                    dest.close();
                    dest = null;
                }

                zis.closeEntry();

                Log.wtf("MyUnzipper", "Unzipped entry " + entry.getName());
            }

            Log.wtf("MyUnzipper", "Unzip done");
        }
        catch(Exception e){
            Log.wtf("MyUnzipper", "Unzip error");
            e.printStackTrace();
            return false;
        }
        finally {
            if( null != zis )
                zis.close();

            if( null != dest )
                dest.close();
        }

        return true;
    }

}

当我逐行调试时,它运行正常,直到它解压缩所有文件,然后它到达zis.closeEntry(),调试器只是“消失”,即下一行({{1永远不会执行。我的AsyncTask永远不会完成,就好像我陷入无限循环一样?!但是看一下Log.wtf(...)的来源,那里似乎没有任何循环或任何可疑的东西?

我还尝试使用ZipInputStream.closeEntry()而不是ZipFile解压缩ZIP存档,但后来我收到以下错误:

ZipInputStream

ZIP文件没有任何问题,我在Mac OSx上使用java.util.zip.ZipException: End Of Central Directory signature not found 进行了测试。我也尝试使用ZIP版本3.0和2.1重新压缩它(原始版本为2.0)。我可以在Mac OSx上解压缩所有版本(使用Unarchiver和Archive Utility)。

这让我疯了,可能出现什么问题?

更新(已解决)

原来是一个非常愚蠢的问题,与解压缩无关。

我在解压缩之前从服务器下载ZIP文件,显然我忘记在开始解压缩操作之前在下载操作的输出流上调用zip -v -T < / p>

也许这个帖子可以帮助那些犯同样愚蠢错误的人。

1 个答案:

答案 0 :(得分:4)

好吧,即使您关闭所有以前的输出和输入流,有时您的ZIP解压缩代码也会卡住。而且,这是a known bugZipInputStream#read可以返回0。

增加:

如果您的ZIP文件包含一些包含非ACSII文件名的文件,则您将面临提取问题。 Android ZipInputStream无法与UTF-8,CP437等配合使用。

在这种情况下,Apache Commons应该是一个解决方案:

private boolean unpack(File zipFile, File targetDir) {
    ZipFile zip = null;
    try {
        zip = new ZipFile(zipFile.getAbsoluteFile());
        final Enumeration<ZipArchiveEntry> entries = zip.getEntries();
        while(entries.hasMoreElements()) {
            ZipArchiveEntry entry = entries.nextElement();
            if (entry.isDirectory()) {
                mkdirsOrThrow(new File(targetDir, entry.getName()));
                continue;
            }
            final File entryDestination = new File(targetDir,  entry.getName());
            mkdirsOrThrow(entryDestination.getParentFile());
            final InputStream in = zip.getInputStream(entry);
            final OutputStream out = new FileOutputStream(entryDestination);
            IOUtils.copy(in, out);
            IOUtils.closeQuietly(in);
            IOUtils.closeQuietly(out);
        }
    } catch (IOException e) {
        throw new RuntimeException(e);
    } finally {
        if (zip!= null) try {
            zip.close();
        } catch (IOException e) {
            e.printStackTrace();
        }
    }
    return true;
}