Linq, IQueryable получить наиболее часто используемый объект элемента

Question

Моя основная проблема заключается в понимании IGrouping<int, SubForm>. Мне нужен самый частый объект (MyClass). Это код, у меня сейчас:

var subForm = 
      classes.GroupBy(c => c.SubFormId) 
      .OrderByDescending(sf => sf.Count()) 
      .FirstOrDefault(); 

Это возвращает IGrouping<int, SubForm>. Чтобы получить фактический объект, я должен сделать еще один FirstOrDefault(), но компилятор показывает, что может быть пустое исключение. Это код для получения фактической подформы, можно ли это сделать лучше?

var subForm = 
      classes.GroupBy(c => c.SubFormId) 
      .OrderByDescending(sf => sf.Count()) 
      .FirstOrDefault().FirstOrDefault().SubForm; 

Answer 1

Вы можете просто проверить, чтобы убедиться, что есть по крайней мере один пункт в вашем classes коллекции (так он гарантированно есть по крайней мере одна группа) либо в конце или в начале:

if(classes.Any()) 
{ 
    var subForm = classes.GroupBy(c => c.SubFormId) 
         .OrderByDescending(sf => sf.Count()) 
         .First().First().SubForm; 

} 

Или:

var topGroup = classes.GroupBy(c => c.SubFormId) 
         .OrderByDescending(sf => sf.Count()) 
         .FirstOrDefault(); 

if(topGroup!=null) 
    subForm = item.First().SubForm; 

Answer 2

Вы могли бы просто использовать First. Это исключало бы исключение, если ничего не найдено.

Answer 3

var subForm = classes.GroupBy(c => c.SubFormId) 
    .OrderByDescending(sf => sf.Count()) 
    .FirstOrDefault(); 

return subForm == null ? default(SubForm) : subForm.Select(s => s.SubForm); 

Answer 4

Вы пропускаете выберите:

var subForm = (
    from c in classes 
    group c by c.SubFormId into g 
    select g.Key) 
    .FirstOrDefault(); 

Answer 5

Я думаю, что вы хотите:

var sfGroup = classes.GroupBy(c => c.SubFormId) 
      .OrderByDescending(sf => sf.Count())     
      .FirstOrDefault(); 

    int count = sfGroup.Count(); 
    MyClass subForm = sfGroup.FirstOrDefault();