Verschachtelte Async-Download - Async innerhalb von Async

Question

Ich habe einige verschachtelte Async-Methoden aufrufen einander und es ist verwirrend. Ich versuche, ein Projekt zu konvertieren, das die Dateien in einem asynchronen Download herunterlädt. Auf dem Klick auf den Download-Button ist dies die Methode ausgelöst:

private async void enableOfflineModeToolStripMenuItem_Click(object sender, EventArgs e) 
     { 
      for(int i = 0; i < _playlists.Count; i++) 
      { 
       DoubleDimList.Add(new List<String>()); 
       for(int j = 0; j < 5; j++) 
       { 
        string sMp3 = IniReadValue(_playlists[i], "Track " + j); 
        DoubleDimList[i].Add(sMp3); 
       } 
       await Task.Run(() => _InetGetHTMLSearchAsyncs(DoubleDimList[i]));    
      } 
     } 

Es schafft eine 2d List, die am Ende wie dieses DoubleDimList[3][20] aussieht. Am Ende von jedem sublist mache ich einen async Download, wie Sie sehen können. Das Verfahren sieht wie folgt aus

private async Task _InetGetHTMLSearchAsyncs(List<string> urlList) 
     { 
      foreach (var url in urlList) 
      { 
       await Task.Run(() => _InetGetHTMLSearchAsync(url)); 
      } 
     } 

die _InetGetHTMLSearchAsync Methode sieht wie folgt aus und ist hier, wo es schwierig wird

private async Task _InetGetHTMLSearchAsync(string sTitle) 
     { 
      Runs++; 
      if (AudioDumpQuery == string.Empty) 
      { 
       //return string.Empty; 
      } 
      string sResearchURL = "http://www.audiodump.biz/music.html?" + AudioDumpQuery + sTitle.Replace(" ", "+"); 
      try 
      { 
       using (var client = new WebClient()) 
       { 
        client.Headers.Add("Referer", @"http://www.audiodump.com/"); 
        client.Headers.Add("user-agent", "Mozilla/5.0(Macintosh; Intel Mac OS X 10_9_3) AppleWebKit/537.75.14(KHTML, like Gecko) Version/7.0.3 Safari/7046A194A"); 
        client.DownloadStringCompleted += Client_DownloadStringCompleted; 
        await Task.Run(() => client.DownloadStringAsync(new Uri(sResearchURL))); 

       } 
      } 
      catch (Exception ex) 
      { 
       Console.WriteLine("Debug message: " + ex.Message + "InnerEx: " + ex.StackTrace); 
       Console.WriteLine("Runs: " + Runs); 
       return; 
      } 
     } 

Auf Client_DownloadStringCompleted gibt es eine weitere async Methode aufgerufen. Hier ist es

private async void Client_DownloadStringCompleted(object sender, DownloadStringCompletedEventArgs e) 
     { 
      string[] sStringArray; 
      string aRet = e.Result; 
      string[] aTable = _StringBetween(aRet, "<BR><table", "table><BR>", RegexOptions.Singleline); 
      if (aTable != null) 
      { 
       string[] aInfos = _StringBetween(aTable[0], ". <a href=\"", "<a href=\""); 
       if (aInfos != null) 
       { 
        for (int i = 0; i < 1; i++) 
        { 
         sStringArray = aInfos[i].Split('*'); 
         sStringArray[0] = sStringArray[0].Replace("&#39;", "'"); 
         aLinks.Add(sStringArray[0]); 
        } 
        await Task.Run(() => DownloadFile(aLinks[FilesDownloaded])); 
       } 
      } 
     } 

Von dort, Überraschung! Ein weiterer Anruf async.

private async Task DownloadFile(string url) 
     { 
      try 
      { 
       using (var client = new WebClient()) 
       { 
        client.Headers.Add("Referer", @"http://www.audiodump.biz/"); 
        client.Headers.Add("user-agent", "Mozilla/5.0(Macintosh; Intel Mac OS X 10_9_3) AppleWebKit/537.75.14(KHTML, like Gecko) Version/7.0.3 Safari/7046A194A"); 
        client.DownloadFileCompleted += Client_DownloadFileCompleted; 
        await Task.Run(() => client.DownloadFileTaskAsync(url, mp3Path + "\\" + count + ".mp3")); 

       } 
      } 
      catch (Exception Ex) 
      { 
       Console.WriteLine("File download error: " + Ex.StackTrace); 
      } 
     } 

Nun ist der erste Teil nach der Gründung des 2d List ist die Download-Links der mp3s abzurufen. Der zweite Teil ist das Herunterladen der mp3, sobald eine gültige URL bereitgestellt wurde. Es funktioniert aber auf eine bizarre Weise. Anstatt die Datei normal herunterzuladen (1., 2., 3., ...), lädt sie die Dateien (1., 5., 8. usw.) nach dem Zufallsprinzip herunter.

Es ist mein erster gehen für async Download und Junge, ich bin schon weit von meinen Grenzen.

Wo mache ich das kaputt? Und die Hauptfrage, wird das jemals so funktionieren, wie es funktionieren soll?

Answer 1

Ihr Code sieht ziemlich gut, bis auf zwei Dinge:

1. Sie nicht Task.Run verwendet werden soll. Der primäre Anwendungsfall für Task.Run ist das Verschieben von CPU-gebundener Arbeit von einem GUI-Thread, sodass die Benutzeroberfläche nicht blockiert wird. Ich habe eine Serie über die proper use of Task.Run, die im Detail darauf eingeht.
2. Sie sollten ein konsistentes asynchrones Muster verwenden, idealerweise TAP. Ihr Code verwendet derzeit TAP überall außer in _InetGetHTMLSearchAsync, die EAP verwendet. Dies verursacht das seltsame Verhalten, das Sie sehen.

Eine feste _InetGetHTMLSearchAsync etwas würde wie folgt aussehen:

private async Task _InetGetHTMLSearchAsync(string sTitle) 
{ 
    Runs++; 
    string sResearchURL = "http://www.audiodump.biz/music.html?" + AudioDumpQuery + sTitle.Replace(" ", "+"); 
    try 
    { 
    using (var client = new WebClient()) 
    { 
     client.Headers.Add("Referer", @"http://www.audiodump.com/"); 
     client.Headers.Add("user-agent", "Mozilla/5.0(Macintosh; Intel Mac OS X 10_9_3) AppleWebKit/537.75.14(KHTML, like Gecko) Version/7.0.3 Safari/7046A194A"); 
     string[] sStringArray; 
     string aRet = await client.DownloadStringTaskAsync(new Uri(sResearchURL)); 
     string[] aTable = _StringBetween(aRet, "<BR><table", "table><BR>", RegexOptions.Singleline); 
     if (aTable != null) 
     { 
     string[] aInfos = _StringBetween(aTable[0], ". <a href=\"", "<a href=\""); 
     if (aInfos != null) 
     { 
      for (int i = 0; i < 1; i++) 
      { 
      sStringArray = aInfos[i].Split('*'); 
      sStringArray[0] = sStringArray[0].Replace("&#39;", "'"); 
      aLinks.Add(sStringArray[0]); 
      } 
      await DownloadFile(aLinks[FilesDownloaded]); // Should really be called "DownloadFileAsync" 
     } 
     } 
    } 
    } 
    catch (Exception ex) 
    { 
    Console.WriteLine("Debug message: " + ex.Message + "InnerEx: " + ex.StackTrace); 
    Console.WriteLine("Runs: " + Runs); 
    return; 
    } 
}