循环中执行过程错误

Question

在循环内部执行过程时，我面临着问题。我的要求是将数据插入到表中。第一列是字母，我试图在循环中得到，接下来的三列对所有行都很常见。但是，在第一次迭代之后，我得到了一个错误，即变量必须是唯一的。

string str = Properties.Settings.Default.con;
SqlConnection con = new SqlConnection(str);

char[] az = Enumerable.Range('A', 'Z' - 'A' + 1).Select(i => (Char)i).ToArray();


SqlCommand cmd = new SqlCommand("Execute InsertPurchase  @ShipTo,@StoreName,@desc,@Alpha", con);
try
{
    cmd.Parameters.AddWithValue("@ShipTo", txtstoreto.Text);
    cmd.Parameters.AddWithValue("@StoreName", txtstorefrom.Text);
    cmd.Parameters.AddWithValue("@desc", Globals.s_desc.ToString());
    con.Open();
    foreach (var c in az)
    {
        cmd.Parameters.AddWithValue("@Alpha", c.ToString());
        cmd.ExecuteNonQuery();
    }
    con.Close();
    j = true;
}
catch (Exception ex)
{
    MessageBox.Show(ex.Message);
}

错误消息：

变量名“@Alpha”已经声明。变量名在查询批处理或存储过程中必须是唯一的。

Answer 1

您一次又一次地在循环中添加参数Alpha。

因此，与其将其添加到循环中，您只需设置其值：

cmd.Parameters.Add("@Alpha", SqlDbType.VarChar);
con.Open();
foreach (var c in az)
{
    cmd.Parameters["@Alpha"].Value = c.ToString();
    cmd.ExecuteNonQuery();
}

还请注意，在大多数情况下，使用AddWithValue不是一个好主意，因为它试图从传递的值中推断参数类型。在大多数情况下，最好显式设置参数类型。

Answer 2

我想你一遍又一遍地添加@Alpha参数..。第一次添加参数并设置值。每次后续的迭代，只需设置值。

例如第一次迭代..。

cmd.Parameters.AddWithValue("@Alpha", c.ToString());

随后的迭代..。

cmd.Parameters["@Alpha"].Value = c.ToString();

祝好运。

Answer 3

作为另一种选择，您可以移动for循环中的所有参数声明，并在每次迭代中对它们进行Clear。

foreach (var c in az)
{
    cmd.Parameters.Clear();
    cmd.Parameters.AddWithValue("@ShipTo", txtstoreto.Text);
    cmd.Parameters.AddWithValue("@StoreName", txtstorefrom.Text);
    cmd.Parameters.AddWithValue("@desc", Globals.s_desc.ToString());
    cmd.Parameters.AddWithValue("@Alpha", c.ToString());
    cmd.ExecuteNonQuery();
}

当然，这在每次迭代中增加了不必要的前3参数，并再次清除它们。

顺便说一句，不要尽可能多地使用AddWithValue。 generate unexpected and surprising results sometimes。使用Add方法重载来指定参数类型及其大小。

还可以使用 statement自动释放连接和命令，而不是手动调用Close或Dispose方法。