Question

我有一个运行一些ExecuteNonQuery命令的循环。偶尔会返回错误消息：

 ExecuteNonQuery requires an open and available connection. The connections 
 current state is closed.

这是我的代码：

private void MyTimerEventHandler(object src, ElapsedEventArgs a)
{
    sqlCon = new SqlConnection("server=" + appConfig.sqlServer + ";Trusted_Connection=yes;database=testdb;connection timeout=30;");
    sqlCon.Open();

    foreach (TagManager tm in tagManagerList)
    {
        foreach (Tag tag in tm.getTags())
        {
             SqlCommand insCmd = new SqlCommand("INSERT INTO tag_values (tag_id, value, time) Values (@tagId, @tagValue, @tagTime);", sqlCon);
             insCmd.Parameters.Add(new SqlParameter("@tagId", tag.tagNameId));
             insCmd.Parameters.Add(new SqlParameter("@tagValue", tag.value));
             insCmd.Parameters.Add(new SqlParameter("@tagTime", tag.time));

             insCmd.ExecuteNonQuery();
        }
    }

    sqlCon.Close();
}

此代码在每隔15秒运行的Timer的事件处理程序中执行。如果有任何不同，计时器将由GC.KeepAlive()保持活动状态。

Answer 1

为每个计时器回调创建一个新的连接对象：

private void MyTimerEventHandler(object src, ElapsedEventArgs a)
{
       SqlConnection sqlCon = new SqlConnection( [...]

重用连接通常是个坏主意。在您的情况下，如果在另一个线程中使用该连接，则可能会遇到竞争条件。创建新连接不应影响性能，因为它们是从连接池中提取的。

Answer 2

您可以在执行查询之前检查连接是否仍然打开

   if (sqlCon.State == System.Data.ConnectionState.Open)
                    {
                        insCmd.ExecuteNonQuery();
                    }
                    else
                    {
                        sqlCon.Open();
                        insCmd.ExecuteNonQuery();
                    }
                }

Answer 3

您不是通过使用StringBuilder准备插入语句并将其作为文本传递给SQL命令，以便通过SQL SERVER发送一次以进行插入。在这种情况下，对于所有600个循环，您需要连接到DB一次。只是一个想法

随机获取ExecuteNonQuery需要一个开放且可用的连接

3 个答案: