循环错误执行过程

时间:2016-01-28 08:13:07

标签: c# sql .net stored-procedures ado.net

我在循环内执行程序时遇到问题。我的要求是将数据插入表中。第一列是字母表,我试图进入循环,接下来的三列是所有行的共同点。但是我在第一次迭代后得到错误,说变量必须是唯一的。

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'已经宣布。变量名在查询批处理或存储过程中必须是唯一的。

4 个答案:

答案 0 :(得分:3)

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

因此,不必在循环中添加它,只需设置其值:

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

另请注意,在大多数情况下使用AddWithValue并不是一个好主意,因为它会尝试从传递的值中推导出参数类型。在大多数情况下,最好明确设置参数类型。

答案 1 :(得分:0)

我认为你一遍又一遍地添加@Alpha参数...... 首次添加参数并设置值。 每次后续迭代,只需设置值。

例如,第一次迭代......

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

后续迭代......

cmd.Parameters["@Alpha"].Value = c.ToString();
祝你好运。

答案 2 :(得分:0)

作为替代方案,您可以在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个参数并再次清除它们。

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

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

答案 3 :(得分:0)

您必须清除变量才能再次添加:

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