3

概念：我正在制作一个从给定 URL 下载文件的 C# 应用程序。文本框，添加的 url，文件下载，每个事件都以正确的方式发生。

我正在尝试重新创建此程序以逐个下载多个文件。我有一个带有一个 url/line 的文本框，解析正确，我将所有链接放在一个字符串数组中，该数组放在文本框中。然后它开始异步下载，我想让它只下载一个，所以我在foreach循环中做了一个while循环，因为我不想在当前一个url完成下载之前转到下一个url。

问题是：我进入了一个无限循环（虽然我之前做了这个工作（idk how），如果我在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;
}

我研究了这个线程：WebClient.DownloadFileAsync - 一次下载一个文件，但我无法让它以这种方式工作。（也许我误解了什么？）

有人可以给我一些提示，我做错了什么？我从来没有真正使用过事件，所以遇到错误只是时间问题。非常感谢每一点帮助，这个程序对我来说将是一个有用的程序。

4

3 回答 3

9

好的，首先，了解为什么您的代码现在不起作用。

想象一下，办公室里有两个人。两者都有“盒子”。他们的工作流程是：他们检查收件箱。如果收件箱中有任务，他们会执行任务直到完成，然后再次检查他们的收件箱，重复。

工人 1 在他们的收件箱中收到一条消息，说明您的下一个任务是：

  • 关掉开关
  • 将标签更改为“正在下载”
  • 告诉工人 2 下载文件
  • 检查开关是否打开——如果打开则跳出循环；如果没有，那就睡一秒钟。
  • 返回上一步
  • 将标签更改为“完成”
  • 此任务现已完成

工人 1 关闭开关并将以下任务放入工人 2 的收件箱：

  • 下载文件
  • 告诉工人 1 打开开关
  • 这个任务完成了

工人 1 然后检查开关是否打开。不是，所以工人 1 去睡觉了。

工人 2 下载文件，然后在工人 1 的收件箱中放入一条消息，内容为：

  • 打开开关
  • 这个任务完成了

现在你明白为什么工人 1 永远睡觉了，对吧？该开关永远不会被翻转，因为工人 1 的工作是翻转该开关，而工人 1在它被翻转之前一直在睡觉。工作人员 1 在当前任务完成之前不会查看他们的收件箱，并且当前任务在切换该开关之前不会完成。

这给了我们一个解决方案的想法，但这不是一个好的解决方案。

解决这个问题的廉价、肮脏、危险和不明智的方法是使用“DoEvents”而不是“Sleep”。这将任务更改为：

  • 关掉开关
  • 将标签更改为“正在下载”
  • 告诉工人 2 下载文件
  • 检查开关是否打开——如果打开则跳出循环；如果没有，请检查您的收件箱中的消息并执行您在其中找到的任何操作。
  • 返回上一步
  • 将标签更改为“完成”
  • 这个任务完成了

这解决了您的直接问题，但它引入了新问题。我们现在不再有一个干净的工作流程；一个收件箱任务可以产生第二个收件箱任务，而后者又可以产生第三个收件箱任务。任务可以成为“重入”，其中一个任务最终会启动其自身的第二个版本。这个解决方案是不优雅的，并且会导致难以调试的情况。理想情况下，您希望收件箱任务具有在旧任务完成后启动新任务的属性，而不是在旧任务仍有工作要做时启动。

对于您的问题（如果您使用的是 C# 5），一个更好的廉价和肮脏的解决方法是使用

await Task.Delay(1000);

而不是Sleepor DoEvents。这对工作流程进行了微妙的改变。基本上它变成：

  • 关掉开关
  • 将标签更改为“正在下载”
  • 告诉工人 2 下载文件
  • 检查开关是否打开
  • 如果它打开，则将标签更改为“完成”；这个任务完成了。
  • 如果没有，请让工人 3 在一秒钟内向我发送任务；这个任务完成了。

如果工人 3 被告知要向工人 1 发送一个新任务，那么它发送的新任务是：

  • 检查开关是否打开
  • 如果它打开，则将标签更改为“完成”；这个任务完成了。
  • 如果没有，请让工人 3 在一秒钟内向我发送任务；这个任务完成了。

您看到这如何巧妙而正确地改变了工作流程？现在工人 1 将标签更改为下载，向工人 2 发送消息，检查开关，向工人 3 发送消息，然后返回其收件箱。工作人员 2 进行下载并向工作人员 1 发送消息。工作人员 1 拨动开关并返回收件箱。工人 3 向工人 1 发送消息。工人 1 检查开关，将标签更改为完成，然后返回收件箱。

现在没有任务告诉您在收件箱中查找更多任务。每个收件箱任务都按顺序处理：迟到的任务总是在先到的任务完成后开始。

然而，最好的解决方案是有一个版本DownloadClientAsync本身可以返回一个可以等待的任务。不幸的是，它是返回无效的。构建一个返回Task可以等待的特殊版本的 DownloadClientAsync 留作练习。一旦有了这样的辅助方法，代码就变得微不足道了；你只是await那个任务。

于 2013-03-07T16:33:33.197 回答
2

您是否尝试将事件订阅代码从当前行转移到以下行：

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);

当我这样做时，其余的对我来说都很好，否则就像你说的那样屏幕是空白的。

于 2013-03-07T16:42:52.853 回答
1

我认为这与 BackgroundWorker 类非常相似，其中 CompletedEventHandler 在主线程上运行。（我目前找不到任何证实这一点的结果。）

这意味着您的主循环将永远不会被中断 - 您必须退出主 URL 循环并返回 UI，然后才能触发 webc_DownloadFileCompleted。

一种可能的解决方法是在您的第一个 URL 上运行 SINGLE webclient 异步下载，然后返回主 UI。您的 webc_DownloadFileCompleted 函数可以重新发出下一个异步下载调用。

于 2013-03-07T16:41:27.817 回答