在循环中执行过程时出错

本文关键字：出错 过程 执行 循环 | 更新日期: 2023-09-27 18:32:18

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

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"已被声明。变量名称在查询批处理或存储过程中必须是唯一的。

在循环中执行过程时出错

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

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

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

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

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

例如第一次迭代....

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

后续迭代....

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

祝你好运。

或者，您可以在 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。它有时可能会产生意想不到和令人惊讶的结果。使用Add方法重载指定参数类型及其大小。

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

您必须清除变量才能再次添加 像这样： 

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();
    }
相关文章：