C＃の配列を使用して複数のファイルをダウンロード

Question

リモートサーバーからエンドユーザーのコンピュータに5つのファイルをダウンロードするダウンローダを作成しようとしています。それに問題があります。 Function after Functionを使用することで、ダウンロードしますが、リモートファイルを読み込んでファイル名を取得し、配列に入れて一つの関数から一つずつダウンロードする必要があります。これはAsynchを使用してC＃で書かれています。

以下のマイコード。どんな助けもありがとう。

// Start Downloads 
    private void getPatch() 
    { 
     sw.Start(); 
     string patchlist = Settings1.Default.patchlist; 
     label1.Text = "Downloading"; 
     button2.Image = Properties.Resources.PlayButtonDisabled; 
     button4.Visible = false; 
     progressBar1.Visible = true; 
     Directory.CreateDirectory("WTF"); 

     string[] files = new string[6]; 
     files[0] = "libeay32.dll"; 
     files[1] = "libmysql.dll"; 
     files[2] = "libssl32.dll"; 
     files[3] = "ssleay32.dll"; 
     files[4] = "connection_patcher.exe"; 
     files[5] = "Config.conf"; 

     // Loop 
     foreach (var file in files) 
     { 

      var webClient = new WebClient(); 
      webClient.DownloadFile(Settings1.Default.baseURL + file, file); 
      webClient.DownloadProgressChanged += new DownloadProgressChangedEventHandler(ProgressChanged); 
      webClient.DownloadStringCompleted += OnGetDownloadedStringCompleted; 
     } 
    } 

Answer 1

foreach（...）ループを詳しく見てください。あなたが実際によくない文字列「http://somedomain.com/file_list」、の各文字をループすることがあります、それのルックスから

foreach (var url in filelist) 

。ここで 'var'キーワードを避けることはコンパイル時にこれを捕らえてしまいます。

webClient.DownloadFileTaskAsync(new Uri(Settings1.Default.baseURL + rFile), rFile); 

RFILEは、実際にはどちらか私たちに何か良いをしない単一の文字列に一緒にサーバからダウンロードしたファイルの内容全体を、その5行を持つことになります。

ファイルリストの場所から内容を取得したら、それを分割して各行にループする必要があります。

string rFile = webClient.DownloadString(filelist); 
foreach (string singleFile in rFile.Split(new string[] { Environment.NewLine }, StringSplitOptions.None)) 
{ 
    await webClient.DownloadFileTaskAsync(new Uri(Settings1.Default.baseURL + singleFile), singleFile); 
} 

Answer 2

私はすぐにあなたが見落としているかもしれないいくつかのことを見ています。

まず、foreach（ファイルリストのURL）を反復していますが、ループ内のどこでも 'url'を使用しているとは思われません。ダウンロード文字列は、ループの各ステップですべて同じです。

2番目は、非同期で実行されるDownloadFileTaskAsyncを使用しています。これは、すべてのファイルが同時にダウンロードされていることを意味します。エラーが出るかもしれませんが、ループの最後の反復だけが実際に動作していると想像して見てください。いずれにせよ、ここにあなたのコードと私はそれを修正すると思います。

また、実行前にイベントハンドラをフックアップすることをお勧めします。あなたはコードを一度に1つのファイルを引っ張ってくることやろうとしているしているかのアイデアを想定

//あなたのコード

private void getFiles() 
{ 
    string filelist = "http://somedomain.com/file_list"; 
      label1.Text = "Downloading"; 
      button2.Image = Properties.Resources.ButtonDisabled; 
      progressBar1.Visible = true; 
      var webClient = new WebClient(); 
      string rFile = webClient.DownloadString(filelist); 
      webClient.DownloadProgressChanged += new DownloadProgressChangedEventHandler(ProgressChanged); 
      foreach (var url in filelist) 
      { 
       webClient.DownloadFileTaskAsync(new Uri(Settings1.Default.baseURL + rFile), rFile); 
      } 
      webClient.DownloadStringCompleted += OnGetDownloadedStringCompleted; 

} 

//。 DownloadFileAsyncタスクを適切に使用するためにawait/asyncを追加しました。うまくいけば、これは助けとして私がちょうど翼を立てているのであなたが見落としているかもしれないものを見ることを助けるか、または許可する。

private async void getFiles() 
    { 
     string filelist = "http://somedomain.com/file_list"; 
     //label1.Text = "Downloading"; 
     //button2.Image = Properties.Resources.ButtonDisabled; 
     //progressBar1.Visible = true; 
     using (var webClient = new WebClient()) 
     { 
      webClient.DownloadStringCompleted += OnGetDownloadedStringCompleted; 
      webClient.DownloadProgressChanged += new DownloadProgressChangedEventHandler(ProgressChanged); 
      var rFile = webClient.DownloadString(filelist); 

      //rFile needs broken up somehow since it is a string and not an array... unless you want the characters to be the array... which is odd. 
      //So for dummying the code to work for now... 
      var rFiles = rFile.Split(','); 

      foreach (var url in rFiles) 
       await webClient.DownloadFileTaskAsync(new Uri(Settings1.Default.baseURL + url), rFile); 
     } 
    }