使用WebClient下载多个文件时无限循环
概念：我正在制作一个从给定URL下载文件的C＃应用程序。 文本框，url添加，文件下载，每个事件都以正确的方式发生。
我正在尝试重新创建此程序以逐个下载多个文件。 我有一个带有一个url / line的文本框，解析正确，我在字符串数组中放置了所有链接，放在文本框中。 然后它开始下载异步，我想让它一个接一个地下载，所以我在foreach循环中进行了一个while循环，因为我不想在下一个url之前直到当前的下载完成。
问题是：我进入一个无限循环（虽然我之前做了这个工作（idk如何），如果我在while循环中放置了一个消息框（注意：我在一分钟前重试过，这次没有做到这一点） ）。
我只是展示代码片段：
foreach (string url in urllist) { isdonwloaded = false; string filename = url.Split('/').Last(); label3.Text = filename; webclient.DownloadFileAsync(new Uri(url), @"C:\Users\Krisz" + @"\" + filename); while (!isdonwloaded) // this was the first idea, but with webclient.IsBusy it did the same thing { // MessageBox.Show(counter); Thread.Sleep(1000); label8.Text = "Download in progress..."; } counter++; label8.Text = "Done!"; } // Events: webclient.DownloadProgressChanged += new DownloadProgressChangedEventHandler(webc_DownloadProgressChanged); webclient.DownloadFileCompleted += new AsyncCompletedEventHandler(webc_DownloadFileCompleted); // The DownloadFileCompleted event: void webc_DownloadFileCompleted(object sender, AsyncCompletedEventArgs e) { label7.Text = String.Format("Files {0} / {1}", counter, arraylength(urllist)); isdonwloaded = true; } 我已经研究过这个post： WebClient.DownloadFileAsync – 一次下载一个文件 ，但我无法让它以这种方式工作。 （也许我误会了什么？）
有人能给我一些提示吗，我做错了什么？ 我从来没有真正使用过事件，因此遇到错误只是时间问题。 每一点帮助都有很大的帮助，这个程序对我来说很有用。
好的，首先，了解为什么你的代码现在不能正常工作。
想象一下，你在办公室里有两个人。 两者都有“在盒子里”。 他们的工作流程是：他们检查他们的收件箱。 如果收件箱中有任务，则他们执行任务直到完成，然后再次检查他们的收件箱，重复。
Worker 1在收件箱中收到一条消息，说明您的下一个任务是：
- 关闭开关
- 将标签更改为“下载”
- 告诉worker 2下载文件
- 检查开关是否打开 – 如果它是打开循环; 如果没有，那就去睡一秒钟。
- 回到上一步
- 将标签更改为“已完成”
- 此任务现已完成
工作人员1关闭开关并将以下任务放入工作人员2的收件箱中：
- 下载文件
- 告诉工人1打开开关
- 这个任务完成了
然后，工人1检查开关是否打开。 不是，所以工人1去睡觉。
Worker 2下载文件，然后在worker 1的收件箱中发送一条消息，说明：
- 打开开关
- 这个任务完成了
现在你明白为什么工人1会永远睡觉，对吧？ 该开关永远不会被翻转，因为翻转该开关是工作者1的工作，而工作人员1正在睡觉直到它被翻转 。 在当前任务完成之前，工作人员1不会查看其收件箱，并且当切换该开关之前，当前任务不会完成。
这给了我们一个解决方案的想法，但它不是一个好的解决方案。
解决这个问题的廉价，肮脏，危险和不明智的方法是使用“DoEvents”而不是“Sleep”。 这将任务更改为：
- 关闭开关
- 将标签更改为“下载”
- 告诉worker 2下载文件
- 检查开关是否打开 – 如果它是打开循环; 如果没有，请检查您的收件箱中的邮件，并执行您在那里找到的任何内容。
- 回到上一步
- 将标签更改为“已完成”
- 这个任务完成了
这解决了您的直接问题，但它引入了新问题。 我们现在不再拥有干净的工作流程; 一个收件箱任务可以生成第二个收件箱任务，这可以反过来产生第三个收件箱任务。 任务可以变得“重入”，其中一个任务最终会启动自身的第二个版本。 该解决方案不够优雅，并且适用于难以调试的情况。 理想情况下，您希望收件箱任务具有在旧的任务完成后启动新的属性， 而不是旧的任务仍有工作要做 。
对你的问题（如果你使用C＃5）更好的廉价和肮脏的修复是使用
await Task.Delay(1000);
而不是Sleep或DoEvents 。 这对工作流程进行了微妙的改变。 基本上它变成：
- 关闭开关
- 将标签更改为“下载”
- 告诉worker 2下载文件
- 检查开关是否打开
- 如果它打开然后将标签更改为“完成”; 这个任务完成了。
- 如果没有，那么请求工人3在一秒钟内向我发送任务; 这个任务完成了。
如果告诉worker 3向worker 1发送一个新任务，那么它发送的新任务是：
- 检查开关是否打开
- 如果它打开然后将标签更改为“完成”; 这个任务完成了。
- 如果没有，那么请求工人3在一秒钟内向我发送任务; 这个任务完成了。
你看到如何巧妙但正确地改变工作流程？ 现在，worker 1将标签更改为下载，向worker 2发送消息，检查交换机，向worker 3发送消息，然后返回其收件箱。 Worker 2执行下载并向worker发送消息1. Worker 1翻转开关并返回收件箱。 工作人员3向工作人员发送消息1.工作人员1检查交换机，将标签更改为完成，然后返回收件箱。
现在没有任务告诉您在收件箱中查找更多任务 。 每个收件箱任务按顺序处理：后到达的任务总是在较早到达的任务完成后启动。
然而，最好的解决方案是拥有一个版本的DownloadClientAsync ，它本身可以返回一个可以等待的任务。 不幸的是，这是无效的。 构建一个特殊版本的DownloadClientAsync，它返回一个可以等待的Task ，留作练习。 一旦你有了这样的帮助方法，那么代码变得微不足道; 你await那个任务。
您是否尝试将事件订阅代码从当前行转移到以下行之后：
webclient.DownloadFileAsync(new Uri(url), @"C:\Users\Krisz" + @"\" + filename); webclient.DownloadFileCompleted -= new AsyncCompletedEventHandler(webc_DownloadFileCompleted); //Also it's a good practice to unsubscribe to event once after we are out-of-scope. webclient.DownloadFileCompleted += new AsyncCompletedEventHandler(webc_DownloadFileCompleted);
当我这样做时，剩下的工作对我来说很好，否则你告诉屏幕是空白的。
我认为这与BackgroundWorker类非常相似，其中CompletedEventHandler在主线程上运行。 （目前我找不到任何确认结果的结果。）
这意味着您的主循环永远不会被中断 – 您必须退出主URL循环并返回到UI，然后才能触发webc_DownloadFileCompleted。
一种可能的解决方法是在您的第一个URL上运行SINGLE webclient异步下载，然后返回到主UI。 您的webc_DownloadFileCompleted函数可以重新发出下一个异步下载调用。