Question

In my below code I’m comparing values (email addresses) in my Arrarylist (called emails) with values in my Datatable (called dtResult). I was hoping to be able to remove rows from the datatable if it has an email address that is found in the arraylist but I’m starting to think I’m better off creating a new datatable that does not have those rows instead (I just don’t know how to do that). Obviously my code below bombs after a row is deleted because it loses its place in the foreach loop. How can I fix this?

Thanks.

foreach (DataRow row in dtResult.Rows)
{
  var tmpl = row[3];
  for (int x = 0; x < emails.Count; x++)  //emails is an ArrayList of email addresses
  {
    string one = emails[x].ToString().ToUpper();
    string two = tmpl.ToString().ToUpper();
    //compare datatable value to arraylist value..
    if (one == two)
    {
      //if they are equal then I want to remove them (or maybe create a new datatable??)
      dtResult.Rows.Remove(row);
      dtResult.AcceptChanges();
    }
  }
}

Answer 1

The easiest way would be to loop through the DataTable backward:

for (int i = dtResult.Rows.Count - 1; i >= 0; i--)
{
    var row = dtResult.Rows[i];
    // ...
}

You also might want to consider whether you can just mark the row state on those rows as deleted (and then observe that status later) instead of actually removing them from the table.