Wie kann ich Elemente in einem Listenfeld durchlaufen und diese Elemente dann entfernen?

Question

Ich erhalte den Fehler unten, wenn ich versuche, ein Listenfeld durchzulaufen und dann das Element zu entfernen.

Liste, an die dieser Enumerator gebunden ist, wurde geändert. Ein Enumerator kann nur verwendet werden, wenn sich die Liste nicht ändert.

foreach (string s in listBox1.Items) 
{ 
    MessageBox.Show(s); 
    //do stuff with (s); 
    listBox1.Items.Remove(s); 
} 

Wie kann ich den Artikel entfernen oder noch Schleife durch den Inhalt?

Answer 1

Möchten Sie alle Elemente entfernen? Wenn dies der Fall ist, tun Sie zuerst die foreach, dann verwenden Sie einfach Items.Clear(), um alle danach zu entfernen.

Ansonsten vielleicht Schleife rückwärts durch Indexer:

listBox1.BeginUpdate(); 
try { 
    for(int i = listBox1.Items.Count - 1; i >= 0 ; i--) { 
    // do with listBox1.Items[i] 

    listBox1.Items.RemoveAt(i); 
    } 
} finally { 
    listBox1.EndUpdate(); 
} 

Answer 2

Sie haben mit dem ersten durch die Sammlung aus dem letzten Punkt zu gehen. Dieser Code ist in vb

for i as integer= list.items.count-1 to 0 step -1 
.... 
list.items.removeat(i) 
next 

Answer 3

Jefferson ist richtig, Sie müssen es rückwärts tun.

Hier ist die C# -Äquivalent:

for (var i == list.Items.Count - 1; i >= 0; i--) 
{ 
    list.Items.RemoveAt(i); 
} 

Answer 4

Jeder andere Antwort „nach hinten los“ wurde gebucht, also werde ich die Alternative geben: Erstellen Sie eine Liste der Elemente, die Sie entfernen möchten, entfernen Sie sie dann am Ende :

List<string> removals = new List<string>(); 
foreach (string s in listBox1.Items) 
{ 
    MessageBox.Show(s); 
    //do stuff with (s); 
    removals.Add(s); 
} 

foreach (string s in removals) 
{ 
    listBox1.Items.Remove(s); 
} 

manchmal ist die „Arbeit nach hinten“ Methode besser ist, manchmal über das ist besser - vor allem, wenn Sie mit einer Art zu tun hat, die eine RemoveAll(collection) Methode hat. Gut zu wissen aber beide.

Answer 5

Hier ist meine Lösung ohne rückwärts zu gehen und ohne temporäre Liste

while (listBox1.Items.Count > 0) 
{ 
    string s = listBox1.Items[0] as string; 
    // do something with s 
    listBox1.Items.RemoveAt(0); 
} 

Answer 6

Wie wäre:

foreach(var s in listBox1.Items.ToArray()) 
{ 
    MessageBox.Show(s); 
    //do stuff with (s); 
    listBox1.Items.Remove(s); 
} 

Die ToArray eine Kopie der Liste macht, so müssen Sie nicht darum kümmern, Sie ändern die Liste, während Sie sie bearbeiten.

Answer 7

Sie können keine Änderungen an der Sammlung vornehmen, die im ForEach-Block wiederholt wird.

Eine schnelle Lösung besteht darin, über eine Kopie der Sammlung zu iterieren. Eine einfache Möglichkeit, diese Kopie zu erstellen, ist der ArrayList-Konstruktor. Die DataRowView-Objekte in der kopierten Sammlung beziehen sich auf dieselben zugrunde liegenden Daten wie Ihr Code und können dieselben ändern.

For Each item As DataRowView In New System.Collections.ArrayList(lbOrdersNeedToBeVoided.Items) 

bitte lesen http://social.msdn.microsoft.com/Forums/en-AU/vbgeneral/thread/b4d1f649-d78a-4e5b-8ad8-1940e3379bed

Answer 8

while(listbox.Items.Remove(s)) ; sollte auch funktionieren. Ich denke jedoch, die Rückwärtslösung ist die schnellste.