c# - InvalidOperationException 连接未关闭。连接的当前状态为打开

Question

为什么此代码会引发无效操作异常？

private SqlCommand cmd; // initialized in the class constructor

public void End(string spSendEventNotificationEmail) {
  try {
    cmd.CommandText = spSendEventNotificationEmail;
    cmd.Parameters.Clear();
    cmd.Parameters.Add("@packetID", SqlDbType.Int).Value = _packetID;
    cmd.Parameters.Add("@statusID", SqlDbType.Int).Value = _statusID;
    cmd.Parameters.Add("@website", SqlDbType.NVarChar, 100).Value = Tools.NextStep;
    cmd.Connection.Open();
    cmd.ExecuteNonQuery();
  } finally {
    cmd.Connection.Close();
    cmd.Parameters.Clear();
    cmd.Dispose();
  }
  endCall = true;
}

Answer 1

您正在尝试打开一个已经打开的连接，这会导致异常。

解决方案1（推荐）：

检查您的代码，检查cmd.Connection打开连接的所有部分并确保它始终正确关闭。

解决方案 2（快速修复）：

行前

cmd.Connection.Open();

添加以下检查/清理代码：

if (cmd.Connection.State == ConnectionState.Open)
{
    cmd.Connection.Close();
}

Answer 2

很少需要将 Sql* 对象保持在类级别，尤其是根据您所展示的内容。尝试自己进行连接池也将失去连接池的好处。

使用此方法，您可以消除错误的可能性，因为您没有共享任何对象

private readonly _connectionString = "...";

public void End(string spSendEventNotificationEmail) {
  using(var conn = new SqlConnection(_connectionString))
  using(var cmd = conn.CreateCommand())
  {
    cmd.CommandText = spSendEventNotificationEmail;
    cmd.Parameters.Add("@packetID", SqlDbType.Int).Value = _packetID;
    cmd.Parameters.Add("@statusID", SqlDbType.Int).Value = _statusID;
    cmd.Parameters.Add("@website", SqlDbType.NVarChar, 100).Value = Tools.NextStep;
    conn.Open();
    cmd.ExecuteNonQuery();
  }
  endCall = true;
}