Linq groupBy Перегруппировка в список <> простым/чистым способом

Question

Я создал рефакторинг при этом методе. прежде чем у нас было много циклов и фиктивный код. Это как можно дальше, но я думаю, что я мог бы намного проще.

Я думаю, что я мог бы направить из группы * &, выбрав правильный список.

Возможно ли это? или у меня должен быть этот цикл?

private IEnumerable<ESMatchDate> groupBydate(IEnumerable<ESMatch> matches) 
{ 
    List<ESMatchDate> lMatchDates = new List<ESMatchDate>(); 

    if (null != matches && matches.Any()) 
    { 
     var dates = from ESMatch in matches 
       group ESMatch by ESMatch.BeginDate.ToShortDateString() into newGroup 
       orderby Convert.ToDateTime(newGroup.Key) 
       select newGroup; 

     foreach (var d in dates) 
      lMatchDates.Add(new ESMatchDate 
       { Date = Convert.ToDateTime(d.Key), Matches = d.ToList()}); 

    } 
    else 
     lMatchDates = null; 

    return lMatchDates; 
} 

Answer 1

Просто сделать проекцию в select:

private IEnumerable<ESMatchDate> groupBydate(IEnumerable<ESMatch> matches) 
{ 
    if (matches == null || !matches.Any()) 
    { 
     return null; 
    } 

    return (from ESMatch in matches 
      group ESMatch by ESMatch.BeginDate.Date into newGroup 
      orderby Convert.ToDateTime(newGroup.Key) 
      select new ESMatchDate 
      { 
       Date = newGroup.Key, 
       Matches = newGroup.ToList() 
      }).ToList(); 
} 

Answer 2

Ты так близко, но вам не нужно вручную Add к списку - LINQ может сделать все это для вас

private IEnumerable<ESMatchDate> groupBydate(IEnumerable<ESMatch> matches) 
{ 

    if (null == matches || !matches.Any()) 
    { 
     return null; 
    } 
    return matches.GroupBy(x => x.BeginDate.Date) 
        .OrderBy(x => x.Key) 
        .Select(x => new ESMatchDate{ Date = x.Key, Matches = x.ToList() }) 
        .ToList(); 
} 

Обратите внимание, что это аналогично предыдущему ответу, просто используя синтаксис Lamda вместо сравнения. Я лично предпочитаю этот синтаксис, но это вопрос вкуса.