c# - 使用 c#/ASP.net 应用程序获取 SqlParameter 已包含在另一个 SqlParameterCollection 错误中

Question

我有一个将参数添加到 cmd.parameters 集合的通用方法。当 2 个人同时访问同一个存储过程时，出现以下错误:

SqlParameter 已包含在另一个 SqlParameterCollection 中。

我已经在 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(网络表单)应用程序，因此代码反射(reflect)了这一点。

错误信息的开头是这样的:

[ArgumentException: The SqlParameter is already contained by another SqlParameterCollection.] System.Data.SqlClient.SqlParameterCollection.Validate(Int32 index, Object value) +5955779 System.Data.SqlClient.SqlParameterCollection.Add(Object value) +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);
    }

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

internal void AddParameter(string name, double? value) {

internal void AddParameter(string name, byte[] value) {

等...

Answer 1

查看问题中的代码......其中很多都没有实现作者的意图。例如，try/catch/finally 完全没有值(value)，因为 catch block 只是重新抛出相同的异常，使用 Fill() 方法意味着不需要 finally block 。其他代码也有类似的问题。

除了事务之外，您可以将代码减少到这样，假设 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 对象引起的。您需要查看其他代码来解决问题。