从DataTable中删除特定行

问题描述 投票:64回答:13

我想从DataTable中删除一些行,但它给出了这样的错误,

收集被修改;枚举操作可能无法执行

我用来删除这段代码,

foreach(DataRow dr in dtPerson.Rows){
    if(dr["name"].ToString()=="Joe")
        dr.Delete();
}

那么,问题是什么以及如何解决?你建议采用哪种方法?

c# delete-row
13个回答
135
投票

如果从集合中删除项目,则该集合已更改,您无法继续通过它进行枚举。

相反,使用For循环,例如:

for(int i = dtPerson.Rows.Count-1; i >= 0; i--)
{
    DataRow dr = dtPerson.Rows[i];
    if (dr["name"] == "Joe")
        dr.Delete();
}
dtPerson.AcceptChanges();

请注意,您正在反向迭代以避免在删除当前索引后跳过一行。


1
投票

我知道这是非常古老的问题,我前几天也有类似的情况。

问题是,在我的表中是约。 10000行,所以循环通过qazxsw poi行非常慢。

最后,我找到了更快的解决方案,在那里我制作了具有所需结果的源DataTable的副本,清除源DataTableDataTable结果从临时merge到源1。

注意:改为在DataTable中搜索Joe,名为DataRow你必须搜索所有没有名称name的记录(与搜索方式相反)

有例子(Joe):

vb.net

我希望这个较短的解决方案可以帮助某人。

'Copy all rows into tmpTable whose not contain Joe in name DataRow Dim tmpTable As DataTable = drPerson.Select("name<>'Joe'").CopyToTable 'Clear source DataTable, in Your case dtPerson dtPerson.Clear() 'merge tmpTable into dtPerson (rows whose name not contain Joe) dtPerson.Merge(tmpTable) tmpTable = Nothing 代码(不确定是否正确,因为我使用在线转换器:():

c#

当然,如果没有结果,我使用了//Copy all rows into tmpTable whose not contain Joe in name DataRow DataTable tmpTable = drPerson.Select("name<>'Joe'").CopyToTable; //Clear source DataTable, in Your case dtPerson dtPerson.Clear(); //merge tmpTable into dtPerson (rows whose name not contain Joe) dtPerson.Merge(tmpTable); tmpTable = null; (例如,如果你的Try/Catch不包含dtPerson name它将抛出异常),所以你对你的表没有任何作用,它保持不变。


0
投票

我的应用程序中有一个数据集,我去设置更改(删除一行),但Joe是只读的。然后我找到了这个解决方案

这是一个wpf ds.tabales["TableName"]应用程序,

C#

0
投票

您尝试这样做以从数据表中获取和删除id列

try {
    var results = from row in ds.Tables["TableName"].AsEnumerable() where row.Field<string>("Personalid") == "47" select row;                
    foreach (DataRow row in results) {
        ds.Tables["TableName"].Rows.Remove(row);                 
    }           
}

0
投票

我在这里看到了正确答案的各种点点滴滴,但是让我把它们全部放在一起并解释一些事情。

首先,if (dt1.Columns.Contains("ID")) { for (int i = dt1.Rows.Count - 1; i >= 0; i--) { DataRow dr = dt1.Rows[i]; if (dr["ID"].ToString() != "" && dr["ID"].ToString() != null) { dr.Delete(); } } dt1.Columns.Remove("ID"); } 应仅用于将表上的整个事务标记为已验证和已提交。这意味着如果您使用DataTable作为DataSource绑定到例如SQL服务器,那么手动调用AcceptChanges将保证更改永远不会保存到SQL服务器。

是什么让这个问题更加混乱的是,实际上有两种情况会引发异常,我们必须阻止它们。

1.修改IEnumerable的集合

我们无法在枚举的集合中添加或删除索引,因为这样做可能会影响枚举器的内部索引。有两种方法可以解决这个问题:要么在for循环中进行自己的索引,要么使用单独的集合(未修改)进行枚举。

2.试图读取已删除的条目

由于DataTable是事务集合,因此可以将条目标记为删除但仍显示在枚举中。这意味着如果你要求列AcceptChanges的删除条目,那么它将抛出异常。这意味着我们必须在查询列之前检查是否"name"

把它们放在一起

我们可能会弄乱并且手动完成所有这些操作,或者我们可以让DataTable为我们完成所有工作,并通过执行以下操作使语句更像SQL调用:

dr.RowState != DataRowState.Deleted

通过调用DataTable的string name = "Joe"; foreach(DataRow dr in dtPerson.Select($"name='{name}'")) dr.Delete(); 函数,我们的查询会自动避免DataTable中已删除的条目。由于Select函数返回一个匹配数组,因此当我们调用Select时,我们所枚举的集合不会被修改。我还用字符串插值调整了Select表达式,以允许变量选择而不会使代码产生噪音。


