所有行都作为一行存储过程c#

Question

我将数据表的数据发送到存储过程，该存储过程只是将数据插入表中，但出现错误

过程或函数 usp_InsertData 指定了太多参数。

当我使用分析器查看发生了什么时，我发现所有行都作为一行进入存储过程。

String strConnString = ConfigurationManager.ConnectionStrings["MyConnectionString"].ConnectionString;
SqlConnection con = new SqlConnection();

public static void InsertData(DataTable dt)
{
        String strConnString = ConfigurationManager.ConnectionStrings["MyConnectionString"].ConnectionString;
        SqlConnection con = new SqlConnection();
        con.ConnectionString = strConnString;

        SqlCommand cmd = new SqlCommand();
        cmd.CommandType = CommandType.StoredProcedure;
        cmd.CommandText = ("usp_InsertData");
        cmd.Connection = con;

        con.Open();

        for (int i = 1; i < dt.Rows.Count; i++)
        {
            cmd.Parameters.AddWithValue("@FirstIssue", dt.Rows[i][0].ToString());
            cmd.Parameters.AddWithValue("@RTitle", dt.Rows[i][1].ToString());
            cmd.Parameters.AddWithValue("@RFirstName", dt.Rows[i][2].ToString());
            cmd.Parameters.AddWithValue("@RLastName", dt.Rows[i][3].ToString());
            cmd.Parameters.AddWithValue("@RAddress1", dt.Rows[i][4].ToString());
            cmd.Parameters.AddWithValue("@RAddress2", dt.Rows[i][5].ToString());
            cmd.Parameters.AddWithValue("@RAddress3", dt.Rows[i][6].ToString());
            cmd.Parameters.AddWithValue("@RCity", dt.Rows[i][7].ToString());
            cmd.Parameters.AddWithValue("@RZip", dt.Rows[i][8].ToString());
            cmd.Parameters.AddWithValue("@RCounty", dt.Rows[i][9].ToString());
            cmd.Parameters.AddWithValue("@RCountry", dt.Rows[i][10].ToString());
            cmd.Parameters.AddWithValue("@PTitle", dt.Rows[i][11].ToString());
            cmd.Parameters.AddWithValue("@PFirstName", dt.Rows[i][12].ToString());
            cmd.Parameters.AddWithValue("@PLastName", dt.Rows[i][13].ToString());
            cmd.Parameters.AddWithValue("@PAddress1", dt.Rows[i][14].ToString());
            cmd.Parameters.AddWithValue("@PAddress2", dt.Rows[i][15].ToString());
            cmd.Parameters.AddWithValue("@PAddress3", dt.Rows[i][16].ToString());
            cmd.Parameters.AddWithValue("@PCity", dt.Rows[i][17].ToString());
            cmd.Parameters.AddWithValue("@PZip", dt.Rows[i][18].ToString());
            cmd.Parameters.AddWithValue("@PCounty", dt.Rows[i][19].ToString());
            cmd.Parameters.AddWithValue("@PCountry", dt.Rows[i][20].ToString());
            cmd.Parameters.AddWithValue("@SubscriberAccountNumber", dt.Rows[i][21].ToString());
            cmd.Parameters.AddWithValue("@OnSaleDate", dt.Rows[i][22].ToString());
            cmd.Parameters.AddWithValue("@Subscriber", dt.Rows[i][23].ToString());

            try
            {
                cmd.ExecuteNonQuery();
            }
            catch (Exception ex)
            {
                throw ex;
            }
        }

        MessageBox.Show("data inserted");
        con.Close();
        con.Dispose();
    }

谁能告诉我如何解决这个问题？

Answer 1

您可以使这变得更简单并且性能更好。与其通过痛苦的行来插入行，不如使用表值参数。您将创建自己的自定义表类型以匹配您要插入的列的类型。然后，您只需调用一个过程并将整个数据集作为一个参数传递，而不是所有这些迭代。

https://msdn.microsoft.com/en-us/library/bb675163%28v=vs.110%29.aspx https://msdn.microsoft.com/en-us/library/bb510489.aspx

Answer 2

您的代码在到达第二个循环时会引发异常，因为命令的参数集合已经填充了参数名称，并且再次调用具有相同名称的 AddWithValue 会引发异常。

您可以在每个循环中调用Parameters.Clear() 方法，但更好的解决方案是在进入循环之前创建所有参数，然后在循环内设置当前行的值而不重新创建参数列表。

String strConnString = ConfigurationManager.ConnectionStrings["MyConnectionString"].ConnectionString;
using(SqlConnection con = new SqlConnection(strConnString))
using(SqlCommand cmd = new SqlCommand("usp_InsertData", con))
{
    cmd.CommandType = CommandType.StoredProcedure;
    con.Open();

    cmd.Parameters.Add("@FirstIssue", SqlDbType.NVarChar);
    .... add the other parameters specifying the correct datatype .....

    for (int i = 1; i < dt.Rows.Count; i++)
    {
        cmd.Parameters["@FirstIssue"].Value = dt.Rows[i][0].ToString());
        ... other parameters values follows here ....

        // If you don't use the exception don't catch it....
        cmd.ExecuteNonQuery();
    }
}

还要注意，像连接和命令这样的一次性对象应该在 using 块内创建，以确保在出现异常时也能正确关闭和处理

Answer 3

您正在为每个循环的参数列表添加一个具有相同键的参数。

在循环末尾添加cmd.Parameters.Clear()语句，在下一个循环开始之前清除参数，然后它应该可以工作了。

顺便说一句，您的代码中存在很多性能问题。