Est-ce que "connection.Close()" est le moyen le plus sûr ou le plus sûr de fermer une connexion SQL?

Question

J'utilise

connection.Close(); 

Mais ceci est mon premier projet en .NET et je ne sais pas si je ferme correctement.

Je ne veux pas que mon site web meure instantanément après l'avoir diffusé sur un vrai hébergement.

D'habitude, je le fais de cette façon:

SqlConnection conn = new SqlConnection(ConfigurationManager.ConnectionStrings["ConnectionString"].ToString()); 

    SqlCommand cmd = new SqlCommand(); 

    cmd.Connection = conn; 

    // <some code> 

    conn.Open(); 
    /<some code> 
    conn.Close(); 

Answer 1

Vous devez placer la connexion dans une déclaration à l'aide;

using(var connection = new SqlConnection) 
{ 
    connection.Open(); 
    ...other code using it here. 
} 

L'instruction using garantit que SqlConnection est éliminée lorsque vous avez terminé. Dans votre scénario (en fonction de l'endroit où vous placez la méthode close), si une exception est fermée, elle n'atteindra jamais la méthode Close() et la connexion restera ouverte.

Encore mieux, puisque vous construisez un site web. Il devrait y avoir une méthode sur l'objet Request (qui devrait être disponible) appelée RegisterForDispose. Cela va automatiquement éliminer la connexion (qui la ferme) lorsque la requête se termine. Vous pouvez l'utiliser comme ceci:

var connection = new SqlConnection(); 
Request.RegisterForDispose(connection); 

Les deux accomplissent la même chose à la fin, mais la seconde permet plus de flexibilité.

Answer 2

Faites ce que Kevin a dit et mettre votre appel en .Open() à l'intérieur du bloc à l'aide:

using(var conn = new SqlConnection(ConfigurationManager.ConnectionStrings["ConnectionString"].ToString())) 
{ 
    var cmd = new SqlCommand("my sql command here", conn); 

    conn.Open(); 

    cmd.(whatever method you are using) 
}