ASP.NET中使用复选框删除DataTable行的异常问题求助

嘿，这个问题我之前碰到过好多次，核心原因是正向遍历GridView行并同步删除DataTable行时，索引会发生偏移，尤其是全选删除的时候，这个问题会直接爆发出来。

问题根源拆解 #

举个简单例子：假设你的DataTable有3行，索引是0、1、2。当你删除索引0的行后，原来的索引1会自动变成新的索引0，索引2变成1。但你的循环还是按照GridView原来的row.RowIndex去删除，比如接下来要删原来的索引1（现在已经是新的索引0了），这时候DataTable里已经没有索引1的行，自然就抛出“位置X处没有行”的错误。

全选时一次性要删多行，索引混乱得更快，错误触发后DataTable的状态已经异常，所以后续单个删除也会失效。

两种可行的解决方案 #

方案1：倒序遍历GridView行

倒序遍历从最后一行开始处理，这样删除后面的行不会影响前面行的索引，完美避开偏移问题：

// 从最后一行往第一行遍历
for (int i = GridView1.Rows.Count - 1; i >= 0; i--)
{
    GridViewRow row = GridView1.Rows[i];
    CheckBox chk = row.FindControl("chkInvoice") as CheckBox;
    if (chk != null && chk.Checked)
    {
        dt.Rows.RemoveAt(i);
    }
}
GridView1.DataSource = dt;
GridView1.DataBind();

方案2：先收集要删除的索引，再批量删除

先把所有需要删除的行索引存到列表里，再统一删除（注意要倒序删，避免索引偏移），这种方法逻辑更清晰，也方便后续扩展：

List<int> rowsToDelete = new List<int>();

// 第一步：收集所有要删除的行索引
foreach (GridViewRow row in GridView1.Rows)
{
    CheckBox chk = row.FindControl("chkInvoice") as CheckBox;
    if (chk != null && chk.Checked)
    {
        rowsToDelete.Add(row.RowIndex);
    }
}

// 第二步：倒序删除收集到的索引
foreach (int index in rowsToDelete.OrderByDescending(x => x))
{
    dt.Rows.RemoveAt(index);
}

GridView1.DataSource = dt;
GridView1.DataBind();

补充说明 #

为什么单个删除没问题？因为只删一行的时候，索引偏移不会影响后续循环（循环只执行一次），但全选时多次删除就会触发索引越界。解决了遍历逻辑的问题后，后续单个删除也会恢复正常。

内容的提问来源于stack exchange，提问作者user9356915