在循环中执行过程时出错

Question

我在循环内执行过程时遇到问题。我的要求是将数据插入 table。第一列是我尝试进入循环的字母表，接下来的三列对于所有行都是通用的。但是我在第一次迭代后遇到错误，说变量必须是唯一的。

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);
}

错误信息：

The variable name '@Alpha' has already been declared. Variable names must be unique within a query batch or stored procedure.

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。 It may generate unexpected and surprising results sometimes。使用 Add 方法重载来指定您的参数类型及其大小。

还可以使用 using statement 自动处理您的连接和命令，而不是手动调用 Close 或 Dispose 方法。

Answer 4

你必须清除变量才能再次添加像这样的东西：

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();
    }

在循环中执行过程时出错

error executing procedure in the loop

.net

c#

sql

ado.net

stored-procedures