Question

我有一个通用方法，它将参数添加到cmd.parameters集合中。当两个人同时点击相同的存储过程时，出现以下错误：

另一个SqlParameterCollection已包含SqlParameter。

我已经在StackOverflow内以及网络上的其他位置进行了搜索，但到目前为止，我仍然无法解决此问题。我的代码如下：

protected DataSet ExecuteDataSet(string StoredProcName, List<TParameter> Params) {
bool internalOpen = false;
DataSet resultDataSet = null;
TDataAdapter dataAdapter;
TCommand cmd;

try {
    resultDataSet = new DataSet();
    dataAdapter = new TDataAdapter();
    cmd = new TCommand();
    cmd.CommandType = CommandType.StoredProcedure;
    cmd.CommandText = StoredProcName;

    if (transaction != null) {
        cmd.Transaction = transaction;
        cmd.Connection = transaction.Connection;
    } else {
        cmd.Connection = connection;
    }

    if (Params != null && Params.Count > 0) {
        foreach (TParameter param in Params) {
            cmd.Parameters.Add(param);
        }
    }

    dataAdapter.SelectCommand = cmd;

    if (connection.State == ConnectionState.Closed) {
        connection.Open();
        internalOpen = true;
    }

    dataAdapter.Fill(resultDataSet);
    dataAdapter.SelectCommand.Parameters.Clear();

    cmd.Parameters.Clear();
    return resultDataSet;
} catch {
    throw;
} finally {
    if (internalOpen) {
        connection.Close();
    }

}

该错误是在foreach循环中发生的，但是尽管清除了参数并应用了其他一些尝试的修复程序，但我仍无法阻止此错误的发生。

这是使用以下方法从另一种方法调用的：

 result = base.ExecuteDataSet(procName,this.ConvertArrayList(this.ParamsList));

这是我正在使用的较旧的c＃/ asp.net（网络表单）应用程序，因此代码反映了这一点。

错误消息开始如下：

[ArgumentException：另一个其他SqlParameterCollection已包含SqlParameter。] System.Data.SqlClient.SqlParameterCollection.Validate（Int32索引，对象值）+5955779 System.Data.SqlClient.SqlParameterCollection.Add（对象值）+34

我正在测试使用2个浏览器同时以2个用户身份登录，然后单击指向可提取多个记录的页面的链接进行测试。其中一个用户收到错误，而另一个用户得到正确的页面，并按预期返回了结果。

任何帮助将不胜感激。抱歉，如果我还没有足够好地解释这一点，但是我整天都在关注这件事，并且无法取得进展。

谢谢。

编辑：

像这样设置/获取ParamsList：

public ArrayList ParamsList {
        get {
            //Check to see if the parameters list has been initialised.
            if (m_paramsList == null) {
                //Create a new empty parameters list to pass back.
                m_paramsList = new ArrayList();
            }
            return m_paramsList;
        }
        set {
            m_paramsList = value;
        }
    }

并且像这样填充：

internal void AddParameter(string name, string value) {
        IDbDataParameter param = CreateStringParameter(name);

        param.Value = GetValueFromString(value);

        Dal.ParamsList.Add(param);
    }

对于每种类型的参数类型。...

内部无效AddParameter（字符串名称，双精度值）{

内部无效AddParameter（字符串名称，字节[]值）{

等...

Answer 1

仔细阅读问题中的代码...很多都没有实现作者可能想要的目的。例如，try / catch / finally完全一文不值，因为catch块只是重新引发了相同的异常，并且使用Fill()方法意味着不需要finally块。其他代码也有类似的问题。

除交易外，假设TCommand和公司完全实现了ADO.Net提供程序，则可以将代码减少到仅此数目，减少的代码实际上会提高性能，安全性以及实用程序：

protected DataSet ExecuteDataSet(string StoredProcName, IEnumerable<TParameter> Params = null) 
{    
    DataSet resultDataSet = new DataSet();
    using (var cn = new TConnection(connection.ConnectionString))
    using (var cmd = new TCommand(StoredProcName, cn))
    using (var adapter = new TAdapter(cmd))
    {
        cmd.CommandType = CommandType.StoredProcedure;

        if (Params != null) 
        {
            foreach (TParameter param in Params)
            {
                cmd.Parameters.Add(param);
            }
        }    
        adapter.Fill(resultDataSet);
    }
    return resultDataSet;        
}

但是我们确实拥有那个transaction的值，这足以打破这里的using模式。因此，您需要有效地将代码长度加倍，以解决这两种变体。是的， using模式确实非常重要，您可以有效地将代码长度加倍以保持其长度：

protected DataSet ExecuteDataSet(string StoredProcName, IEnumerable<TParameter> Params = null) 
{  
    DataSet resultDataSet = new DataSet(); 
    if (transaction == null)
    { 
        using (var cn = new TConnection(connection.ConnectionString))
        using (var cmd = new TCommand(StoredProcName, cn))
        using (var adapter = new TAdapter(cmd))
        {
            cmd.CommandType = CommandType.StoredProcedure;

            if (Params != null) 
            {
                foreach (TParameter param in Params)
                {
                    cmd.Parameters.Add(param);
                }
            }    
            adapter.Fill(resultDataSet);
        }
    }
    else
    {
        using (var cmd = new TCommand(StoredProcName, transaction.Connection))
        using (var adapter = new TAdapter(cmd))
        {
            cmd.Transaction = transaction;
            cmd.CommandType = CommandType.StoredProcedure;

            if (Params != null) 
            {
                foreach (TParameter param in Params)
                {
                    cmd.Parameters.Add(param);
                }
            }    
            adapter.Fill(resultDataSet);
        }
    }
    return resultDataSet; 
}

最后，这些都不能解决您的问题。您看到的问题是由其他地方的代码过于努力地重用Parameter对象所引起的。您需要查看其他代码来解决此问题。

Answer 2

听起来this.ParamsList中的对象正在为每个查询重用。如果您显示该列表的定义和填充方式，我们可以确认，但这就是它的样子。

解决方案基本上是不这样做。为每个查询使用全新的SqlParameter对象重建参数列表。

否则，您将必须采用某种锁定方式以确保两个请求不能同时执行，但这似乎是一个更差的解决方案。

Answer 3

我猜想TCommand是SqlCommand的一种，而TParameter是SqlParameter的一种。我认为问题出在这里：

    foreach (TParameter param in Params) {
        cmd.Parameters.Add(param);
    }

param是Params的成员，Param是一种引用类型；这意味着您可能在多次调用ExecuteDataSet的过程中共享相同的Params列表。相反，请执行以下操作：

    foreach (TParameter param in Params) {
        cmd.Parameters.Add(new SqlParameter(param.ParameterName, param.SqlDbType, param.Size)).Value = param.Value;
    }

此外，您可能潜在地在多个线程之间共享相同的SqlConnection，这可能是一个问题。通常，我建议在ExecuteDataSet方法中创建一个新的SqlConnection。新的SqlConnection仍然可以在任何正在进行的事务中注册。

最后，对所有IDisposable对象使用“ using”块，例如SqlCommand，SqlCommand，DataSet等。

使用C＃/ ASP.net应用程序的另一个SqlParameterCollection错误已包含SqlParameter

3 个答案: