我有一个代码
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)
但是上面对我来说似乎不是一个好方法,因为我必须做另一个循环并使用额外的内存
答案 0 :(得分:3)
在这种特定情况下,您可以在包含几列的小型集合中循环,只需创建一个新集合(通过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);
}
答案 1 :(得分:0)
使用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);
}
}
答案 2 :(得分:0)
另一种方法是在删除列后降低索引。
for (int i = 0; i < datatable.Columns.Count; i++)
{
if (datatable.Columns[i].ColumnName.Contains("Column"))
{
datatable.Columns.RemoveAt(i);
i--;
}
}
答案 3 :(得分:0)
if (dt.Columns.Contains("RecordID")){
dt.Columns.Remove("RecordID");
dt.AcceptChanges();
}