java: beschädigte Zip-Datei erstellt, wenn Kopie nio

Question

Verwendung Ich habe folgenden Code implementieren Datei (Binär-Datei) Code

private void copyFileWithChannels(File aSourceFile, File aTargetFile) { 
     log("Copying files with channels.");   
     FileChannel inChannel = null; 
     FileChannel outChannel = null; 
     FileInputStream inStream = null; 
     FileOutputStream outStream = null;  
     try { 
      inStream = new FileInputStream(aSourceFile); 
      inChannel = inStream.getChannel(); 
      outStream = new FileOutputStream(aTargetFile);   
      outChannel = outStream.getChannel(); 
      long bytesTransferred = 0; 
      while(bytesTransferred < inChannel.size()){ 
       bytesTransferred += inChannel.transferTo(0, inChannel.size(), outChannel); 
      } 
     } 
     catch(FileNotFoundException e){ 
      log.error("FileNotFoundException in copyFileWithChannels()",e); 
     } 
     catch (IOException e) { 
      log.error("IOException in copyFileWithChannels()",e);   
     } 
     catch (Exception e) { 
      log.error("Exception in copyFileWithChannels()",e); 
     } 
     finally { 
      try{ 
       if (inChannel != null) inChannel.close(); 
       if (outChannel != null) outChannel.close(); 
       if (inStream != null) inStream.close(); 
       if (outStream != null) outStream.close(); 
      }catch(Exception e){ 
       log.error("Exception in copyFileWithChannels() while closing the stream",e); 
      } 
     } 

    } 

Ich habe Testcode mit einer ZIP-Datei zu kopieren. Wenn ich die Datei verifiziere, habe ich festgestellt, dass die erzeugte Datei beschädigt ist (die Größe wurde erhöht). Quell-Zip-Datei ist etwa 9 GB.

Answer 1

Die transferTo Methode das erste Argument gibt die verweist Position von welcher zu tra nsfer, nicht relativ zum Speicherort des Streams, sondern relativ zum Anfang der Datei. Da Sie 0 dort eingeben, wird es immer vom Anfang der Datei übertragen. So daß die Leitung

bytesTransferred += inChannel.transferTo(bytesTransferred , inChannel.size(), outChannel); 

mavarazy in seiner Antwort erwähnt werden muss ist er nicht sicher, ob Sie eine Schleife benötigen, wenn inChannel.size() verwenden, da die Erwartung, dass, wenn Sie die ganze Größe liefern wird die gesamte Datei kopieren. Die tatsächliche Übertragung kann jedoch geringer sein als die angeforderte Anzahl von Bytes, wenn der Puffer des Ausgabekanals weniger Platz zur Verfügung hat. Sie brauchen also die Schleife wie in seinem zweiten Code-Snippet.

Answer 2

Versuchen Sie folgendes:

while(bytesTransferred < inChannel.size()){ 
     bytesTransferred += inChannel.transferTo(bytesTransferred, inChannel.size() - bytesTransferred, outChannel); 
    } 

Auch ich IOUtils Implementierung als Referenz

https://github.com/apache/commons-io/blob/master/src/main/java/org/apache/commons/io/FileUtils.java

speziell

private static void doCopyFile(final File srcFile, final File destFile, final boolean preserveFileDate) 

Answer 3

Sofern Sie keinen guten Grund haben, Files.copy(Path, Path, CopyOption...) zu verwenden.