Еогеасп заявление дает ошибку

Question

Я использую следующий код:

private string covertRss(string url) 
    { 
     var s = RssReader.Read(url); 
     StringBuilder sb = new StringBuilder(); 
     foreach (RssNews rs in s) //ERROR LINE 
     { 
      sb.AppendLine(rs.Title); 
      sb.AppendLine(rs.PublicationDate); 
      sb.AppendLine(rs.Description); 
     } 

     return sb.ToString(); 
    } 

Я получаю сообщение об ошибке:

Error 1 foreach statement cannot operate on variables of type 'System.Threading.Tasks.Task(System.Collections.Generic.List(Cricket.MainPage.RssNews))' because 'System.Threading.Tasks.Task(System.Collections.Generic.List(Cricket.MainPage.RssNews))' does not contain a public definition for 'GetEnumerator'

Класс RssNews является:

public class RssNews 
    { 
     public string Title; 
     public string PublicationDate; 
     public string Description; 

    } 

Что код должен Я добавлю, что ошибка удалена, и цель кода не компилируется? Спасибо заранее!

Код для RssReader.Read()

public class RssReader 
    { 
     public static async System.Threading.Tasks.Task<List<RssNews>> Read(string url) 
     { 
      HttpClient httpClient = new HttpClient(); 

      string result = await httpClient.GetStringAsync(url); 

      XDocument document = XDocument.Parse(result); 

      return (from descendant in document.Descendants("item") 
        select new RssNews() 
        { 
         Description = descendant.Element("description").Value, 
         Title = descendant.Element("title").Value, 
         PublicationDate = descendant.Element("pubDate").Value 
        }).ToList(); 
     } 
    } 

Answer 1

Я думаю, что вам не хватает в await заявление.

Это s имеет тип Task<List<RssNews>>.

Вы либо должны это

var s = await RssReader.Read(url); 

или

var s = RssReader.Read(url).Result;//this is blocking call 

Конечно, при использовании await необходимо отметить метод async, а также.

Вот как вы идете

private async Task<string> covertRss(string url) 
{ 
    var s = await RssReader.Read(url); 
    StringBuilder sb = new StringBuilder(); 
    foreach (RssNews rs in s) //ERROR LINE 
    { 
     sb.AppendLine(rs.Title); 
     sb.AppendLine(rs.PublicationDate); 
     sb.AppendLine(rs.Description); 
    } 

    return sb.ToString(); 
} 

Answer 2

Вы используете саму задачу в цикле. Вам необходимо использовать свойство «Результат»

foreach (RssNews rs in s.Result) 

поскольку из-за исключения представляется список, который будет возвращен.

Answer 3

Вы должны использовать await:

foreach (RssNews rs in await s) 

или:

var s = await RssReader.Read(url); 

ли не использование Result; если вы это сделаете, вы можете легко cause a deadlock, что я опишу в своем блоге.

В качестве примечания к сведению, я рекомендую вам прочитать и следовать рекомендациям в Task-based Asynchronous Pattern documentation. Если вы это сделаете, вы обнаружите, что ваш метод Read должен быть назван ReadAsync, который дает ваш код вызова сильный намек, что он должен использовать await:

var s = await RssReader.ReadAsync(url);