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”

c# linq linq-to-sql

— Codewerks
źródło

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