LINQ Group By w obiekcie słownika

161

Próbuję użyć LINQ, aby utworzyć Dictionary<string, List<CustomObject>>z List<CustomObject>. Mogę to uruchomić używając "var", ale nie chcę używać typów anonimowych. Oto co mam

var x = (from CustomObject o in ListOfCustomObjects
      group o by o.PropertyName into t
      select t.ToList());

Kiedyś próbowałem również użyć Cast<>()z biblioteki LINQ x, ale pojawiają się problemy z kompilacją, które powodują, że jest to nieprawidłowe rzutowanie.

linq dictionary

— Atari2600
źródło

Co jeśli spróbujesz var x = (z CustomObject o w grupie ListOfCustomObjects o przez o.PropertyName do t select t) .ToList ();

— esastincy

44

Czy jest jakiś powód, dla którego musisz to zrobić, zamiast używać ToLookup, który jest przeznaczony do tego?

— Jon Skeet

1

Jon, czy mógłbyś opublikować przykład, jak działa ToLookup w tej sytuacji? Nie znam tej metody LINQ.

— Atari2600

8

@JonSkeet Jesteś niesamowity! (To znaczy, wszyscy już to wiedzieli, ale nadal.) Powodem, dla którego nie planowałem używać ToLookup, było to, że nigdy o tym nie słyszałem, aż do teraz. Teraz wiem!

— neminem

1

Dla zachowania kompletności, używanie varnie jest typem „anonimowym”, lecz typem „niejawnym”. Typy anonimowe to nowe klasy utworzone przez kompilator do obsługi konstrukcji new { thing = "stuff" };. Typy niejawne są istniejącymi klasami, varsą po prostu wygodnym sposobem na odwołanie się do nich, gdy zmienna jest przypisywana natychmiastowo, typ zmiennej można wywnioskować z typu przypisywanego do niej obiektu. Możesz nawet niejawnie wpisać zmienną odwołującą się do typu anonimowego, np .:var a = new { thing = "stuff" };

— Michael Blackburn

351

Dictionary<string, List<CustomObject>> myDictionary = ListOfCustomObjects
    .GroupBy(o => o.PropertyName)
    .ToDictionary(g => g.Key, g => g.ToList());

— Yuriy Faktorovich
źródło

6

Jeśli nie potrzebujesz właściwości z 'CustomObject' jako wartości listy (nie pokazanej w tej odpowiedzi), warto sprawdzić jego kodowość komentarz Jona Skeeta do pytania polecającego ToLookup ().

— Shaun

3

jest to sposób na zrobienie tego, jeśli pożądany jest niezmienny wynik. ToLookup jest niezmienna.

— Amit

1

Moje 2 centy (tylko dlatego, że walczyłem przez godzinę :)): podczas grupowania według nieruchomości upewnij się, że właściwość MA wartość! W przeciwnym razie metoda Todict nie generuje klucza (dla właściwości String przynajmniej ...) :)

— dba

.GroupBy(o => o.PropertyName).ToDictionary(g => g.Key, g => g.ToList())Może to być część rozszerzenia biblioteki Linq. więc musimy tylko zrobić.ToDictionary(o=>o.PropertyName)

— Jaider

3

@Jaider istnieje już taka funkcjonalność: po prostu zastąpić ToDictionaryz ToLookup.

— Robert Synoradzki

19

Nie mogę komentować @Michael Blackburn, ale myślę, że dostałeś głos przeciw, ponieważ GroupBy nie jest w tym przypadku konieczne.

Używaj go jak:

var lookupOfCustomObjects = listOfCustomObjects.ToLookup(o=>o.PropertyName);
var listWithAllCustomObjectsWithPropertyName = lookupOfCustomObjects[propertyName]

Dodatkowo widziałem, że to działa o wiele lepiej niż przy użyciu GroupBy () .ToDictionary ().

— RuudvK
źródło

Wykonałem transliterację, nie odpowiadając na pytanie w najlepszy możliwy sposób.

— Michael Blackburn

1

Dla @ atari2600 odpowiedź wyglądałaby tak, jak przy użyciu ToLookup w składni lambda:

var x = listOfCustomObjects
    .GroupBy(o => o.PropertyName)
    .ToLookup(customObject => customObject);

Zasadniczo pobiera IGrouping i materializuje go dla Ciebie w słowniku list, z wartościami PropertyName jako kluczem.

— Michael Blackburn
źródło

Dlaczego głos przeciw? Czy to nie jest dokładne / nie odpowiada na pytanie?

— Michael Blackburn

1

Na wypadek, gdybyś to przegapił, @RuudvK wspomniał w swojej odpowiedzi , że podejrzewa, że głos przeciwny wynika z tego, że GroupBy jest niepotrzebne. ToLookupma przeciążenie, które wykona zadanie.

— Jeff B

Brakowało mi tej odpowiedzi, dzięki za oznaczenie mnie. Ma sens, ponieważ grupowanie jest składniowo niepotrzebne, zostawiłem je tylko po to, aby przejście od składni zapytania do składni metody było jaśniejsze.

— Michael Blackburn

GroupBy (przeciążenie z tylko jednym parametrem) zwraca IEnumerable <IGrouping <TKey, TSource >>, a następnie ToLookup przekształca go w dość skomplikowany typ, który nie jest nawet podobny do IDictionary <TKey, IList <TSource>> tylko ToLookup zwraca właściwy typ.

— xtofs

-1

Poniższe zadziałały dla mnie.

var temp = ctx.Set<DbTable>()
  .GroupBy(g => new { g.id })
  .ToDictionary(d => d.Key.id);

— Leo Barbas
źródło