Linq-to-SQL ToDictionary ()

81

Jak poprawnie przekonwertować dwie kolumny z SQL (2008) za pomocą Linq na słownik (do buforowania)?

Obecnie wykonuję pętlę przez IQueryable b / c Nie mogę uruchomić metody ToDictionary. Jakieś pomysły? To działa:

var query = from p in db.Table
            select p;

Dictionary<string, string> dic = new Dictionary<string, string>();

foreach (var p in query)
{
    dic.Add(sub.Key, sub.Value);
}

To, co naprawdę chciałbym zrobić, to coś takiego, co wydaje się nie działać:

var dic = (from p in db.Table
             select new {p.Key, p.Value })
            .ToDictionary<string, string>(p => p.Key);

Ale pojawia się ten błąd: nie można przekonwertować z „System.Linq.IQueryable” na „System.Collections.Generic.IEnumerable”

Codewerks
źródło

Odpowiedzi:

121
var dictionary = db
    .Table
    .Select(p => new { p.Key, p.Value })
    .AsEnumerable()
    .ToDictionary(kvp => kvp.Key, kvp => kvp.Value)
;
yfeldblum
źródło
Przepraszam, może nie było jasne, próbowałem już wcześniej, ale to tworzy Dictionary <string, #Anonymous Type>, a nie Dictionary <string, string>
Codewerks
Spróbuj kvp => kvp.Value as string. Punktem odpowiedzi było .AsEnumerable().
yfeldblum
Dzięki, tak, pomyślałem, że AsEnumerable jest potrzebny, ale wywołanie ToDictionary nadal nie działa. Obie kolumny są varcharami w SQL, więc wracają jako łańcuchy, ale nie mogę wymyślić, jak zmusić je do umieszczenia w Dic ...
Codewerks
Nie wiem, czy to jest nadal aktualne, ale dlaczego jest to AsEnumerablepotrzebne? U mnie to też działa bez tego, ale prawdopodobnie Linq już się zmienił (minęło 7 lat)
Alexander Derck
1
@AlexanderDerck - W tamtym czasie AsEnumerablebyło to konieczne. Nie pamiętam dokładnie, dlaczego po tak długim czasie, ale możliwe wyjaśnienie jest takie, że ToDictionarybyła to metoda rozszerzająca IEnumerable, ale interfejs Linq-to-SQL nie IEnumerable.
yfeldblum
16

Definiujesz tylko klucz, ale musisz również uwzględnić wartość:

var dic = (from p in db.Table
             select new {p.Key, p.Value })
            .ToDictionary(p => p.Key, p=> p.Value);
Christian C. Salvadó
źródło
3
-1 Błąd polega na tym, że <string, string>część powoduje, że używa public static Dictionary<TKey, TSource> ToDictionary<TSource, TKey>(this IEnumerable<TSource> source, Func<TSource, TKey> keySelector, IEqualityComparer<TKey> comparer);przeciążenia zamiast public static Dictionary<TKey, TElement> ToDictionary<TSource, TKey, TElement>(this IEnumerable<TSource> source, Func<TSource, TKey> keySelector, Func<TSource, TElement> elementSelector);przeciążenia.
user247702
Myślę, że to jeszcze potrzebuje .AsEnumerable()wcześniej.ToDictionary()
AceMark
działało idealnie ... w końcu ten, którego szukałem
Brian
9

Dzięki, wasze odpowiedzi pomogły mi to naprawić, powinny wyglądać tak:

var dic = db
        .Table
        .Select(p => new { p.Key, p.Value })
        .AsEnumerable()
        .ToDictionary(k=> k.Key, v => v.Value);
Codewerks
źródło
5
Powodem, dla którego potrzebujesz AsEnumerable (), jest to, że LINQ to SQL nie łączy przetwarzania lokalnego i zdalnego (SQL), więc powoduje to, że pierwsza część jest wykonywana na serwerze SQL, a ostatnia kolejna część jest wykonywana lokalnie przy użyciu LINQ to Objects, co może do Dictionarys :)
DamienG
Ma sens. Również drugi parametr w ToDictionary potrzebuje własnych argumentów Func, co wcześniej mnie zaskoczyło.
Codewerks
1

Dlaczego miałbyś utworzyć anonimowy obiekt dla każdego elementu w tabeli tylko po to, aby go przekonwertować?

Możesz po prostu użyć czegoś takiego: IDictionary<string, string> dic = db.Table.ToDictionary(row => row.Key, row => row.Value); Być może będziesz musiał dołączyć wywołanie AsEnumerable () między Table i ToDictionary (). Nie znam dokładnego typu tabeli db.Table.


Popraw również pierwszą próbkę, druga zmienna pętli jest niezgodna w deklaracji i użyciu.

TWiStErRob
źródło
2
Ponieważ ToDictionaryjest w pamięci. Brak tworzenia anonimowego obiektu oznacza, że ​​wszystkie kolumny są pobierane z bazy danych, a nie tylko te, których potrzebujesz.
user247702
ToDictionarynie wie o podstawowej strukturze, po prostu wywołuje dwa wywołania zwrotne klucza i wartości (w tym przypadku prosta właściwość wybierająca lambdy), więc to ToDictionarywywołanie powinno być funkcjonalnie równoważne z foreach w podanym przez niego przykładzie. O ile wiem, wszystkie metody LINQ są odroczone, co oznacza, że ​​wywołania zwrotne są wywoływane tylko wtedy, gdy jest to konieczne.
TWiStErRob
3
Jeśli spojrzysz za pomocą programu SQL Server Profiler, zobaczysz, że w przypadku obiektu anonimowego wybrane są tylko dwie kolumny, a bez niego wszystkie kolumny są zaznaczone.
user247702