entfernen würde, einen Schlüssel aus einem Wörterbuch in der foreach-ein problem? oder sollte ich besser zu konstruieren, die ein neues Wörterbuch?

Beispiel:

1.

         foreach (var item in myDic)
                {
                  if (item.value == 42)
                        myDic.remove(item.key);
                }

würde der iterator funktioniert einwandfrei, egal wie die Aussagen in den inneren Klammern könnte möglicherweise beeinflussen myDic?

2.

    var newDic = myDic.where(x=>x.value!=42).ToDictionary(x=>x.key,x=>x.value);

Ist der 2. Ansatz eine gute Praxis? funktionale Programmierung und unveränderlich?

InformationsquelleAutor colinfang | 2011-08-24

  1. 27

    Der erste Ansatz Absturz zur Laufzeit, da der enumerator stellt sicher, dass niemand löscht aus der zugrunde liegenden Erhebung, während es aufzählen.

    Der zweite Ansatz ist ein netter Gedanke, aber C# dictionaries sind veränderlich und es ist weder idiomatisch, noch effizienter, um Sie zu kopieren, um, wenn Sie können das gleiche erreichen mit mutation.

    Dies ist eine typische Art und Weise:

    var itemsToRemove = myDic.Where(f => f.Value == 42).ToArray();
foreach (var item in itemsToRemove)
    myDic.Remove(item.Key);

    EDIT: In Antwort auf Ihre Frage in den Kommentaren. Hier ist, wie das Beispiel in Ihrem andere Frage funktioniert:

    myList = myList.where(x=>x>10).select(x=>x-10);

    Diese Codezeile nicht ausgeführt werden kann alles; es ist Total faul. Sagen wir mal um des Arguments Willen, wir haben eine foreach nach, um es so Aussehen mehr wie in dieser Frage zum Vorbild nehmen.

    foreach (int n in myList)
    Console.WriteLine(n);

    Wenn das ausgeführt wird, hier ist, was passieren wird, die bei jeder iteration:

    1. Nennen MoveNext auf der enumerator
    2. Der enumerator findet der nächste Wert größer als zehn
    3. Dann dauert es den Wert minus zehn und legt die Current Eigenschaft, dass
    4. Bindet die Current Eigenschaft der Variablen n
    5. Console.WriteLines

    Können Sie sehen, dass es kein Geheimnis und keine Endlosschleife und kein was auch immer.

    Vergleichen Sie jetzt zu meinem Beispiel, angenommen, dass wir linken aus der ToArray.

    var itemsToRemove = myDic.Where(f => f.Value == 42);
foreach (var item in itemsToRemove)
    myDic.Remove(item.Key);
    1. Nennen MoveNext auf der enumerator
    2. Der enumerator findet das nächste paar mit dem Wert 42 und setzt die Current Eigenschaft, dass
    3. Bindet die Current Eigenschaft der Variablen item
    4. Removes

    Dies nicht funktioniert, weil, während es vollkommen in Ordnung zu WriteLine etwas aus einer Sammlung, während Sie haben ein enumerator auf Sie zu öffnen, Sie sind nicht erlaubt zu Remove etwas aus einer Sammlung, während Sie haben ein enumerator auf Sie zu öffnen.

    Wenn Sie anrufen ToArray vorne, dann beginnen Sie durch die Aufzählung über das Wörterbuch und Auffüllen der array. Wenn wir uns auf die foreach, die foreach - Anweisung hat einen enumerator öffnen Sie auf der array, nicht das Wörterbuch. Sie sind berechtigt, zu entfernen aus dem Wörterbuch, wie Sie Durchlaufen das array.

    ToList() ist schneller als ToArray().
    Der performance-Unterschied ist unbedeutend genug, dass ich in der Regel lieber nur je nachdem, was scheint mehr semantisch korrekte (das heißt, wenn ich nicht plan auf hinzufügen oder entfernen, halte ich mich mit ToArray.)
    Nein, kann man nicht weglassen. Wenn man es faul, dann wäre es immer noch aufzählen über das Wörterbuch, wenn Remove genannt wird, und Sie würden wieder da, wo Sie begonnen haben (mit einer Ausnahme zur Laufzeit.) Sie müssen alles tun, die Arbeit zu bauen, die Liste der Elemente zu entfernen, bevor Sie tatsächlich beginnen entfernen.
    Haben Sie verpasst .select(x=>x.Taste), um zur reform des Wörterbuch in der Liste?
    Warum nicht einen one-liner? myDic.Where(kvp => kvp.Value == 42).ToList.ForEach(kvp => myDic.Remove(kvp.Key));

    InformationsquelleAutor mquander

  2. 6

    Auch Sie können Durchlaufen die kopieren aus Ihrer Sammlung:

    foreach (var item in myDic.ToList())
{
    if (item.value == 42)
    myDic.remove(item.key);
}

    bemerken myDic.ToList() im foreach - Anweisung.

    InformationsquelleAutor Vladislav

Schreibe einen Kommentar