listBox1.Items. Remove help

[Bilo]

I dont know what i am doing false.
the code :

private void button2_Click(o bject sender, System.EventArg s e)
{
foreach (string filename in listBox1.Select edItems)
listBox1.Items. Remove(filename );
}
gives me the error :

System.InvalidO perationExcepti on: The list that this enumerator is bound to
has been modified. An enumerator can only be used if the list doesn't
change.
at
System.Windows. Forms.EntryEnum erator.System.C ollections.IEnu merator.MoveNex t
()

Anyone can help?

[Julie]

Bilo wrote:

I dont know what i am doing false.
the code :

private void button2_Click(o bject sender, System.EventArg s e)
{
foreach (string filename in listBox1.Select edItems)
listBox1.Items. Remove(filename );
}
gives me the error :

System.InvalidO perationExcepti on: The list that this enumerator is bound to
has been modified. An enumerator can only be used if the list doesn't
change.
at
System.Windows. Forms.EntryEnum erator.System.C ollections.IEnu merator.MoveNex t
()

Anyone can help?

You can't remove items from the listbox while you are iterating over that very
listbox. The problem is because it becomes difficult to maintain consistency
if items within the list are being removed.

What you need to do is to create an intermediate structure to hold the items to
be removed, then remove those items:

ArrayList files = new ArrayList();
foreach (string filename in listBox1.Select edItems)
files.Add(filen ame);
foreach (string filename in files)
listBox1.Items. Remove(filename );

[Bjorn Abelli]

"Bilo" wrote...

private void button2_Click(o bject sender, System.EventArg s e)
{
foreach (string filename in listBox1.Select edItems)
listBox1.Items. Remove(filename );
}
The error actually spells it out:
System.InvalidO perationExcepti on:
The list that this enumerator is bound to
has been modified. An enumerator can only
be used if the list doesn't change.

When you Remove an item from the ListBox, you also affect the "contents" of
its SelectedItems, which makes it impossible to continue the iteration, as
the iteration is *based* on it.

You could try to do something like this instead:

string[] sa = new string[listBox1.Select edItems.Count];
listBox1.Select edItems.CopyTo( sa,0);

foreach (string filename in sa)
{
listBox1.Items. Remove(filename );
}

// Bjorn A

[Bilo]

Ahh first store the items.
Thx you.

"Bjorn Abelli" <bj**********@D oNotSpam.hotmai l.com> schrieb im Newsbeitrag
news:Ox******** ******@TK2MSFTN GP11.phx.gbl...

"Bilo" wrote...

private void button2_Click(o bject sender, System.EventArg s e)
{
foreach (string filename in listBox1.Select edItems)
listBox1.Items. Remove(filename );
}
The error actually spells it out:

System.InvalidO perationExcepti on:
The list that this enumerator is bound to
has been modified. An enumerator can only
be used if the list doesn't change.

When you Remove an item from the ListBox, you also affect the "contents"

of its SelectedItems, which makes it impossible to continue the iteration, as
the iteration is *based* on it.

You could try to do something like this instead:

string[] sa = new string[listBox1.Select edItems.Count];
listBox1.Select edItems.CopyTo( sa,0);

foreach (string filename in sa)
{
listBox1.Items. Remove(filename );
}

// Bjorn A

[someone]

"Bilo" <ju********@yah oo.com> wrote in
news:#p******** ******@TK2MSFTN GP12.phx.gbl:

I dont know what i am doing false.
the code :

private void button2_Click(o bject sender, System.EventArg s e)
{
foreach (string filename in listBox1.Select edItems)
listBox1.Items. Remove(filename );
}
gives me the error :

System.InvalidO perationExcepti on: The list that this enumerator is
bound to has been modified. An enumerator can only be used if the list
doesn't change.
at
System.Windows. Forms.EntryEnum erator.System.C ollections.IEnu merator.Mov
eNext ()

Anyone can help?

to keep from duplicating all those strings, how bout this?
while (listBox1.Selec tedItems.Count > 0)
{
listBox1.Items. Remove(listBox1 .SelectedItems[0]);
}