C#：从SQL DataTable删除DataGridView选中行及变量作用域问题

问题 #

我有一个绑定了SQL DataTable数据的DataGridView，还有用于编辑、添加、删除数据的按钮。核心需求是实现删除按钮功能，删除用户选中行对应的SQL DataTable数据，且只需要用到每行第一列（Column.Index=0）的数据。

我尝试把选中行的Row.Index存入数组，方便后续定位删除，但数组是foreach循环内的局部变量，循环外无法访问（比如执行Console.WriteLine时提示“SelectedRowIndexes不存在于当前上下文”）。请问思路是否有误？有无解决办法？

原代码如下：

private void BtnRemoveClick(object sender, System.EventArgs e)
{
    foreach (DataGridViewRow row in datagridview1.SelectedRows)
    {
        int[] SelectedRowIndexes = { row.Index };
    }


    for (int i=0; i<SelectedRowIndexes.Length; i++)
    {
        Console.WriteLine(SelectedRowIndexes[i]); // error CS0103: The name 'SelectedRowIndexes' does not exist in the current context
    }
}

解决方案 #

1. 修复变量作用域问题 #

你遇到的错误核心是变量作用域问题：SelectedRowIndexes在foreach循环内部声明，只能在循环块内访问，循环外自然找不到它。

正确的做法是把集合（推荐用List<int>，比数组更灵活，因为选中行数量不确定）声明在循环外面：

private void BtnRemoveClick(object sender, System.EventArgs e)
{
    // 把集合声明在循环外，扩大作用域
    List<int> selectedRowIndexes = new List<int>();
    foreach (DataGridViewRow row in datagridview1.SelectedRows)
    {
        selectedRowIndexes.Add(row.Index);
    }

    // 现在可以正常访问集合了
    foreach (int index in selectedRowIndexes)
    {
        Console.WriteLine(index);
    }
}

2. 直接删除DataTable数据的最优写法 #

你最终目的是删除绑定的SQL DataTable数据，不需要单独存索引再二次遍历，而且要注意倒序删除（如果正序删除，前面的行删除后，后面行的索引会前移，导致漏删或删错行）：

private void BtnRemoveClick(object sender, System.EventArgs e)
{
    // 先获取绑定的DataTable
    DataTable dt = (DataTable)datagridview1.DataSource;
    
    // 倒序遍历选中行，避免索引偏移问题
    for (int i = datagridview1.SelectedRows.Count - 1; i >= 0; i--)
    {
        DataGridViewRow selectedRow = datagridview1.SelectedRows[i];
        // 获取当前行对应的DataRow
        DataRow dr = ((DataRowView)selectedRow.DataBoundItem).Row;
        
        // 从DataTable中删除该行
        dt.Rows.Remove(dr);
        
        // 如果需要，还可以同步删除数据库中的数据（这里假设第一列是主键ID）
        int id = Convert.ToInt32(dr[0]); // Column.Index=0，取第一列数据
        // 执行SQL删除语句：DELETE FROM 表名 WHERE ID = @id
        // 这里需要你自己实现数据库操作逻辑，比如用SqlCommand执行
    }
    
    // 刷新DataGridView显示
    datagridview1.Refresh();
}

3. 思路纠正 #

你的初始思路（存索引）本身没问题，但没必要绕弯子——直接通过DataBoundItem获取选中行对应的DataRow，就能直接操作绑定的DataTable，比单独存索引更高效，也避免了作用域和索引偏移的问题。

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