2012-08-15 80 views
1

我看了很多例子,并试图理解我做错了什么,但没有成功,也许你可以帮助我。它总是停在第二个文件上,但第一个文件只是在c:\上大小为0kb。 files_server.get(i)是ArrayList,其中包含我希望下载的所有文件。Java FTP下载进度

我的代码:

public FTPConnection() { 

StartD std = new StartD(); 
std.start(); 
} 

class StartD extends Thread{ 

     @Override 
     public void run() 
     { 
     for (int i = 0; i < files_server.size(); i++) { 
      err = ftpDownload(files_server.get(i), "C:/"+ files_server.get(i)); 
      if (!err) 
       { 
       System.out.println("Error in download, breaking"); 
       break; 
       } 
      } 
     } 

     public boolean ftpDownload(String srcFilePath, String desFilePath) 
     { 
     try { 
      FileOutputStream desFileStream = new FileOutputStream(desFilePath); 
      InputStream input = mFTPClient.retrieveFileStream(srcFilePath); 
      byte[] data = new byte[1024]; 
      int count; 
      while ((count = input.read(data)) != -1) 
      { 
       desFileStream.write(data, 0, count); 
      } 
      desFileStream.close(); 

     } catch (Exception e) { 

      return false; 
     } 
    return true; 

    }} 

如果我使用finction:

public boolean ftpDownload(String srcFilePath, String desFilePath) { 
     boolean status = false; 
     try { 

      FileOutputStream desFileStream = new FileOutputStream(desFilePath); 
      status = mFTPClient.retrieveFile(srcFilePath, desFileStream); 
      desFileStream.close(); 
      return status; 
     } catch (Exception e) { 

     } 
     return status; 
    } 

代替,一切都运行得很好,但我不能老是监视文件下载进度。

+0

很多问题。例如,由于您没有说出所有这些FTP的来源,代码不能被复制。前1024个字节的数据永远丢失。你应该附加一个可运行的测试用例 – Raffaele 2012-08-15 08:11:54

+0

你是什么意思“代码不能被复制”? 已修复“前1024个字节的数据永远丢失”。还是行不通。 – DanM 2012-08-15 09:05:40

+0

[this](http://pastebin.com/ng5rw4sV)是可复制的,自包含代码 – Raffaele 2012-08-15 10:28:16

回答

2

我只用它进行文件解压缩,而不是FTP,但在这种情况下的InputStream缓冲区可以返回零,所以我会说这是值得尝试改变你的while循环是这样的:

while ((count = input.read(data)) >= 0) 

public int read(byte [] b)throws IOException

从输入流中读取一定数量的字节并将它们存储到缓冲区数组b中。 实际读取的字节数作为整数返回。此方法会阻止,直到输入数据可用,检测到文件结尾, 或抛出异常。

如果b的长度为零,则不读取字节并返回0;

这也可能是你的两倍分配数量,这可能去砍第一个字节关数据:

int count = input.read(data); 
while ((count = input.read(data)) != -1) 

所以不指定任何东西,当你把它声明计数。

1

我们假设你的库是来自commons-net包的FTP客户端。要弄清楚代码出了什么问题并不容易,因为我们无法运行它,并且因为您的描述(第二个文件停止)是不够的(它会抛出异常吗?它会永久挂起吗?没有任何一方会完成影响?)。反正我有几个建议的:

  1. 使用CountingOutputStream(Apache的公共-10)监测进展
  2. 使用ProtocolCommandListener登录是怎么回事

另外请注意,第一1024字节总是丢失。最终,我不知道将它放在C:\中的安全性与它在服务器上的名称相同。在最好的情况下,它可能会导致许可问题,最糟糕的是它可能引发安全漏洞 - 无论如何,如果你对文件名有一定程度的控制,这并不适用,但是你可以考虑这个建议。

这是一个简单的客户端

public class FTP { 

    public static void main(String[] args) throws SocketException, IOException { 
     FTPClient client = new FTPClient(); 

     client.addProtocolCommandListener(new ProtocolCommandListener(){ 

      @Override 
      public void protocolCommandSent(ProtocolCommandEvent evt) { 
       logger.debug(evt.getMessage()); 

      } 

      @Override 
      public void protocolReplyReceived(ProtocolCommandEvent evt) { 
       logger.debug(evt.getMessage()); 
      } 

     }); 

     client.connect("ftp.mozilla.org"); 
     client.login("anonymous", ""); 
     client.enterLocalPassiveMode(); 

     OutputStream out = new CountingOutputStream(new NullOutputStream()) { 
      @Override 
      public void beforeWrite(int count) { 
       super.beforeWrite(count); 
       logger.info("Downloaded " + getCount() + " bytes"); 
      } 
     }; 

     for (String filename: new String[] {"MD5SUMS", "SHA1SUMS"}) 
      client.retrieveFile("pub/firefox/releases/15.0b4/" + filename, out); 

     out.close(); 
     client.disconnect(); 
    } 

    private static Logger logger; 

    static { 
     logger = Logger.getLogger(FTP.class.getCanonicalName()); 
    } 

} 

配置完成后,记录器将输出所有的原始套接字的对话,并且它可以帮助你更好地理解这个问题,只要它是在FTP侧,而不是在应用程序IO