InvalidOperationException La connexion n'a pas été fermée. L'état actuel de la connexion est ouvert.

Question

Pourquoi ce code lance-t-il une exception d'opération non valide?

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

Vous essayez d'ouvrir une connexion qui est déjà ouvert, il en résulte une exception.

Solution 1 (recommandé):

votre code Inspectez, vérifiez toutes les pièces où la connexion cmd.Connection est ouverte et assurez-vous qu'il est toujours fermé.

Solution 2 (quick'n'dirty fixe):

avant la ligne

cmd.Connection.Open(); 

ajouter le code de contrôle/nettoyage suivant:

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

Answer 2

Il y a besoin de très peu Garder les objets Sql * au niveau de la classe, surtout en fonction de ce que vous montrez. Vous perdrez également les avantages de la mise en commun des connexions en essayant de le faire vous-même.

Avec cette méthode, vous supprimez la possibilité de votre erreur parce que vous n'êtes pas partager des objets

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