Assume fooList is an ArrayList

foreach(string s in fooList) {
if (some condition) {
fooList.Remove(s);
}
}


I get the following runtime error:

An unhandled exception of type 'System.InvalidOperationException' occurred
in mscorlib.dll
Additional information: Collection was modified; enumeration operation may
not execute.


How can I fix this?


Thanks

Keith,

you cannot do what you want in this manner.
the foreach statement operates based on the GetEnumerator() reply, and
therefor when you change the content of the arraylist the enumerator is no
longer valid.

try doing it in an other way.

maybe: create a second arraylist for deleted items:

<code>
ArrayList recycleBin = new ArrayList();
foreach (string s in fooList)
{
if (some condition)
{
recycleBin.Add(s);
}
}

foreach (string s in recycleBin)
{
fooList.Remove(s);
}
</code>

HTH
Picho
"Keith O" <(E-Mail Removed)> wrote in message
news:RLKdneKWvd662I3fRVn-(E-Mail Removed)...
> Assume fooList is an ArrayList
>
> foreach(string s in fooList) {
> if (some condition) {
> fooList.Remove(s);
> }
> }
>
>
> I get the following runtime error:
>
> An unhandled exception of type 'System.InvalidOperationException' occurred
> in mscorlib.dll
> Additional information: Collection was modified; enumeration operation may
> not execute.
>
>
> How can I fix this?
>
>
> Thanks
>
>

Within the foreach you are modifying the arraylist. That shouldn't be done.

- Sahil Malik
http://codebetter.com/blogs/sahil.malik/




"Keith O" <(E-Mail Removed)> wrote in message
news:RLKdneKWvd662I3fRVn-(E-Mail Removed)...
> Assume fooList is an ArrayList
>
> foreach(string s in fooList) {
> if (some condition) {
> fooList.Remove(s);
> }
> }
>
>
> I get the following runtime error:
>
> An unhandled exception of type 'System.InvalidOperationException' occurred
> in mscorlib.dll
> Additional information: Collection was modified; enumeration operation may
> not execute.
>
>
> How can I fix this?
>
>
> Thanks
>
>

"Keith O" <(E-Mail Removed)> wrote in message
news:RLKdneKWvd662I3fRVn-(E-Mail Removed)...
> Assume fooList is an ArrayList
> foreach(string s in fooList) {
> if (some condition) {
> fooList.Remove(s);
> }
> }

Or you could use a for-loop and traverse the list backwards.

- Michael S

you can use an index for loop rather then the foreach loop.


On Mon, 14 Feb 2005 01:28:56 -0500, "Keith O" <(E-Mail Removed)>
wrote:

>Assume fooList is an ArrayList
>
>foreach(string s in fooList) {
> if (some condition) {
> fooList.Remove(s);
> }
>}
>
>
>I get the following runtime error:
>
>An unhandled exception of type 'System.InvalidOperationException' occurred
>in mscorlib.dll
>Additional information: Collection was modified; enumeration operation may
>not execute.
>
>
>How can I fix this?
>
>
>Thanks
>

You can subvert the read-only characteristics of the foreach construct using
this code

foreach(string s in new System.Collection.ArrayList(fooList)) { //this
opening brace should be on
//the next line by the way
if (some condition) {
fooList.Remove(s);
}
}

Is it a recommended approach? I sincerely doubt it but that's a question for
another thread?

--
Regards,
Alvin Bruney [Microsoft MVP ASP.NET]

[Shameless Author plug]
The Microsoft Office Web Components Black Book with .NET
Now Available @ http://www.lulu.com/owc
----------------------------------------------------------


"Keith O" <(E-Mail Removed)> wrote in message
news:RLKdneKWvd662I3fRVn-(E-Mail Removed)...
> Assume fooList is an ArrayList
>
> foreach(string s in fooList) {
> if (some condition) {
> fooList.Remove(s);
> }
> }
>
>
> I get the following runtime error:
>
> An unhandled exception of type 'System.InvalidOperationException' occurred
> in mscorlib.dll
> Additional information: Collection was modified; enumeration operation may
> not execute.
>
>
> How can I fix this?
>
>
> Thanks
>
>