Question

我正在尝试启动下载网页内容的异步任务（在.NET 4.5上），但不知怎的，这个任务永远不会完成。

我的PageDownloader课程：

using System.Net;
using System.Text;
using System.IO;
using System.Net.Http;
using System.Threading.Tasks;
using System;

namespace ParserConsole.WebClient
{
public class PageDownloader
{
    private System.Net.Http.HttpClient _client;

    public PageDownloader()
        : this(Encoding.UTF8) { }

    private Encoding _encoding;

    public PageDownloader(Encoding encoding)
    {
        _encoding = encoding;
        _client = new HttpClient() { Timeout = TimeSpan.FromSeconds(10)};
    }

    private HttpRequestMessage _request;
    private HttpResponseMessage _response;
    private string _responseString;

    public string GetPageData(string link)
    {
        _request = new HttpRequestMessage(HttpMethod.Get, link);
        _request.Headers.Add("User-Agent", "Chrome/21.0.1180.89");
        _request.Headers.Add("Accept", "text/html");


        GetResponse().Wait();
        GetStringFromResponse().Wait();
        return _responseString;            
    }

    private async Task<HttpResponseMessage> GetResponse() {
        return _response = await _client.GetAsync(_request.RequestUri);
    }

    private async Task<string> GetStringFromResponse() {
        return _responseString = await _response.Content.ReadAsStringAsync();
    }

}
}

我开始通过调用

下载页面

new PageDownloader().GetPageData(url);

当我尝试调试代码时，一切都很好，直到GetResponse().Wait()。但不知怎的GetResponse()任务永远不会完成 - 下一行的断点永远不会到达。我没有例外，应用程序继续运行。有什么建议吗？

Answer 1

这是您启动async操作然后阻止返回任务时获得的标准死锁条件。

Here是博客文章讨论的主题。

基本上，await调用确保它连接任务的延续将在你最初的上下文中运行（这是非常有用的）但是因为你在同一个地方调用Wait上下文它是阻塞的，所以延续从不运行，并且延续需要运行等待结束。经典僵局。

关于修复;通常它意味着你不应该在异步操作上做阻塞等待;这与整个系统的设计相反。你应该“一直异步”。在这种情况下，这意味着GetPageData应该返回Task<string>而不是string，而不是等待返回任务的其他操作，您应该await对它们进行操作

现在，已经说过，是在异步操作上进行阻塞等待的方法，而不会发生死锁。虽然可以做到，但它首先实际上违背了使用async / await的目的。使用该系统的主要优点是主要上下文不被阻止;当你阻止整个优势消失时，你也可以一直使用阻塞代码。 async / await实际上更像是一种全有或全无的范例。

以下是我将如何构建该类：

public class PageDownloader
{
    private System.Net.Http.HttpClient _client;
    private Encoding _encoding;

    public PageDownloader()
        : this(Encoding.UTF8) { }

    public PageDownloader(Encoding encoding)
    {
        _encoding = encoding;
        _client = new HttpClient() { Timeout = TimeSpan.FromSeconds(10) };
    }

    public async Task<string> GetPageData(string link)
    {
        HttpRequestMessage request = new HttpRequestMessage(HttpMethod.Get, link);
        request.Headers.Add("User-Agent", "Chrome/21.0.1180.89");
        request.Headers.Add("Accept", "text/html");

        HttpResponseMessage response = await _client.GetAsync(request.RequestUri);

        return await response.Content.ReadAsStringAsync(); ;
    }
}

Answer 2

如果你想拥有这样的功能，为什么不这样做呢？

public string GetPageData(string link)
{
    _request = new HttpRequestMessage(HttpMethod.Get, link);
    _request.Headers.Add("User-Agent", "Chrome/21.0.1180.89");
    _request.Headers.Add("Accept", "text/html");


    var readTask = _client.GetStringAsync(link);
    readTask.Wait();
    return readTask.Result;
}

最好将Task一直返回并在调用代码中使用async / await处理它。

public Task<string> GetPageData(string link)
{
    _request = new HttpRequestMessage(HttpMethod.Get, link);
    _request.Headers.Add("User-Agent", "Chrome/21.0.1180.89");
    _request.Headers.Add("Accept", "text/html");


    return _client.GetStringAsync(link);
}

异步任务没有结束

2 个答案: