Группировать в LINQ

1063

Давайте предположим, что у нас есть такой класс:

class Person { 
    internal int PersonID; 
    internal string car; 
}

Теперь у меня есть список этого класса: List<Person> persons;

Теперь этот список может иметь несколько экземпляров с одинаковыми PersonID, например:

persons[0] = new Person { PersonID = 1, car = "Ferrari" }; 
persons[1] = new Person { PersonID = 1, car = "BMW"     }; 
persons[2] = new Person { PersonID = 2, car = "Audi"    }; 

Есть ли способ, которым я могу сгруппироваться PersonIDи получить список всех автомобилей, которые у него есть?

Например, ожидаемый результат будет

class Result { 
   int PersonID;
   List<string> cars; 
}

Поэтому после группировки я получу:

results[0].PersonID = 1; 
List<string> cars = results[0].cars; 

result[1].PersonID = 2; 
List<string> cars = result[1].cars;

Из того, что я сделал до сих пор:

var results = from p in persons
              group p by p.PersonID into g
              select new { PersonID = g.Key, // this is where I am not sure what to do

Может ли кто-нибудь, пожалуйста, указать мне в правильном направлении?

test123
источник
1
Есть еще один пример, включающий Countи Sumздесь stackoverflow.com/questions/3414080/…
Разработчик
@ Мартин Келлман: я согласен с Крисом Уолшем. Скорее всего, приложение, которое имеет класс «Таблица (-и)» OP, уже будет иметь «нормальный» «класс (-ы)» (класс) «Лицо (-ы)», которое имеет тип usu. Свойства / столбцы (т.е. имя, пол, DOB). Класс (таблица) «Персона (ов)» ОП должен быть дочерним классом (таблицей) класса («обычного») «персонажа (ов)» (т. Е. Класса «OrderItem (s)» (таблица). ) против класса "Заказ (ы)" (таблица)). ОП, скорее всего, не использовал фактическое имя, которое использовал бы, если бы он находился в той же области, что и его класс «Таблица» (человек) «Человек», и / или, возможно, упростил его для этого поста.
Том

Ответы:

1736

Абсолютно - вы в основном хотите:

var results = from p in persons
              group p.car by p.PersonId into g
              select new { PersonId = g.Key, Cars = g.ToList() };

Или как выражение без запроса:

var results = persons.GroupBy(
    p => p.PersonId, 
    p => p.car,
    (key, g) => new { PersonId = key, Cars = g.ToList() });

По существу, содержимое группы (если рассматривать ее как IEnumerable<T>) - это последовательность значений, которые были в проекции ( p.carв данном случае) для данного ключа.

Подробнее о том GroupBy, как работает, смотрите мой пост Edulinq на эту тему .

(Я переименовал PersonIDв PersonIdв выше, чтобы следовать .NET соглашения о присвоении имен .)

В качестве альтернативы вы можете использовать Lookup:

var carsByPersonId = persons.ToLookup(p => p.PersonId, p => p.car);

Вы можете легко получить автомобили для каждого человека:

// This will be an empty sequence for any personId not in the lookup
var carsForPerson = carsByPersonId[personId];
Джон Скит
источник
11
@jon Скит, что если я захочу добавить еще одно свойство, например name
user123456
22
@ Мохаммед: Затем вы включаете это в анонимный тип.
Джон Скит
7
@ user123456 Вот хорошее объяснение группирования, оно также включает пример группировки по составному ключу: Как: Результаты группового запроса (Руководство по программированию в C #)
Матье Дьепман,
14
@ Мохаммед, вы можете сделать что-то подобное, .GroupBy(p => new {p.Id, p.Name}, p => p, (key, g) => new { PersonId = key.Id, PersonName = key.Name, PersonCount = g.Count()})и вы получите всех людей, которые встречаются с идентификатором, именем и числом вхождений для каждого человека.
Крис
11
@kame: Я сознательно следовал соглашениям об именах .NET, по сути, исправляя имена ОП. Выясню это в ответе.
Джон Скит
52
var results = from p in persons
              group p by p.PersonID into g
              select new { PersonID = g.Key,
                           /**/car = g.Select(g=>g.car).FirstOrDefault()/**/}
Таллат
источник
37
var results = from p in persons
              group p by p.PersonID into g
              select new { PersonID = g.Key, Cars = g.Select(m => m.car) };
Йогендра Паудьял
источник
32

Вы также можете попробовать это:

var results= persons.GroupBy(n => new { n.PersonId, n.car})
                .Select(g => new {
                               g.Key.PersonId,
                               g.Key.car)}).ToList();
Шуво Саркер
источник
Это неправильно возвращает тот же список не группирует
Викас Гупта
Это работа, я думаю, что чего-то не хватает, поэтому это не работает в вашем коде.
Шуво Саркер
29

пытаться

persons.GroupBy(x => x.PersonId).Select(x => x)

или

чтобы проверить, повторяется ли кто-либо в вашем списке, попробуйте

persons.GroupBy(x => x.PersonId).Where(x => x.Count() > 1).Any(x => x)
Код первый
источник
13

Я создал пример рабочего кода с синтаксисом запроса и синтаксисом метода. Я надеюсь, что это помогает другим :)

Вы также можете запустить код на .Net Fiddle здесь:

using System;
using System.Linq;
using System.Collections.Generic;

class Person
{ 
    public int PersonId; 
    public string car  ; 
}

class Result
{ 
   public int PersonId;
   public List<string> Cars; 
}

