数据行出错,Collection被修改;枚举操作可能无法执行

时间:2013-03-17 04:36:34

标签: c# foreach datarow

我有for-each循环,其中数据行被更新,因此异常, Collection被修改;生成枚举操作可能无法执行。有什么办法解决吗?我见过To-List函数,但它没有使用数据行,这是我的代码:

foreach (DataRow row in dataTable.Rows) {
  temp = row[0].ToString();
  foreach (DataRow rows in dataTable.Rows) {
    if (temp == rows[0].ToString()) {
      tempdatatable.Rows.Add(row[0],row[1]);
      dataTable.Rows.Remove(rows);
      //Update happens here
    }
    tempdatatable.DefaultView.Sort = "gscitations DESC";
    dataGridView1.DataSource = tempdatatable;
  }
}

3 个答案:

答案 0 :(得分:6)

使用Enumerator枚举时无法修改集合,这发生在foreach语句(MDSN link)的场景后面。

解决此问题的一种可能方法是在第一个枚举中收集要删除的行,然后在单独的循环中删除它们,如下所示:

var rowsToDelete = new List<DataRow>();

foreach (DataRow row in dataTable.Rows)
     {
         temp = row[0].ToString();
         foreach (DataRow rows in dataTable.Rows)
         {
             if (temp == rows[0].ToString())
             {
                 tempdatatable.Rows.Add(row[0],row[1]);
                 rowsToDelete.Add(rows);
             }
             tempdatatable.DefaultView.Sort = "gscitations DESC";
             dataGridView1.DataSource = tempdatatable;
         }
     }

rowsToDelete.ForEach( x => dataTable.Rows.Remove(x) );

您也可以将foreach循环替换为for,但在删除元素时需要正确处理当前索引。

答案 1 :(得分:3)

试试这个:

for (int i = 0; i < dataTable.Rows.Count; i++)
{
    var tempRow = dataTable.Rows[i];
    var temp = dataTable.Rows[i][0];
    for (int j = 0; j < dataTable.Rows.Count; j++)
    {
        DataRow rows = dataTable.Rows[j];
        if (temp == rows[0].ToString())
        {
            tempdatatable.Rows.Add(tempRow[0], tempRow[1]);
            dataTable.Rows.Remove(rows);      //Update happen here
        }
        tempdatatable.DefaultView.Sort = "gscitations DESC";
        dataGridView1.DataSource = tempdatatable;
    }
}

答案 2 :(得分:0)

我想说你应该创建一个单独的条目表,而不是调用datatable.Rows.Remove(rows),将行“rows”添加到另一个表中。然后,每当行或行迭代时,运行if语句以检查它是否已被“删除”,即在已删除行的列表中。枚举结束后,您可以从表中永久删除这些行。

修改

以下是代码实现:

DataTable duplicates = dataTable;
duplicates.Rows.Clear(); /* Produces an empty duplicate of the 
dataTable table to put the duplicates in */
foreach (DataRow row in dataTable.Rows)
{
     if (!duplicates.Rows.Contains(row))
     {    
         temp = row[0].ToString();
         foreach (DataRow rows in dataTable.Rows)
         {
             if (temp == rows[0].ToString()&&!duplicates.Rows.Contains(rows)) //need unique key
             {
                 tempdatatable.Rows.Add(row[0],row[1]);

             }
             tempdatatable.DefaultView.Sort = "gscitations DESC";
             dataGridView1.DataSource = tempdatatable;
         }
    }
}
foreach (DataRow row in duplicates.Rows)
{
    dataTable.Rows.Remove(row);
}

如果您没有唯一密钥,可以尝试将!duplicates.Rows.Contains(/*specific row*/)切换为duplicates.Rows.IndexOf(/*specific row*/)>0。这应该提供足够的替代品。