A common issue that I see in the
microsoft.public.dotnet.framework.adonet group is that if you want to
delete a DataRow from a collection of DataRows (e.g. myTable.Rows), you
can't do this using a foreach loop like this:
foreach(DataRow row in myTbl.Rows)
if(someCondition)
row.Delete();
You must instead use a regular for loop and loop from back to front
like this:
for(int i = myTbl.Rows.Count - 1; i >= 0; i--)
if(someCondition)
myTbl.Rows[i].Delete();
I understand why you must do this. My question is whether this is a
standard situation in .NET collection classes.
I have an XML file that I'm processing using the DOM classes
(XmlDocument, XmlNode, etc.) and I need to loop through the children of
an XmlNode and call a function on each one that may or may not delete
that child. Of course, if I use a foreach loop or a for loop that loops
from front to back, the loop counter gets messed up when I delete a
node. It seems to work when I use the same solution as listed above
with the DataRows, but I wanted to confirm that it was a valid solution
in this case.
I can't find any Microsoft documentation regarding this so I was hoping
somebody would be able to point me to some or maybe there would be
somebody with some "inside" knowledge about this issue.
Thanks,
Dave
Lloyd Dupont - 10 Aug 2005 02:09 GMT
yep, it's a common situation
I find it annoying too, but makes sense....
>A common issue that I see in the
> microsoft.public.dotnet.framework.adonet group is that if you want to
[quoted text clipped - 30 lines]
> Thanks,
> Dave
headware - 12 Aug 2005 17:14 GMT
Of course, if Micorsoft changes the code so that when an item is
deleted from the list, it fills in the empty space from front to back
instead of back to front, that's going to break lots of code.
Peter Sestoft - 10 Aug 2005 07:35 GMT
> A common issue that I see in the
> microsoft.public.dotnet.framework.adonet group is that if you want to
[quoted text clipped - 11 lines]
> if(someCondition)
> myTbl.Rows[i].Delete();
This can be marvellously inefficient if the row set is large and more
than a few rows are to be deleted: the runtime is most likely
quadratic.
Peter