ftp no descarga el archivo correctamente en java?

Question

Cuando descargo el archivo con el siguiente código, simplemente escribo el archivo en el destino local pero el tamaño del archivo es cero. ¿Alguien puede decir por qué sucede esto y cómo solucionarlo?

import org.apache.commons.net.ftp.FTPClient; 
import org.apache.commons.net.ftp.FTPFile; 
import java.io.FileOutputStream; 
import java.io.IOException; 

public class FtpDownload { 
public static void main(String[] args) { 
    FTPClient client = new FTPClient(); 
    FileOutputStream fos = null; 
    String filename = "config.zip"; 
    try { 
     client.connect("ftpsrv"); 
     client.login("user", "user"); 

     for (FTPFile file : client.listFiles()) { 
      filename = "C:\\tmp\\user\\" + file.getName(); 
      if (file.isFile()) { 
       fos = new FileOutputStream(filename); 
       client.retrieveFile(filename, fos); 
       System.out.println(file.getName()); 
      } else if (file.isDirectory()) { 
       System.out.println("directory: " + file.getName()); 
      } 
     } 

    } catch (IOException e) { 
     e.printStackTrace(); 
    } finally { 
     try { 
      if (fos != null) { 
       fos.close(); 
      } 
      client.disconnect(); 
     } catch (IOException e) { 
      e.printStackTrace(); 
     } 
    } 

} 
} 

Answer 1

después de buscar en la documentación de Apache que venga a la conclusión de que BINARY_FILE_TYPE aún no está resuelto. he agregado el siguiente código y todo está bien. gracias a todos ustedes que ayudaron.

try { 
    client.connect(ftpServer); 
    client.login(username, password); 
    // following line fixed my zip file corruption issue. 
    client.setFileType(FTP.BINARY_FILE_TYPE); 

Answer 2

intente cerrar la corriente en cada iteración del bucle

fos.close(); 

El código no lo está haciendo correctamente porque tiene un bucle que crea nueva fuente de cada iteración, pero sólo cierra la última

Answer 3

Usted está utilizando la variable de ruta local para que apunte al archivo remoto:

filename = "C:\\tmp\\user\\" + file.getName(); 
... 
client.retrieveFile(filename, fos); 

Usted quiere apuntar al archivo remoto, no el destino local, así:

client.retrieveFile(file.getName(), fos); 

También, asegúrese de cerrar el flujo de fos después de leer cada archivo, en lugar de la derecha al final. Puede usar IOUtils.closeQuietly(fos); para esto si obtiene la biblioteca commons.io, que evita los bloques try-catch anidados.

También debe utilizar binario tipo archivo y modo de transferencia BLOQUE para la transferencia de archivos zip:

client.setFileTransferMode(FTPClient.BLOCK_TRANSFER_MODE); 
client.setFileType(FTPClient.BINARY_FILE_TYPE);