ASP.NET事务结合foreach循环保存带复选框GridView数据报错

看起来你遇到的问题大概率是重复添加SQL参数导致的，结合你的代码来看，主要有几个关键问题需要修正：

核心问题分析 #

• 重复参数添加：在foreach循环里每次调用AddWithValue，当有多行满足同一个复选框条件时，会重复添加同名参数，SQLCommand会抛出“已添加具有相同键的项”这类错误。
• 命令执行时机错误：你现在是循环完所有行后才执行ExecuteNonQuery，这意味着只有最后一次添加的参数会生效，前面的行数据根本没被插入，完全不符合业务逻辑。
• 事务逻辑不严谨：当前的提交判断i>0 && j>0太苛刻——如果只有现金或只有支票的行需要保存，另一个命令的影响行数是0，就会导致事务无法提交；而且没有异常捕获，一旦执行出错不会自动回滚事务。
• 控件绑定错误：原代码直接用cash.Checked，这很可能不是当前行的复选框，而是页面全局控件！会导致所有行都判断同一个控件的状态，完全不符合需求。

修正后的代码示例 #

using (SqlConnection con = new SqlConnection("你的数据库连接字符串"))
{
    con.Open();
    using (SqlTransaction trns = con.BeginTransaction())
    {
        try
        {
            // 提前初始化命令并绑定事务
            SqlCommand cashCmd = new SqlCommand("procedure", con, trns);
            SqlCommand chequeCmd = new SqlCommand("procedure1", con, trns);

            // 提前声明参数（指定类型更安全，避免AddWithValue的类型推断问题）
            cashCmd.Parameters.Add("@parameter", SqlDbType.VarChar, 50); // 根据实际字段类型调整
            chequeCmd.Parameters.Add("@parameter1", SqlDbType.VarChar, 50);

            foreach (GridViewRow row in gvrecept.Rows)
            {
                // 必须获取当前行的复选框实例
                CheckBox cashChk = row.FindControl("cash") as CheckBox;
                CheckBox chequeChk = row.FindControl("cheque") as CheckBox;

                if (cashChk != null && cashChk.Checked)
                {
                    // 重置参数值，而非重复添加参数
                    cashCmd.Parameters["@parameter"].Value = row.Cells[0].Text; // 替换成当前行的实际数据
                    cashCmd.ExecuteNonQuery();
                }
                else if (chequeChk != null && chequeChk.Checked)
                {
                    chequeCmd.Parameters["@parameter1"].Value = row.Cells[0].Text; // 替换成当前行的实际数据
                    chequeCmd.ExecuteNonQuery();
                }
            }

            // 只要没有异常就提交事务
            trns.Commit();
            // 这里可以添加成功提示逻辑，比如弹出提示框
        }
        catch (Exception ex)
        {
            // 出错时强制回滚事务，避免数据不一致
            trns.Rollback();
            // 处理异常：可以记录日志、给用户提示等
            throw ex; // 根据业务需求调整异常处理方式
        }
    }
}

额外优化建议 #

• 避免AddWithValue：它可能导致SQL Server的参数类型推断错误，甚至引发性能问题（无法重用执行计划），建议始终指定明确的参数类型和长度。
• 空值处理：如果参数值可能为空，记得设置DBNull.Value，避免传入C#的null导致报错。
• 事务范围：用using语句包裹连接和事务，确保即使出现异常，资源也能自动释放。

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