Task.wait 没有像我想象的那样工作

Question

我正在尝试下载文件，等待文件完成下载，然后再读取文件。我有以下方法可以做到这一点：

private async Task startDownload(string link, string savePath)
{      
        WebClient client = new WebClient();
        client.DownloadProgressChanged += new DownloadProgressChangedEventHandler(client_DownloadProgressChanged);
        client.DownloadFileCompleted += new AsyncCompletedEventHandler(client_DownloadFileCompleted);
        await client.DownloadFileTaskAsync(new Uri(link), savePath);              
}

private void checkUpdateButton_Click(object sender, EventArgs e)
{           
    Task task = Task.Factory.StartNew(() => startDownload(versionLink, versionSaveTo));
    task.Wait();

    if (task.IsCompleted)
    {
        checkVersion();
    }
}

checkVersion() 方法读取下载的文件。这是抛出一个IOException 说该文件正在被其他东西使用并且无法读取。我认为拥有task.Wait 会阻止方法的其余部分在任务完成之前执行？

Answer 1

Task.Wait 将阻塞当前线程（在本例中为 UI 线程）并等待任务完成。在这种情况下，任务完成时出现错误，因此 Task.Wait 将抛出包裹在 AggregateException 中的错误。

正如其他人所说，您应该使用await 而不是Wait。此外，DownloadFileCompleted 没有意义，因为您使用的是DownloadFileTaskAsync（而不是DownloadFileAsync）；并且StartNew 是不必要的，因为下载是异步的。

哦，让我们处理 WebClient 并确保我们的命名约定遵循 Task-based Asynchronous Pattern。

private async Task startDownloadAsync(string link, string savePath)
{      
  using (var client = new WebClient())
  {
    client.DownloadProgressChanged += new DownloadProgressChangedEventHandler(client_DownloadProgressChanged);
    await client.DownloadFileTaskAsync(new Uri(link), savePath);              
  }
}

private async void checkUpdateButton_Click(object sender, EventArgs e)
{           
  await startDownloadAsync(versionLink, versionSaveTo);
  checkVersion();
}

Answer 2

函数 startDownload 已经是异步的，所以它将启动任务并立即返回。您可以在调用 checkVersion() 之前使用 ContinueWith 确保任务完成。

    private void checkUpdateButton_Click(object sender, EventArgs e)
    {
        var task = startDownload(versionLink, versionSaveTo);
        task.ContinueWith((x) => checkVersion());
    }

正如 Servy 指出的那样，另一种方法是在 Click 事件中使用 async/await。

    private async void checkUpdateButton_Click(object sender, EventArgs e)
    {
        await startDownload(versionLink, versionSaveTo);
        checkVersion();
    }

Answer 3

您需要等待您的 Task.Factory.StartNew(...) 调用，这样它就不会阻塞 UI 线程。

private async void button1_Click(object sender, EventArgs e)
{
    Task task = await Task.Factory.StartNew(() => startDownload("http://www.zwaldtransport.com/images/placeholders/placeholder1.jpg", "" + "sd.jpg"));
}

private async Task startDownload(string link, string savePath)
{
    WebClient client = new WebClient();
    client.DownloadProgressChanged += Client_DownloadProgressChanged;
    client.DownloadFileCompleted += new AsyncCompletedEventHandler(client_DownloadFileCompleted);
    await client.DownloadFileTaskAsync(new Uri(link), savePath);
}

private void client_DownloadFileCompleted(object sender, AsyncCompletedEventArgs e)
{
            checkVersion();
    Console.WriteLine("Done, unless error or cancelled.");
}

private void Client_DownloadProgressChanged(object sender, DownloadProgressChangedEventArgs e)
{
    Console.WriteLine("Progress changed.");
}

图片占位符由 Google 图片和其他一些网站提供。