SqlParameter一个容易出错的问题(@2.0)

近日在做培训系统的时候遇到了一个一下的问题,主要涉及的问题如下:

        public Models.Timu ListTimu(IList<Models.TimuDesc> TimuDescs, bool IsKaoshi)
        ...{

            Models.Timu rtnValue = new Models.Timu();
            SqlParameter[] param = ...{ new SqlParameter("@spoint", 0), new SqlParameter("@tid", Guid.Empty) };
            SqlConnection cn = new SqlConnection(CNSTRING);
            cn.Open();
            foreach (Models.TimuDesc td in TimuDescs)
            ...{
                string SQL = string.Format(SQL_SelectTimuNotKaoshiFormat, td.Point1);
                for (int i = 1; i <= 3; i++)
                ...{
                    param[0].Value = i;
                    param[1].Value = td.Guid;
                    DBUtility.SQLHelper.Fill(rtnValue._Timu, cn, CommandType.Text, SQL, param);
                }
            }
            cn.Close();
            return rtnValue;
        }



        public static void Fill(DataTable TypedDataSetsTable, SqlConnection conn, CommandType cmdType, string cmdText, params SqlParameter[] commandParameters)
        ...{
            SqlConnection conection = conn;
            SqlCommand cmd = new SqlCommand();
            PrepareCommand(cmd, conn, null, cmdType, cmdText, commandParameters);
            SqlDataAdapter ada = new SqlDataAdapter(cmd);
            ada.Fill(TypedDataSetsTable);
        }



        private static void PrepareCommand(SqlCommand cmd, SqlConnection conn, SqlTransaction trans, CommandType cmdType, string cmdText, SqlParameter[] cmdParms)
        ...{

            if (conn.State != ConnectionState.Open)
                conn.Open();

            cmd.Connection = conn;
            cmd.CommandText = cmdText;

            if (trans != null)
                cmd.Transaction = trans;

            cmd.CommandType = cmdType;

            if (cmdParms != null)
            ...{
                foreach (SqlParameter parm in cmdParms)
                    cmd.Parameters.Add(parm);
            }
        }

明眼人一看就知道,最后一段是SqlHelper里的一个方法,而第二段代码是我仿造SqlHelper扩展的代码,从这里,当执行的时候,会报以下错误

System.ArgumentException: 另一个 SqlParameterCollection 中已包含 SqlParameter。

SqlParameter[] param = ......{ new SqlParameter("@spoint", 0), new SqlParameter("@tid", Guid.Empty) };
            SqlConnection cn = new SqlConnection(CNSTRING);
            cn.Open();
            foreach (Models.TimuDesc td in TimuDescs)
            ......{
                string SQL = string.Format(SQL_SelectTimuNotKaoshiFormat, td.Point1);
                for (int i = 1; i <= 3; i++)
                ......{
                    param[0].Value = i;
                    param[1].Value = td.Guid;
                    DBUtility.SQLHelper.Fill(rtnValue._Timu, cn, CommandType.Text, SQL, param);
                }
            }
            cn.Close();

这里是唯一存入参数的地方肯定这里就问题所在了,我之所以这样写法是因为希望节省一点 SqlParameter 的new时间，而在Fill 方法中SqlCommand的对象是被new出来的，所以每次调用new方法Fill 方法 cmd (SqlCommand) 都是新的唯一能做到多次调用都能被记录住的“公用地方”就是SqlConnection对象了。 但是没有理由我没次调用Fill 都要Open和Close Connection 一次吧，于是我想试一下把这些都 new 一下看看。

于是将代码改成如下：

        public Models.Timu ListTimu(IList<Models.TimuDesc> TimuDescs, bool IsKaoshi)
        ...{

            Models.Timu rtnValue = new Models.Timu();
            
            SqlConnection cn = new SqlConnection(CNSTRING);
            cn.Open();
            foreach (Models.TimuDesc td in TimuDescs)
            ...{
                string SQL = string.Format(SQL_SelectTimuNotKaoshiFormat, td.Point1);
                for (int i = 1; i <= 3; i++)
                ...{
                    SqlParameter[] param
                        = ...{ new SqlParameter("@spoint", i), new SqlParameter("@tid", td.Guid) };
                    DBUtility.SQLHelper.Fill(rtnValue._Timu, cn, CommandType.Text, SQL, param);
                }
            }
            cn.Close();
            return rtnValue;
        }

运行，问题解决