web-dev-qa-db-ja.com

生成されたZipファイルが破損しているかどうかを確認するにはどうすればよいですか？

システムにZipファイルを生成するコードがあります。すべて問題ありませんが、このZipファイルがFilZipまたはWinZipで開かれている場合、破損していると見なされることがあります。

だからここに私の質問です：生成されたZipファイルが破損しているかどうかをプログラムで確認するにはどうすればよいですか？

Zipファイルの生成に使用するコードは次のとおりです。

try {
    ZipOutputStream zos = new ZipOutputStream(new FileOutputStream(tmpFile));
    byte[] buffer = new byte[16384];
    int contador = -1;
    for (DigitalFile digitalFile : document.getDigitalFiles().getContent()) {
       ZipEntry entry = new ZipEntry(digitalFile.getName());
       FileInputStream fis = new FileInputStream(digitalFile.getFile());
       try {
          zos.putNextEntry(entry);
          while ((counter = fis.read(buffer)) != -1) {
             zos.write(buffer, 0, counter);
          }
          fis.close();
          zos.closeEntry();
       } catch (IOException ex) {
          throw new OurException("It was not possible to read this file " + arquivo.getId());
       }
    }
    try {
      zos.close();
    } catch (IOException ex) {
      throw new OurException("We couldn't close this stream", ex);
    }

ここで間違っていることはありますか？

編集：実際には、上記のコードは完全に大丈夫です。私の問題は、誤ったストリームをユーザーにリダイレクトしていることでした。したがって、Zipファイルを開く代わりに、まったく異なるものを開きます。 Mea culpa :(

しかし、主な質問は残ります：指定されたZipファイルが破損していないかどうかをプログラムで確認するにはどうすればよいですか？

javazip
27
Kico Lobo

ZipFileクラスを使用してファイルを確認できます。

 static boolean isValid(final File file) {
    ZipFile zipfile = null;
    try {
        zipfile = new ZipFile(file);
        return true;
    } catch (IOException e) {
        return false;
    } finally {
        try {
            if (zipfile != null) {
                zipfile.close();
                zipfile = null;
            }
        } catch (IOException e) {
        }
    }
}
28
Arnaud

これが投稿されたのは久しぶりです。皆さんから提供されたコードを使用してこれを思いつきました。これは、実際の質問に最適です。 Zipファイルが破損しているかどうかの確認

private boolean isValid(File file) {
    ZipFile zipfile = null;
    ZipInputStream zis = null;
    try {
        zipfile = new ZipFile(file);
        zis = new ZipInputStream(new FileInputStream(file));
        ZipEntry ze = zis.getNextEntry();
        if(ze == null) {
            return false;
        }
        while(ze != null) {
            // if it throws an exception fetching any of the following then we know the file is corrupted.
            zipfile.getInputStream(ze);
            ze.getCrc();
            ze.getCompressedSize();
            ze.getName();
            ze = zis.getNextEntry();
        } 
        return true;
    } catch (ZipException e) {
        return false;
    } catch (IOException e) {
        return false;
    } finally {
        try {
            if (zipfile != null) {
                zipfile.close();
                zipfile = null;
            }
        } catch (IOException e) {
            return false;
        } try {
            if (zis != null) {
                zis.close();
                zis = null;
            }
        } catch (IOException e) {
            return false;
        }

    }
}
9
user1707141

Zipファイルの生成中に、対応する例外スタックトレースが表示されると思います。したがって、例外処理を強化する必要はおそらくないでしょう。

3
John Doe

私の実装ではそれはそのように見えます。多分それはあなたを助ける：

//[...]

try {
    FileInputStream fis = new FileInputStream(file);
    BufferedInputStream bis = new BufferedInputStream(fis);

    zos.putNextEntry(new ZipEntry(file.getName()));

    try {
        final byte[] buf = new byte[BUFFER_SIZE];
        while (true) {
            final int len = bis.read(buf);
            if (len == -1) {
                break;
            }
            zos.write(buf, 0, len);
        }
        zos.flush();
        zos.closeEntry();
    } finally {
        try {
            bis.close();
        } catch (IOException e) {
            LOG.debug("Buffered Stream closing failed");
        } finally {
            fis.close();
        }
    }
} catch (IOException e) {
    throw new Exception(e);
}

//[...]
zos.close
2
Sören

コードは基本的に問題ありませんが、破損したZipファイルの原因となっているファイルを見つけてください。 digitalFile.getFile（）が常に有効でアクセス可能な引数をFileInputStreamに返すかどうかを確認します。コードにビットロギングを追加するだけで、何が問題なのかがわかります。

1
stacker

おそらく次の2行を入れ替えますか？;

fis.close();
zos.closeEntry();

CloseEntry（）がストリームから一部のデータを読み取ることは想像できます。

1
Waverick
new ZipFile(file)

ファイルを再度圧縮するので、作業を重複させることができますが、それはあなたが探しているものではありません。 1つのファイルのみをチェックするという事実と、nファイルを圧縮する質問にもかかわらず。

これを見てみましょう： http://www.kodejava.org/examples/336.html

Zipのチェックサムを作成します。

CheckedOutputStream checksum = new CheckedOutputStream(fos, new CRC32());
ZipOutputStream zos = new ZipOutputStream(new BufferedOutputStream(checksum));
...

そして、あなたが圧縮を終えたらそれを見せてください

System.out.println("Checksum   : " + checksum.getChecksum().getValue());

Javaまたは他のツールでチェックサムが一致するかどうかをチェックして、同じようにZipを読み取る必要があります。

詳細は https://stackoverflow.com/a/10689488/848072 を参照してください

1
albfan

ZipOutputStream 閉じない 基になるストリーム。

あなたがする必要があるのは：

FileOutputStream fos = new FileOutputStream(...);
ZipOutputStream zos = new ZipOutputStream(fos);

次に、終了ブロックで：

zos.close();
fos.flush(); // Can't remember whether this is necessary off the top of my head!
fos.close();
0
user7094