我有一个代码
foreach (DataColumn dataTableCol in this.dataTable.Columns)
{
bool columnFound = false;
foreach (GRTColumnView uiColumn in descriptor.UIColumns)
{
if (dataTableCol.ColumnName.Equals(uiColumn.Name))
{
columnFound = true;
break;
}
}
if (!columnFound)
{
if (this.dataTable.Columns.Contains(dataTableCol.ColumnName))
this.dataTable.Columns.Remove(dataTableCol.ColumnName);
}
}
如果在其他集合中找不到一些"东西",我想从集合中删除它们。
当我运行上述程序时,我得到
迭代可能无法执行,因为集合已被修改
"集合已修改" - 因为删除必须已被击中
那么有什么方法可以实现这样的事情呢?
我能想到的是记下所有要删除的"东西",然后
foreach( aThing in all_things_to_remove)
remove_from_collection(aThing)
但是上面对我来说似乎不是一个好方法,因为我必须再做一次循环并且正在使用额外的内存
在这种特定情况下,您正在循环访问包含几列的小集合,您可以创建一个新集合(通过 ToList()
),这样您就不会迭代要修改的同一集合:
foreach (var dataTableCol in dataTable.Columns.Cast<DataColumn>().ToList())
{
...
dataTable.Columns.Remove(dataTableCol.ColumnName);
}
建议的方法(尤其是在集合很大的情况下)是向后枚举:
for (var i = dataTable.Columns.Count - 1; i >= 0; i--)
{
...
dataTable.Columns.Remove(dataTable.Columns[i].ColumnName);
}
if (dt.Columns.Contains("RecordID")){
dt.Columns.Remove("RecordID");
dt.AcceptChanges();
}
在使用 foreach 循环枚举集合时,不能从集合中删除项。使用Collection.ToArray()
创建集合的副本,并在副本上运行 foreach 并从实际集合中删除您的项目。
由于 DataTable.Columns 没有ToArray
或ToList
方法,因此可以使用 CopyTo()
方法并将整个列复制到 ColumnsArray。
如果您不想创建副本,则可以使用 for 循环而不是 foreach 循环。您可以通过以下方式编辑代码:
for (int i = 0; i < dataTable.Columns.Count; i++)
{
bool columnFound = false;
foreach (GRTColumnView uiColumn in descriptor.UIColumns)
{
if (dataTable.Columns[i].Name.Equals(uiColumn.Name))
{
columnFound = true;
break;
}
}
if (!columnFound)
{
if (this.dataTable.Columns.Contains(dataTableCol.ColumnName))
this.dataTable.Columns.Remove(dataTableCol.ColumnName);
}
}
另一种方法是在删除列后降低索引。
for (int i = 0; i < datatable.Columns.Count; i++)
{
if (datatable.Columns[i].ColumnName.Contains("Column"))
{
datatable.Columns.RemoveAt(i);
i--;
}
}