ADO.NET中SQL连接内循环执行命令报@medicine参数已声明错误如何解决

错误原因 #

你抛出该错误的核心是：SqlCommand的参数集合在单次实例化后是全局共享的，你在foreach循环里反复调用AddWithValue添加同名的@medicine参数，第一次循环执行后参数已经存在，第二次循环重复添加就触发了变量名重复的冲突。

解决方案1：最小改动适配现有逻辑 #

每次执行SQL前清空参数集合，完全贴合你现有的逐次执行逻辑，改动手脚最小：

using (SqlConnection conn = new SqlConnection(cstring))
{
    using(SqlCommand cmd = new SqlCommand("Update Medicine set Quantity = Quantity - 1 where Name = @medicine", conn))
    {
        conn.Open();
        foreach(string item in cartMedicine)
        {
            // 清空之前添加的参数，避免重名冲突
            cmd.Parameters.Clear();
            cmd.Parameters.AddWithValue("@medicine", item);
            cmd.ExecuteNonQuery();
        }
        conn.Close();
        listMedicine.Items.Clear();
    }
}

解决方案2：性能更优的写法 #

不用反复清空/创建参数对象，提前在循环外定义参数，循环内仅修改参数值即可，减少不必要的资源开销：

using (SqlConnection conn = new SqlConnection(cstring))
{
    using(SqlCommand cmd = new SqlCommand("Update Medicine set Quantity = Quantity - 1 where Name = @medicine", conn))
    {
        conn.Open();
        // 循环外提前定义参数，指定和表结构匹配的字段类型、长度，避免AddWithValue的类型推断隐患
        var medicineParam = new SqlParameter("@medicine", SqlDbType.NVarChar, 50);
        cmd.Parameters.Add(medicineParam);
        foreach(string item in cartMedicine)
        {
            // 循环内仅修改参数值
            medicineParam.Value = item;
            cmd.ExecuteNonQuery();
        }
        conn.Close();
        listMedicine.Items.Clear();
    }
}

额外优化建议 #

如果加购列表的药品数量较多，建议搭配事务保证操作原子性，避免部分扣减成功、部分扣减失败导致的库存数据不一致问题：

using (SqlConnection conn = new SqlConnection(cstring))
{
    conn.Open();
    // 开启事务保证所有扣减操作同成功同失败
    using (SqlTransaction tran = conn.BeginTransaction())
    using(SqlCommand cmd = new SqlCommand("Update Medicine set Quantity = Quantity - 1 where Name = @medicine", conn, tran))
    {
        var medicineParam = new SqlParameter("@medicine", SqlDbType.NVarChar, 50);
        cmd.Parameters.Add(medicineParam);
        try
        {
            foreach(string item in cartMedicine)
            {
                medicineParam.Value = item;
                cmd.ExecuteNonQuery();
            }
            // 所有操作执行成功后提交事务
            tran.Commit();
            listMedicine.Items.Clear();
        }
        catch
        {
            // 任意步骤报错则回滚所有扣减操作
            tran.Rollback();
            throw;
        }
        finally
        {
            conn.Close();
        }
    }
}

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