106
投票

在每个人都跳过“你无法删除枚举中的行”之前,你需要先了解DataTables是事务性的,并且在调用AcceptChanges()之前不要在技术上清除更改。

如果在调用Delete时看到此异常,则表示您已处于挂起更改数据状态。例如,如果刚从数据库加载,如果你在foreach循环中,调用Delete会抛出异常。

但!但!

如果从数据库加载行并调用函数'AcceptChanges()',则将所有这些挂起的更改提交给DataTable。现在,您可以在世界范围内遍历调用Delete()的行列表,因为它只是将行标记为Deletion,但在您再次调用AcceptChanges()之前不会提交。

我意识到这个响应有点陈旧,但我最近不得不处理类似的问题,希望这为未来开发10年代码的开发人员节省了一些痛苦:)


附:这是Jeff添加的一个简单代码示例:

C#

YourDataTable.AcceptChanges(); 
foreach (DataRow row in YourDataTable.Rows) {
    // If this row is offensive then
    row.Delete();
} 
YourDataTable.AcceptChanges();

VB.net

ds.Tables(0).AcceptChanges()
For Each row In ds.Tables(0).Rows
    ds.Tables(0).Rows(counter).Delete()
    counter += 1
Next
ds.Tables(0).AcceptChanges()

15
投票

使用此解决方案:

for(int i = dtPerson.Rows.Count-1; i >= 0; i--) 
{ 
    DataRow dr = dtPerson.Rows[i]; 
    if (dr["name"] == "Joe")
        dr.Delete();
} 

如果要在删除行后使用数据表,则会出现错误。所以你可以做的是:用dr.Delete();替换dtPerson.Rows.Remove(dr);


13
投票

这对我有用,

List<string> lstRemoveColumns = new List<string>() { "ColValue1", "ColVal2", "ColValue3", "ColValue4" };
List<DataRow> rowsToDelete = new List<DataRow>();

foreach (DataRow row in dt.Rows) {
    if (lstRemoveColumns.Contains(row["ColumnName"].ToString())) {
        rowsToDelete.Add(row);
    }
}

foreach (DataRow row in rowsToDelete) {
    dt.Rows.Remove(row);
}

dt.AcceptChanges();

8
投票
DataRow[] dtr=dtPerson.select("name=Joe");
foreach(var drow in dtr)
{
   drow.delete();
}
dtperson.AcceptChanges();

我希望它会对你有所帮助


4
投票

或者只是将DataTable Row集合转换为列表:

foreach(DataRow dr in dtPerson.Rows.ToList())
{
    if(dr["name"].ToString()=="Joe")
    dr.Delete();
}

3
投票

要从DataTable中删除整行,请执行以下操作

DataTable dt = new DataTable();  //User DataTable
DataRow[] rows;
rows = dt.Select("UserName = 'KarthiK'");  //'UserName' is ColumnName
foreach (DataRow row in rows)
     dt.Rows.Remove(row);

1
投票

问题出在哪里:禁止在foreach循环中删除集合中的项目。

解决方案:要么像Widor写的那样,要么使用两个循环。在第一次通过DataTable时,您只存储(在临时列表中)对要删除的行的引用。然后在临时列表的第二次传递中删除这些行。


1
投票
<asp:GridView ID="grd_item_list" runat="server" AutoGenerateColumns="false" Width="100%" CssClass="table table-bordered table-hover" OnRowCommand="grd_item_list_RowCommand">
    <Columns>
        <asp:TemplateField HeaderText="No">
            <ItemTemplate>
                <%# Container.DataItemIndex + 1 %>
            </ItemTemplate>
        </asp:TemplateField>            
        <asp:TemplateField HeaderText="Actions">
            <ItemTemplate>                    
                <asp:Button ID="remove_itemIndex" OnClientClick="if(confirm('Are You Sure to delete?')==true){ return true;} else{ return false;}" runat="server" class="btn btn-primary" Text="REMOVE" CommandName="REMOVE_ITEM" CommandArgument='<%# Container.DataItemIndex+1 %>' />
            </ItemTemplate>
        </asp:TemplateField>
    </Columns>
</asp:GridView>

 **This is the row binding event**

protected void grd_item_list_RowCommand(object sender, GridViewCommandEventArgs e) {

    item_list_bind_structure();

    if (ViewState["item_list"] != null)
        dt = (DataTable)ViewState["item_list"];


    if (e.CommandName == "REMOVE_ITEM") {
        var RowNum = Convert.ToInt32(e.CommandArgument.ToString()) - 1;

        DataRow dr = dt.Rows[RowNum];
        dr.Delete();

    }

    grd_item_list.DataSource = dt;
    grd_item_list.DataBind();
}
© www.soinside.com 2019 - 2024. All rights reserved.