Task.WaitAll不等待其他异步方法

时间:2013-05-21 16:45:48

标签: c# .net asynchronous async-await portable-class-library

我使用我的可移植类库异步检索一些rss文章,该类使用Microsoft.Bcl库(它没有Task.WhenAll)。每篇文章都有一个RSS注释的URL,我需要异步检索。

下面的代码是我的库。我调用GetArticles()但它没有返回任何创建调用GetComments()以异步获取注释的任务列表。

我尝试在GetArticles中使用Task.WaitAll等待注释,但它不会阻止该线程。任何帮助将不胜感激。

    private const string ArticlesUri = "";

    public async Task<List<ArticleBrief>> GetArticles()
    {
        var results = new List<ArticleBrief>();
        try
        {
            var wfw = XNamespace.Get("http://wellformedweb.org/CommentAPI/");
            var media = XNamespace.Get("http://search.yahoo.com/mrss/");
            var dc = XNamespace.Get("http://purl.org/dc/elements/1.1/");

            var t = await WebHttpRequestAsync(ArticlesUri);
            StringReader stringReader = new StringReader(t);
            using (var xmlReader = System.Xml.XmlReader.Create(stringReader))
            {
                var doc = System.Xml.Linq.XDocument.Load(xmlReader);
                results = (from e in doc.Element("rss").Element("channel").Elements("item")
                           select
                               new ArticleBrief()
                               {
                                   Title = e.Element("title").Value,
                                   Description = e.Element("description").Value,
                                   Published = Convert.ToDateTime(e.Element("pubDate").Value),
                                   Url = e.Element("link").Value,
                                   CommentUri = e.Element(wfw + "commentRss").Value,
                                   ThumbnailUri = e.Element(media + "thumbnail").FirstAttribute.Value,
                                   Categories = GetCategoryElements(e.Elements("category")),
                                   Creator = e.Element(dc + "creator").Value
                               }).ToList();

            }

            var tasks = new Queue<Task>();
            foreach (var result in results)
            {
                tasks.Enqueue(
                    Task.Factory.StartNew(async ()=>
                        {
                            result.Comments = await GetComments(result.CommentUri);
                        }
                    ));
            }

            Task.WaitAll(tasks.ToArray());
        }
        catch (Exception ex)
        {
            // should do some other
            // logging here. for now pass off
            // exception to callback on UI
            throw ex;
        }

        return results;
    }

    public async Task<List<Comment>> GetComments(string uri)
    {
        var results = new List<Comment>();
        try
        {
            var wfw = XNamespace.Get("http://wellformedweb.org/CommentAPI/");
            var media = XNamespace.Get("http://search.yahoo.com/mrss/");
            var dc = XNamespace.Get("http://purl.org/dc/elements/1.1/");

            var t = await WebHttpRequestAsync(uri);
            StringReader stringReader = new StringReader(t);
            using (var xmlReader = System.Xml.XmlReader.Create(stringReader))
            {
                var doc = System.Xml.Linq.XDocument.Load(xmlReader);
                results = (from e in doc.Element("rss").Element("channel").Elements("item")
                           select
                               new Comment()
                               {
                                   Description = e.Element("description").Value,
                                   Published = Convert.ToDateTime(e.Element("pubDate").Value),
                                   Url = e.Element("link").Value,
                                   Creator = e.Element(dc + "creator").Value
                               }).ToList();
            }
        }
        catch (Exception ex)
        {
            // should do some other
            // logging here. for now pass off
            // exception to callback on UI
            throw ex;
        }

        return results;
    }

    private static async Task<string> WebHttpRequestAsync(string url)
    {
        //TODO: look into getting 
        var request = WebRequest.Create(url);
        request.Method = "GET";

        var response = await request.GetResponseAsync();
        return ReadStreamFromResponse(response);
    }

    private static string ReadStreamFromResponse(WebResponse response)
    {
        using (Stream responseStream = response.GetResponseStream())
        using (StreamReader sr = new StreamReader(responseStream))
        {
            string strContent = sr.ReadToEnd();
            return strContent;
        }
    }

    private List<string> GetCategoryElements(IEnumerable<XElement> categories)
    {
        var listOfCategories = new List<string>();

        foreach (var category in categories)
        {
            listOfCategories.Add(category.Value);
        }

        return listOfCategories;
    }

从解决方案更新了代码,只是在Enqueue方法上添加了.UnWrap():

            var tasks = new Queue<Task>();
            foreach (var result in results)
            {
                tasks.Enqueue(
                    Task.Factory.StartNew(async ()=>
                        {
                            result.Comments = await GetComments(result.CommentUri);
                        }
                    ).Unwrap());
            }

            Task.WaitAll(tasks.ToArray());

2 个答案:

答案 0 :(得分:4)

正在等待。问题是你正在创建一个Task来创建另一个任务(即StartNew正在返回一个Task<Task>,而你只是在外Task上等待很快完成({1}}它在内部任务完成之前完成))。

问题将是:

  • 你真的想要那个内心的任务吗?
    • 如果是,那么您可以使用Task.Unwrap来获取代表内部和外部Task完成的代理任务,并将其用于等待。
    • 如果不是,那么您可以在StartNew中删除async / await的使用,这样就没有内部任务(我认为这是首选,但不清楚为什么你需要内在的任务。)
  • 您是否真的需要在异步任务上执行同步Wait?阅读Stephen Cleary的一些博客:http://blog.stephencleary.com/2012/02/async-unit-tests-part-1-wrong-way.html

顺便说一句,如果您不使用C#5,请注意关闭foreach变量result请参阅

答案 1 :(得分:2)

Microsoft.Bcl.Async中,我们无法向Task添加任何静态方法。但是,您可以在TaskEx上找到大多数方法,例如,TaskEx.WhenAll()确实存在。