Question

我遇到错误：

ExecuteNonQuery需要打开且可用的连接。
连接的当前状态为打开。

我使用了con.Open();，但是遇到了同样的错误。
请帮我。

static string connectionString = ConfigurationManager.AppSettings["MessageDB"].ToString();
static SqlConnection _sqlconn = new SqlConnection(connectionString);

public static void Message(MessageLog messageLog)
{
    try
    {
        _sqlconn = new SqlConnection(connectionString);
        _sqlconn.Open();

        SqlCommand cmd = _sqlconn.CreateCommand();
        cmd.CommandType = CommandType.StoredProcedure;
        cmd.CommandText = "INSERT_MESSAGE";

        SqlParameter APPID = cmd.CreateParameter();
        APPID.ParameterName = @"@MessageID";
        APPID.DbType = DbType.Int16;
        APPID.Direction = ParameterDirection.Input;
        APPID.Value = messageLog.MessageID;
        cmd.Parameters.Add(MessageID);

        cmd.ExecuteNonQuery();

        _sqlconn.Close();
    }
    catch (Exception ex)
    {
        if (_sqlconn.State == ConnectionState.Open)
        {
            _sqlconn.Close();
        }

        ExceptionLogger.LogException(ex);
    }
}

Answer 1

恕我直言，您的问题是使用静态连接变量。 static SqlConnection _sqlconn = new SqlConnection(connectionString);，然后您又重新分配了它，这没有任何意义。我会稍微更改您的代码：

static string connectionString = ConfigurationManager.AppSettings["MessageDB"].ToString();
public static void Message(MessageLog messageLog)
{
   using(var _sqlconn = new SqlConnection(connectionString))
   {
      using(var cmd = new SqlCommand())
      {
          cmd.CommandType = CommandType.StoredProcedure;
          cmd.CommandText = "INSERT_MESSAGE";
          cmd.Connection = _sqlconn;

          cmd.Parameters.Add("@MessageID", DbType.Int16).Value = messageLog.MessageID;
          try
          {
              if(_sqlconn.State == ConnectionState.Closed) _sqlconn.Open()

              cmd.ExecuteNonQuery();

          }
          catch (Exception ex)
          {
               ExceptionLogger.LogException(ex);
          } 
          finally
          { 
               if (_sqlconn.State == ConnectionState.Open)
               {
                  _sqlconn.Close();
               }
          }
      }
   }
}

该语法的好处？

所有一次性用品将被处置。
无论异常或正常执行流程如何，连接都会始终关闭。
未尝试打开已打开的连接。
连接始终可用，因为它每次都从连接池中获取，而不是“冻结”静态变量。

Answer 2

我认为您在此提供商上已有连接试试这个：

_sqlconn = new SqlConnection(connectionString);
if(_sqlconn .State == ConnectionState.Closed)            
_sqlconn.Open();
            SqlCommand cmd = _sqlconn.CreateCommand();
            cmd.CommandType = CommandType.StoredProcedure;
            cmd.CommandText = "INSERT_MESSAGE";
            SqlParameter APPID = cmd.CreateParameter();
            APPID.ParameterName = @"@MessageID";
            APPID.DbType = DbType.Int16;
            APPID.Direction = ParameterDirection.Input;
            APPID.Value = messageLog.MessageID;
            cmd.Parameters.Add(MessageID);

            cmd.ExecuteNonQuery();
            _sqlconn.Close();
        }

        catch (Exception ex)
        {
            if (_sqlconn.State == ConnectionState.Open)
            {
                _sqlconn.Close();
            }
            ExceptionLogger.LogException(ex);
        }

ExecuteNonQuery需要一个开放且可用的连接。连接的当前状态为打开

2 个答案: