Remove an item from an arraylist inside foreach

[tshad]

I have an ArrayList of thread pointers that I want delete from my list when
the Thread in not alive and status is stopped. But I want to do it from the
foreach loop I am looping through. threadList is the ArrayList.

foreach (Thread theThread in threadList)
{
if ((!theThread.Is Alive) && (theThread.Thre adState.ToStrin g ==
"Stopped"))
sw.WriteLine(" Status for Thread: {0} is {1} IsAlive: {2}",
theThread.Name, theThread.Threa dState,theThrea d.IsAlive);

// this is where I want to take the pointer out of the list - something
like threadList.curr ent.remove.
}

Thanks,

Tom

[Nicholas Paldino [.NET/C# MVP]]

Tom,

This fails because you can not change the list while iterating through
the list with foreach (i.e. accessing through IEnumerable).

To get around this, you should use a regular for construct, starting
with the last index in the list, and then cycling through and removing them
if appropriate (you start at the last item in the list because if you remove
one while moving forward, you have to readjust your index to account for the
shifting of the indexes of the remaining elements).
--
- Nicholas Paldino [.NET/C# MVP]
- mv*@spam.guard. caspershouse.co m

"tshad" <ts***@dslextre me.comwrote in message
news:eX******** ******@TK2MSFTN GP04.phx.gbl...

>I have an ArrayList of thread pointers that I want delete from my list when
the Thread in not alive and status is stopped. But I want to do it from
the foreach loop I am looping through. threadList is the ArrayList.

foreach (Thread theThread in threadList)
{
if ((!theThread.Is Alive) && (theThread.Thre adState.ToStrin g ==
"Stopped"))
sw.WriteLine(" Status for Thread: {0} is {1} IsAlive: {2}",
theThread.Name, theThread.Threa dState,theThrea d.IsAlive);

// this is where I want to take the pointer out of the list - something
like threadList.curr ent.remove.
}

Thanks,

Tom

[Rene]

You cannot. A collection cannot be modified while being enumerated.

Hi Ignacio,

Well, I am not sure if I would go as far as saying that. I may be wrong but
I think someone could create their own collection (implementing their own
IEnumerator) and on their implementation they don't have to throw an error
if the collection is modified while enumerating.

Now, I am not saying if that is a good idea to do or not, all I am saying is
that just because all .Net collection will throw an error if the collection
is modified while enumerating does not mean that *all* collection will do
the same thing.

I don't believe this behavior governed by the runtime. In other words,
people should not count on this behavior by default... but then again, I may
be wrong!!
Cheers.

[tshad]

"Nicholas Paldino [.NET/C# MVP]" <mv*@spam.guard .caspershouse.c omwrote in
message news:47******** *************** ***********@mic rosoft.com...

Tom,

This fails because you can not change the list while iterating through
the list with foreach (i.e. accessing through IEnumerable).

To get around this, you should use a regular for construct, starting
with the last index in the list, and then cycling through and removing
them if appropriate (you start at the last item in the list because if you
remove one while moving forward, you have to readjust your index to
account for the shifting of the indexes of the remaining elements).

That's as good a way as any.

Thanks,

Tom

>

--
- Nicholas Paldino [.NET/C# MVP]
- mv*@spam.guard. caspershouse.co m

"tshad" <ts***@dslextre me.comwrote in message
news:eX******** ******@TK2MSFTN GP04.phx.gbl...

>>I have an ArrayList of thread pointers that I want delete from my list
when the Thread in not alive and status is stopped. But I want to do it
from the foreach loop I am looping through. threadList is the ArrayList.

foreach (Thread theThread in threadList)
{
if ((!theThread.Is Alive) && (theThread.Thre adState.ToStrin g ==
"Stopped"))
sw.WriteLine(" Status for Thread: {0} is {1} IsAlive: {2}",
theThread.Name , theThread.Threa dState,theThrea d.IsAlive);

// this is where I want to take the pointer out of the list -
something like threadList.curr ent.remove.
}

Thanks,

Tom

[Marc Gravell]

Aside; if this was List<Threadinst ead of ArrayList, you could do
something like below (using C# 3 lambda syntax, but an anonymous
method would work just as well):

threadList.Remo veAll(thread ={
if ((!thread.IsAli ve) &&
(thread.ThreadS tate.ToString() == "Stopped")) {
sw.WriteLine(". ..foo...");
return true;
}
return false;
});

Marc