SqlCommand 参数 - 必须声明标量变量“@positionX”。

Question

我知道这个问题有很多问题。也许再问也没用，但是在尝试了很多之后我无法找到解决方案，所以如果有人帮助我，我将非常感激。代码如下：

String personQuery =
    "INSERT INTO Person(X, Y, Width, Height, Confidence) VALUES (@positionX, @positionY, @width, @height, @confidence)";

SqlCommand personCmd = new SqlCommand(personQuery, connection);

foreach (var objectItem in items)
{
    if (objectItem.Type == "person")
    {

          personCmd.Parameters.AddWithValue("@positionX", objectItem.X);
          personCmd.Parameters.AddWithValue("@positionY", objectItem.Y);
          personCmd.Parameters.AddWithValue("@width", objectItem.Width);
          personCmd.Parameters.AddWithValue("@height", objectItem.Height);
          personCmd.Parameters.AddWithValue("@confidence", objectItem.Confidence);
          personCmd.Parameters.Clear();

          connection.Open();
          personCmd.ExecuteNonQuery();
          connection.Close();

    }
}

运行此代码后，我收到此错误：必须声明标量变量“@positionX”。' 问题是什么？提前谢谢...

Answer 1

查看代码后；问题来了

personCmd.Parameters.Clear();

此行必须在 personCmd.ExecuteNonQuery(); 之后

这可以是完整的代码：

connection.Open();
    foreach (var objectItem in items)
    {
        personCmd.Parameters.AddWithValue("@positionX", objectItem.X);
        personCmd.Parameters.AddWithValue("@positionY", objectItem.Y);
        personCmd.Parameters.AddWithValue("@width", objectItem.Width);
        personCmd.Parameters.AddWithValue("@height", objectItem.Height);
        personCmd.Parameters.AddWithValue("@confidence", objectItem.Confidence);
        personCmd.ExecuteNonQuery();
        personCmd.Parameters.Clear();
    }

    connection.Close();

Answer 2

除了在需要执行参数之前错误地调用 Clear 的问题之外，您的代码还有许多改进之处。

• 没有必要在循环中不断地重新声明和清除参数。您可以声明一次命令和参数，然后在循环的每次迭代中重新分配值
• 不要检查循环内的项目类型，而是在迭代 foreach 之前过滤掉任何不需要的项目。
• 由于命令和连接是IDisposable，因此在using 语句内确定命令和连接的范围将保证任何代码路径（包括异常）的资源尽快释放。
• 由于此代码是 I/O 密集型代码，如果可能，请将代码切换为 async 以简化线程池的使用。所有 ADO.Net 方法都有Async 版本（即OpenAsync、ExecuteNonQueryAsync 等，可以是awaited）

const string personQuery = "INSERT INTO Person(X, Y, Width, Height, Confidence) " +
                           "VALUES (@positionX, @positionY, @width, @height, @confidence)";

// Define a long lived connection and command. Both are Disposable.
using (var connection = new SqlConnection(myConnectionString))
using (var personCmd = new SqlCommand(personQuery, connection))
{
    // Open the connection just the once
    connection.Open();

    // Define the command parameters without assigning them yet
    // Check the data types against your Sql schema ...
    personCmd.Parameters.Add("@positionX", SqlDbType.Int);
    personCmd.Parameters.Add("@positionY", SqlDbType.Int);
    ... etc other parameters

    // Filter the objects before you loop them. 
    // Better would be .Where(i => i is Person) if you have subclasses
    foreach (var objectItem in items.Where(i => i.Type == "person")) 
    {
        personCmd["@positionX"].Value = objectItem.X;
        personCmd["@positionY"].Value = objectItem.Y;
        ... assign other params
        personCmd.ExecuteNonQuery();
    }
} // personCmd and connection will be automatically closed