InvalidOperationException Die Verbindung wurde nicht geschlossen. Der aktuelle Status der Verbindung ist offen

Question

Warum verursacht dieser Code eine ungültige Operation Exception?

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

Sie versuchen, eine Verbindung zu öffnen, die bereits geöffnet ist, dies in Ausnahme führt.

Lösung 1 (empfohlen):

Code Überprüfen Sie, überprüfen Sie alle Teile, bei denen cmd.Connection Verbindung geöffnet wird und sicherzustellen, dass es immer richtig geschlossen ist.

Lösung 2 (quick'n'dirty fix):

vor Zeile

cmd.Connection.Open(); 

fügen Sie die folgende Kontrolle/Bereinigungscode:

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

Answer 2

Es gibt sehr wenig Notwendigkeit Halten Sie die Sql * -Objekte auf Klassenebene, insbesondere basierend auf dem, was Sie zeigen. Sie verlieren auch die Vorteile des Verbindungspoolings, indem Sie versuchen, es selbst zu tun.

Mit dieser Methode Sie die Möglichkeit, Ihre Fehler zu entfernen, weil Sie keine Gegenstände sind nicht

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

teilen