Question

I have a DataSet with several DataTables inside. I’m displaying the DataTables in a ListView (I didn’t know about databinding when I wrote the code). Anyway, I would like to remove rows from the DataTables inside the DataSet.

I have tried this:

foreach (DataRow row in dsData.Tables["Table1"].Rows)
  {
     //find the row that contains the username I'm after
     if (item.SubItems[2].Text == row["LoginName"].ToString())            
       {
         dsData.Tables["Table1"].Rows.Remove(row); //<- main code of interest
       }
  }

I’ve also tried

dsData.Tables["Table1".Rows.Delete(row);

The problem I’m experiencing is that the when you remove a row I get the exception:

Collection was modified; enumeration operation might not execute.

From what I understand it’s because when you remove a row from a ListView the row below it moves up and causes all this trouble. The code itself does what it’s supposed to but it’s not nice to see that exception when when you run it.

I was about to rewrite the whole class with a DataGridView but would rather correct that single line if possible :).

EDIT: I’m not even sure a DataGridView would solve the problem anyway.

Answer 1

change the loop to a for loop counting backwards so you don’t get that message.

for(int i = dsData.Tables["TAble1"].Rows; i > 0; i--)
{
     if(item.SubItems[2].Text == dsData.Tables["Table1"].Rows[i - 1]["LoginName"].ToString())
         dsData.Tables["Table1"].Rows.Remove(i - 1)
}