ASP.NET C#中GridView删除Access数据库数据无响应问题排查

看起来你的GridView删除功能没生效，主要有几个关键问题需要排查，我帮你一步步分析：

1. 事件绑定错误（最可能的原因） #

你当前用的是SelectedIndexChanged事件，但GridView的Delete按钮默认触发的是**RowDeleting/RowDeleted事件**，而且需要确保按钮的CommandName属性设置为"Delete"。点击Delete按钮并不会触发SelectedIndexChanged，所以你的删除逻辑根本没执行。

2. SQL拼接的风险与潜在错误 #

用string.Format拼接SQL语句存在两个问题：

• SQL注入风险：如果path或username包含单引号（比如O'Neil），会直接导致SQL语法错误，执行失败。
• 字符串转义问题：特殊字符会破坏SQL语句结构，导致匹配不到数据库里的数据，自然删不掉。

3. 数据库资源未正确释放 #

你的InsertUpdateDelete方法里没有用using语句管理数据库连接和命令，可能导致连接泄漏，甚至偶尔执行失败。

4. 单元格索引可能不正确 #

Cells[3]是否真的对应path列？GridView的列索引是从0开始的，而且如果有自动生成的列、隐藏列或者模板列，实际索引可能和你预期的不一样，导致获取的path值错误，匹配不到数据库里的数据。

修正后的代码示例 #

第一步：调整GridView的Delete按钮和事件绑定

确保你的GridView里的Delete按钮设置了CommandName="Delete"，然后绑定RowDeleting事件：

<asp:GridView ID="GridView1" runat="server" OnRowDeleting="GridView1_RowDeleting">
    <Columns>
        <!-- 其他列 -->
        <asp:CommandField ShowDeleteButton="True" />
        <!-- 或者自定义按钮 -->
        <asp:ButtonField Text="DELETE" CommandName="Delete" />
    </Columns>
</asp:GridView>

第二步：修改后台事件与删除逻辑

protected void GridView1_RowDeleting(object sender, GridViewDeleteEventArgs e)
{
    // 用DataKey存储关键数据比取单元格更可靠，避免列索引变化的问题
    string path = GridView1.DataKeys[e.RowIndex]["path"].ToString();
    string username = Session["user"]?.ToString();
    
    if (!string.IsNullOrEmpty(username) && !string.IsNullOrEmpty(path))
    {
        Cart cart = new Cart();
        cart.Deleteitem(path, username);
        // 删除后重新绑定GridView刷新数据
        GridView1.DataBind();
    }
}

public void Deleteitem(string path, string username)
{
    // 使用参数化查询，彻底避免注入和语法错误
    string sql = "DELETE FROM Cart WHERE path = ? AND userid = ?";
    DBC dbc = new DBC();
    dbc.InsertUpdateDelete(sql, path, username);
}

// 修改InsertUpdateDelete方法，支持参数化并正确释放资源
public void InsertUpdateDelete(string sql, params object[] parameters)
{
    // using语句会自动释放连接和命令，避免资源泄漏
    using (OleDbConnection con = new OleDbConnection(你的数据库连接字符串))
    using (OleDbCommand cmd = new OleDbCommand(sql, con))
    {
        // 添加参数
        for (int i = 0; i < parameters.Length; i++)
        {
            cmd.Parameters.AddWithValue($"param{i}", parameters[i]);
        }
        con.Open();
        cmd.ExecuteNonQuery();
    }
}

额外建议 #

• 调试时可以在Deleteitem方法里打印path和username的值，确认是否和数据库里的记录匹配。
• 在数据库操作代码里添加try-catch捕获异常，方便排查执行时的具体错误。
• 尽量用DataKeys存储主键/关键数据，比直接读取单元格文本更稳定。

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