ASP.NET GridView选中复选框删除数据，数据库记录未删除且无报错

• 主键绑定完全错位
  当前删除逻辑中，Label1绑定的是数据库表的type字段值，你却将该值转为整数作为enquiryID的匹配条件执行删除，因为传入参数与真实主键enquiryID不匹配，SQL执行后永远命中0条记录，既不会删除数据，也不会抛出报错。
  现有GridView模板中完全没有绑定主键enquiryID，根本无法获取每一行对应的真实主键值。
• 回发时重复绑定导致勾选状态丢失
  若Page_Load事件中每次页面加载都调用refreshdata()绑定GridView，且没有加!IsPostBack判断，会触发按钮回发时先执行Page_Load逻辑重新绑定GridView，将用户勾选的复选框状态全部重置为未选中，后续在按钮点击事件中读取到的chck.Checked永远为false，根本无法进入删除逻辑块。
• 循环内提前刷新导致遍历中断
  当前代码在遍历行的过程中，只要遇到选中项就立刻调用refreshdata()重新绑定GridView，此时正在遍历的GridView1.Rows集合会被全新数据替换，后续遍历逻辑直接中断，哪怕删除逻辑正确，也只能处理第一行选中的数据。
• 连接使用逻辑不规范
  循环内部每次遇到选中项就手动打开、关闭数据库连接，没有异常捕获，一旦连接状态异常，逻辑中断后也看不到具体错误提示。

1. 调整aspx页面的GridView配置 #

给GridView指定主键字段，新增隐藏的主键绑定列，确保能获取行对应的真实enquiryID，修改后的列结构参考：

<asp:GridView ID="GridView1" runat="server" AutoGenerateColumns="False" Width="468px" DataKeyNames="enquiryID">
    <Columns>
        <asp:TemplateField Visible="false">
            <ItemTemplate>
                <asp:Label ID="LblEnquiryID" runat="server" Text='<%# Bind("enquiryID") %>'></asp:Label>
            </ItemTemplate>
        </asp:TemplateField>
        <asp:TemplateField HeaderText="Type">
            <EditItemTemplate>
                <asp:TextBox ID="TextBox1" runat="server" Text='<%# Bind("type") %>'></asp:TextBox>
            </EditItemTemplate>
            <ItemTemplate>
                <asp:Label ID="Label1" runat="server" Text='<%# Bind("type") %>'></asp:Label>
            </ItemTemplate>
        </asp:TemplateField>
        <!-- 原有Description、Email、Role列的结构保持不变 -->
        <asp:TemplateField HeaderText="选择">
            <ItemTemplate>
                <asp:CheckBox ID="CheckBox1" runat="server" />
            </ItemTemplate>
        </asp:TemplateField>
    </Columns>
</asp:GridView>

2. 修正后台代码逻辑 #

首先修改Page_Load事件，仅首次页面加载时绑定GridView，回发时不重复绑定，保留复选框的勾选状态：

protected void Page_Load(object sender, EventArgs e)
{
    // 仅首次加载时绑定数据，回发时不再重复绑定
    if (!IsPostBack)
    {
        refreshdata();
    }
}

再修改删除按钮的点击逻辑，遵循“先收集选中项、再统一执行删除、最后刷新列表”的流程，避免遍历中断：

protected void Button1_Click(object sender, EventArgs e)
{
    // 遍历收集所有选中行的主键，过程中不重新绑定GridView
    List<int> deleteIdList = new List<int>();
    foreach (GridViewRow gvrow in GridView1.Rows)
    {
        CheckBox chck = gvrow.FindControl("CheckBox1") as CheckBox;
        if (chck != null && chck.Checked)
        {
            var idLabel = gvrow.FindControl("LblEnquiryID") as Label;
            if (idLabel != null)
            {
                deleteIdList.Add(int.Parse(idLabel.Text));
            }
        }
    }

    // 未选中任何记录时直接提示返回
    if (deleteIdList.Count == 0)
    {
        ClientScript.RegisterStartupScript(this.GetType(), "tip", "alert('请先勾选要删除的记录');", true);
        return;
    }

    // 统一打开一次连接，批量执行所有删除操作
    con.Open();
    foreach (int id in deleteIdList)
    {
        SqlCommand cmd = new SqlCommand("DELETE FROM Enquiry WHERE enquiryID=@id", con);
        cmd.Parameters.AddWithValue("@id", id);
        cmd.ExecuteNonQuery();
    }
    con.Close();

    // 全部删除完成后再刷新列表
    refreshdata();
}

• 检查数据库连接字符串con指向的实例，确认与服务器资源管理器中查看的数据库为同一个，避免连接到其他测试环境
• 检查数据库账号权限，确保拥有Enquiry表的删除操作权限
• 执行删除时读取cmd.ExecuteNonQuery()的返回值（即受影响的行数），确认SQL语句是否真的命中了目标记录

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