Jaki jest najlepszy sposób ustawienia wszystkich wartości w słowniku C# <string,bool>?

Question

Jaki jest najlepszy sposób ustawienia wszystkich wartości w słowniku C#?

Oto, co robię teraz, ale jestem pewien, że jest lepiej/czystsze sposobem, aby to zrobić:

Dictionary<string,bool> dict = GetDictionary(); 
var keys = dict.Keys.ToList(); 
for (int i = 0; i < keys.Count; i++) 
{ 
    dict[keys[i]] = false; 
} 

Próbowałem kilka innych sposobów, z foreach, ale miałem błędów.

Answer 1

To rozsądne podejście, chociaż wolałbym:

foreach (var key in dict.Keys.ToList()) 
{ 
    dict[key] = false; 
} 

Wezwanie do ToList() czyni tę pracę, ponieważ jest wyciągając i (chwilowo) zapisując listę kluczy, więc prace iteracji.

Answer 2

Można po prostu wyciągnąć ToList() i iteracji bezpośrednio nad słownikowych pozycji

Dictionary<string, bool> dict = GetDictionary(); 
foreach (var pair in dict) 
{ 
    dict[pair.Key] = false; 
} 

Answer 3

zrobić to sposób masz go teraz ... foreach jest powolny. Foreach może być czystszy, ale za dużo wykorzystujesz działanie wydajności.

Edit:
http://www.codeproject.com/KB/cs/foreach.aspx
http://www.madprops.org/blog/for-vs-foreach-performance/

Answer 4

Jeśli nie używasz bools Tri-State, a następnie można użyć HashSet<string> i wywołać Clear() ustawić wartości "false".

Answer 5

Zaprezentowałem różnicę między rozwiązaniami Billy'ego i Reeda. Polaris878, zwróć uwagę na wyniki i pamiętaj, że przedwczesna optymalizacja jest źródłem wszelkiego zła ;-)

Przerobiłem rozwiązania w VB (ponieważ obecnie programuję w tym języku) i użyłem klawiszy int (dla uproszczenia), w przeciwnym razie jest to dokładnie ten sam kod. Uruchomiłem kod ze słownikiem 10 milionów wpisów o wartości "true" dla każdego wpisu. oryginalne rozwiązanie

Billy Szaman za:

Dim keys = dict.Keys.ToList 
For i = 0 To keys.Count - 1 
    dict(keys(i)) = False 
Next 

Upłynęło milisekund: 415

Reed Copsey za rozwiązanie:

For Each key In dict.Keys.ToList 
    dict(key) = False 
Next 

Upłynęło milisekund: 395

Więc w tym przypadku foreach jest rzeczywiście szybsza.

Answer 6

Roztwór jeden wiersz:

dict = dict.ToDictionary(p => p.Key, p => false); 

Answer 7

Nie jestem pewien, czy to jest najlepszy sposób, ale szukałem czegoś w jednej linii, a ten pracował dla mnie

mydict.Keys.ToList().ForEach(k => mydict[k] = false);