Wie überprüft man, ob eine generierte Zip-Datei beschädigt ist?

Question

Wir haben einen Code, der eine Zip-Datei auf unserem System erzeugt. Alles ist in Ordnung, aber manchmal wird diese Zip-Datei, die von FilZip oder WinZip geöffnet wurde, als beschädigt betrachtet.

Also hier ist meine Frage: Wie können wir überprüfen, ob eine generierte ZIP-Datei beschädigt ist?

Hier ist der Code, den wir unsere Zip-Dateien zu erzeugen, verwenden:

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); 
    } 

Gibt es alles, was wir falsch hier?

EDIT: Eigentlich ist der obige Code absolut in Ordnung. Mein Problem war, dass ich den WRONG-Stream für meine Benutzer umgeleitet habe. Anstatt eine Zip-Datei zu öffnen, öffneten sie etwas völlig anderes. Mea culpa :(

aber die Hauptfrage bleibt: wie programmatisch kann ich überprüfen, ob eine bestimmte ZIP-Datei nicht beschädigt ist

Answer 1

können Sie die ZipFile-Klasse verwenden, um Ihre Datei zu überprüfen:

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) { 
     } 
    } 
} 

Answer 2

Ich denke, Sie Korrespondent Ausnahme-Stack-Trace während Zip-Datei Generation sehen?. .. Also, werden Sie wahrscheinlich wan't auf Ihre Ausnahmebehandlung verbessern

Answer 3

ZipOutputStream does not close der zugrunde liegenden Stream

Was Sie tun müssen, ist:

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

Dann in Ihrem Schließblock:

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

Answer 4

Vielleicht die folgenden zwei Zeilen tauschen ?;

Ich kann mir vorstellen, dass der closeEntry() noch einige Daten aus dem Stream lesen wird.

Answer 5

Ihr Code ist grundsätzlich in Ordnung, versuchen Sie herauszufinden, welche Datei für die beschädigte ZIP-Datei verantwortlich ist. Überprüfen Sie, ob digitalFile.getFile() immer ein gültiges und zugreifbares Argument an FileInputStream zurückgibt. Fügen Sie Ihrem Code einfach ein wenig Logging hinzu und Sie werden herausfinden, was nicht stimmt.

Answer 6

in meiner Implementierung sieht es so aus. vielleicht hilft es Ihnen:

//[...] 

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 

Answer 7

new ZipFile(file) 

Kompresse wieder die Datei, so dass doppelte Arbeit, und das ist nicht das, was Sie suchen. Trotz der Tatsache, dass nur eine Datei überprüft und die Frage n-Dateien komprimiert werden.

einen Blick auf diese nehmen: http://www.kodejava.org/examples/336.html

eine Prüfsumme für Ihre Zip erstellen:

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

Und wenn man die Show es Kompression beenden

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

Sie müssen die gleichen Lese tun Die Zip mit Java oder anderen Tools überprüft, ob Prüfsummen übereinstimmen.

siehe https://stackoverflow.com/a/10689488/848072 für weitere Informationen

Answer 8

Ich weiß, es ist eine Weile, dass dies geschrieben wurde, ich den Code verwendet haben, die alle von Ihnen zur Verfügung gestellten und kam mit dieser. Dies funktioniert gut für die eigentliche Frage. Überprüfen, ob die Zip-Datei beschädigt ist oder nicht

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; 
     } 

    } 
}