Près de SqlConnection dans le Enfin, lors de l'utilisation de “l'Aide”

Je veux fermer l'occurrence de SqlConnection dans le Enfin, depuis l'aide pas vraiment le fermer et le pool de connexion est pleine. mais je ne le réalisent pas quel est le bon chemin à fo que depuis la conn objet n'est pas accessible à tout plus la enfin, la section.

try 
{
    using (var conn = new SqlConnection(_dbconnstr)) 
    {
        //...
    }
}
catch (Exception ex)
{
    //...
}
finally 
{
    conn.Close //?!?!?!?!???
}
Vous avez un autre problème. using est implémentée en interne comme un try/finally paire, avec de Disposer d'être appelé dans le finally bloc. Et pour SqlConnection, Close et Dispose sont interchangeables.
Comme mentionné comme une réponse, je vous suggère de vérifier la même chose à propos de DataReaders : la façon dont vous êtes à la fermeture de la Connexion est ok.
Oh, j'ai complètement raté le point de le remplir à l'Aide de bloc à l'intérieur d'un try{ } bloc. Je l'ai lu parce que le support sont en retrait d'une étrange façon.

OriginalL'auteur Erez | 2011-04-06

  1. 12
    using (var conn = new SqlConnection(_dbconnstr)) 
{
    //code
}

    est expaded:

    SqlConnection conn = new SqlConnection(_dbconnstr);
try
{
    //code
}
finally
{
    conn.Dispose();
}

    Donc vous devez gérer les erreurs, mais vous pouvez oublier la fermeture de la connexion.

    (Un petit pinaille: Votre extension est incorrect. Si le constructeur lève alors la tentative d'appel Dispose dans le finally bloc jeter un NullReferenceException et de cacher la véritable erreur. Le compilateur se met à l'affectation à l'extérieur de la try bloc, puis effectue une valeur null vérifier dans le finally bloc de trop, avant d'appeler Dispose.)

    OriginalL'auteur Andrey

  2. 8

    Vous n'avez pas besoin de fermer conn dans le finally bloc. Le using bloc de gérer la fermeture de la connexion pour vous. (En fait, vous n'avez probablement pas besoin de la try...finally à tous dans ce cas, sauf si vous avez d'autres ressources qui doivent traiter avec dans le finally.)

    La using bloc de traduire quelque chose comme ceci:

    var conn = new SqlConnection(/*...*/);
try
{
    //...
}
finally
{
    if (conn != null)
        ((IDisposable)conn).Dispose();
}

    La Dispose méthode de la SqlConnection objet sera appelée dans le finally bloc, et le Dispose méthode fait appel Close pour vous.

    Toujours initialiser à l'intérieur du bloc try.
    Je suis le seul à en expliquant ce qui est généré par le compilateur, et qui effectue la tâche immédiatement avant le try bloc, exactement comme je l'ai montré. Vous devez vérifier les faits avant de downvoting!

    OriginalL'auteur LukeH

  3. 2

    De sortir d'une aide de bloquer les appels .Dispose() sur l'objet, pour un SqlConnection va fermer la connexion et toutes les ressources.

    De sorte que le try, finally bloc n'est pas nécessaire.

    ce n'est pas correct. bloc try est nécessaire si le système veut faire quelque chose avec l'erreur. Par exemple, l'enregistrement du message d'exception qui parfois très utile pendant le mode de débogage.

    OriginalL'auteur CD..

  4. 1

    Selon ma compréhension de la méthode dispose() de l'objet de connexion fermer la Connexion. Vous n'avez pas besoin de Connexion d'appel.Fermer explicitement.

    Disposer d'appels .Close() en interne. La différence est que vous ne pouvez pas le rouvrir quand il est éliminé.
    c'est vrai. Je n'avais pas pensé à ce sujet. Mais si nous sommes à l'aide de "l'aide" mot-clé, pas de point d'appel de la Connexion.Close() à droite?
    Yep, c'est correct

    OriginalL'auteur Anuraj

  5. 1

    Autant que je sache, le suivant à l'aide de déclaration:

    using (var conn = new SqlConnection(_dbconnstr)) 
{

}

    Est équivalent à:

    SqlConnection conn;
try
{
    //`using` scope operations are executed here
    conn = new SqlConnection(_dbconnstr));

}
catch
{
    //exceptions are bubbled
    throw;
}
finally
{
    //Dispose is always called
    conn.Dispose();
}
    Mal. C'est equiviliant à faire: var conn = new SqlConnection(_dbconnstr); { ... } conn.Dispose(); Il n'ont rien à voir avec try/catch. {} représente un champ d'application en C#.
    Jørgensen: cela signifierait assez bien tous réponse à cette question est mal par votre raisonnement? Mal. t tout pour faire avec (au moins) try / finally.
    Non, seulement environ la moitié d'entre eux. (À noter également le commentaire a été mis à jour)
    Jørgensen: Votre premier commentaire n'a pas de sens - tout newing en place une instance et s'attendent à être en mesure d'atteindre le Dispose appeler n'est pas comment using travaux: using garanties Dispose sera appelé, et ce qui se passe à l'intérieur d'un finally bloc.

    OriginalL'auteur Grant Thomas

  6. 1

    La façon dont vous êtes à la clôture de la connexion avec Using est Ok. Peut-être que vous pourriez avoir oublié de fermer certains DataReaders à la place ?

    OriginalL'auteur Larry