public class Program
{
    public static void Main()
    {
        List<Person> persons = new List<Person>()
        {
            new Person { PersonId = 1, car = "Ferrari" },
            new Person { PersonId = 1, car = "BMW" },
            new Person { PersonId = 2, car = "Audi"}
        };

        //With Query Syntax

        List<Result> results1 = (
            from p in persons
            group p by p.PersonId into g
            select new Result()
                {
                    PersonId = g.Key, 
                    Cars = g.Select(c => c.car).ToList()
                }
            ).ToList();

        foreach (Result item in results1)
        {
            Console.WriteLine(item.PersonId);
            foreach(string car in item.Cars)
            {
                Console.WriteLine(car);
            }
        }

        Console.WriteLine("-----------");

        //Method Syntax

        List<Result> results2 = persons
            .GroupBy(p => p.PersonId, 
                     (k, c) => new Result()
                             {
                                 PersonId = k,
                                 Cars = c.Select(cs => cs.car).ToList()
                             }
                    ).ToList();

        foreach (Result item in results2)
        {
            Console.WriteLine(item.PersonId);
            foreach(string car in item.Cars)
            {
                Console.WriteLine(car);
            }
        }
    }
}

Вот результат:

1
Ferrari
БМВ
2
Audi
-----------
1
Ferrari
БМВ
2
Audi

Речев Йылдыз
источник
Пожалуйста, объясните, что ваш код делает ради. Это всего лишь кодовый ответ, который является почти неправильным ответом.
V.7
2

Попробуй это :

var results= persons.GroupBy(n => n.PersonId)
            .Select(g => new {
                           PersonId=g.Key,
                           Cars=g.Select(p=>p.car).ToList())}).ToList();

Но с точки зрения производительности следующая практика лучше и более оптимизирована в использовании памяти (когда наш массив содержит гораздо больше элементов, таких как миллионы):

var carDic=new Dictionary<int,List<string>>();
for(int i=0;i<persons.length;i++)
{
   var person=persons[i];
   if(carDic.ContainsKey(person.PersonId))
   {
        carDic[person.PersonId].Add(person.car);
   }
   else
   {
        carDic[person.PersonId]=new List<string>(){person.car};
   }
}
//returns the list of cars for PersonId 1
var carList=carDic[1];
akazemis
источник
4
g.Key.PersonId? g.SelectMany?? Вы явно не пробовали это.
Герт Арнольд
Вы пишете, я отредактировал некоторые коды кодов в нем и не проверял это. Моим главным пунктом была вторая часть. Но в любом случае спасибо за ваше внимание. Было слишком поздно редактировать этот код, когда я понял, что это неправильно. поэтому g.Key заменяет g.Key.PersonId и выбирает вместо SelectMany! так грязно извините :)))
akazemis
2
@akazemis: я на самом деле пытался создать (использовать термины, эквивалентные домену ОП) SortedDictionary <PersonIdInt, SortedDictionary <CarNameString, CarInfoClass>>. Самое близкое, что я мог получить, используя LINQ IEnumerable <IGrouping <PersonIdInt, Dictionary <CarNameString, PersonIdCarNameXrefClass>>>. Я закончил, используя ваш forметод цикла, который, кстати, был в 2 раза быстрее. Кроме того, я бы использовал: а) foreachпротив forи б) TryGetValueпротив ContainsKey(оба для принципа СУХОЙ - в коде и во время выполнения).
Том
1

Альтернативный способ сделать это может быть выбран отдельно PersonIdи групповое объединение с persons:

var result = 
    from id in persons.Select(x => x.PersonId).Distinct()
    join p2 in persons on id equals p2.PersonId into gr // apply group join here
    select new 
    {
        PersonId = id,
        Cars = gr.Select(x => x.Car).ToList(),
    };

Или то же самое с беглым синтаксисом API:

var result = persons.Select(x => x.PersonId).Distinct()
    .GroupJoin(persons, id => id, p => p.PersonId, (id, gr) => new
    {
        PersonId = id,
        Cars = gr.Select(x => x.Car).ToList(),
    });

GroupJoin создает список записей в первом списке (список PersonIdв нашем случае), каждая из которых имеет группу соединенных записей во втором списке (список persons).

Дмитрий Степанов
источник
1

В следующем примере метод GroupBy используется для возврата объектов, сгруппированных по PersonID.

var results = persons.GroupBy(x => x.PersonID)
              .Select(x => (PersonID: x.Key, Cars: x.Select(p => p.car).ToList())
              ).ToList();

Или

 var results = persons.GroupBy(
               person => person.PersonID,
               (key, groupPerson) => (PersonID: key, Cars: groupPerson.Select(x => x.car).ToList()));

Или

 var results = from person in persons
               group person by person.PersonID into groupPerson
               select (PersonID: groupPerson.Key, Cars: groupPerson.Select(x => x.car).ToList());

Или вы можете использовать ToLookup, в основном ToLookupиспользует EqualityComparer<TKey>.Default для сравнения ключей и делать то, что вы должны делать вручную при использовании группировки по и словарю. я думаю, что это оправдано

 ILookup<int, string> results = persons.ToLookup(
            person => person.PersonID,
            person => person.car);
Реза Дженаби
источник
0

Сначала установите ключевое поле. Затем включите другие поля:

var results = 
    persons
    .GroupBy(n => n.PersonId)
    .Select(r => new Result {PersonID = r.Key, Cars = r.ToList() })
    .ToList()
user3474287
источник
Некомментированный ответ / код - это не то, как работает StackOverflow ...
V.7