Tut SqlDataAdapter schließen der SqlConnection-nach der Fill () - Funktion?

Tut SqlDataAdapter schließen Sie die SqlConnection nach der Fill() Funktion oder muss ich es dann selbst schließen?

string cnStr = @"Data Source=TEST;Initial Catalog=Suite;Persist Security Info=True;User ID=app;Password=Immmmmm";
cn = new SqlConnection(cnStr);
SqlCommand cmd = new SqlCommand("SELECT TOP 10 * FROM Date", cn);
SqlDataAdapter adapter = new SqlDataAdapter(cmd);

DataSet ds = new DataSet();
adapter.Fill(ds);

cn.Close() //????????

Console.WriteLine(ds.Tables[0].Rows.Count);
Console.WriteLine(cn.State);
InformationsquelleAutor Wachburn | 2011-09-12
  1. 14

    In Ihrer aktuellen Nutzung, wird es eng für Sie:

    Wenn die IDbConnection
    geschlossen ist, bevor Sie Füllen, heißt, es wird geöffnet, um Daten abrufen und
    dann geschlossen. Wenn die Verbindung geöffnet ist, bevor Sie Füllen, die aufgerufen wird,
    geöffnet bleibt.

    http://msdn.microsoft.com/en-us/library/zxkb3c3d.aspx

    Ich denke, es ist immer besser, sich explizit eingehen, es sich mit einem using Aussage:

    using (SqlConnection conn = new SqlConnection(""))
{
    conn.Open();

    //Do Stuff.

} //Closes here on dispose.

    Dies ist oft besser lesbar und nicht sich verlassen auf Menschen Verständnis der inneren Abläufe des SqlDataAdapter.Fill nur die using Anweisung und verbindungen.

    Jedoch, wenn Sie wissen wird die Verbindung geschlossen wird, bevor der adapter verwendet werden (wie in, Sie haben gerade die Verbindung erstellt) und es ist nicht für irgendetwas anderes, dein code ist vollkommen sicher und gültig.

    Persönlich, ich würde etwas schreiben wie dieses:

        string cnStr = "Data Source=TEST;Initial Catalog=Suite;Persist Security Info=True;User ID=app;Password=Immmmmm";
    DataSet ds = new DataSet();

    using (SqlConnection cn = new SqlConnection(cnStr))
    using (SqlCommand cmd = new SqlCommand("SELECT TOP 10 * FROM Date", cn))
    using (SqlDataAdapter adapter = new SqlDataAdapter(cmd))
    { 
        conn.Open();
        adapter.Fill(ds);       
    }
    • aber wenn ich folgenden code schreiben, ohne mit Bau-cn oder was auch immer, Geschlossen Zustand vor dem beenden der Funktion
    • in der Tat, Beratung, MSDN überprüft diese. Sorry, ich habe nicht fangen diese das erste mal um.
    • Es Griffe schließen es für Sie, wenn der Befehl erfolgreich ausgeführt, aber was ist in einem Ausnahme Fall? (Deadlocks, Timeouts, etc.)
    • Die using - Anweisung ruft dispose auf Ausnahme, wie gut es einfach kompiliert in einen try-finally-block unter der Haube.
    • Sorry, war mir nicht klar, ich meinte vor der Einführung der using block. Sie begann, indem Sie sagen: In your current usage, it will close for you aber ich wollte klären, ob das wäre der Fall, wenn eine Ausnahme geworfen wird.
    • Oh, in diesem Fall, Nein ist es nicht - es bleibt für die GC zu behandeln, vorausgesetzt, Ausnahmen sind behandelt und nicht die Ursache für die massive app-crash.
    • Nur um klar zu sein: in deinem letzten code-block, der conn.Open(); ist eigentlich nicht nötig. Es ist einfach da, im Falle andere Programmierer wissen nicht, dass adapter.Fill(ds); öffnen (und schließen) der Verbindung selbst. Ist das richtig?
    • Ja, ziemlich viel. Ich glaube, es funktioniert in beide Richtungen, persönliche Präferenz wirklich.
    • Es gibt keine Notwendigkeit für drei separate Anweisungen. Die Verwendung kann eine durch Komma getrennte Liste von Ausdrücken zu erstellen, die IDisposable-Objekten, und es wird erstellen Sie verschachtelte try-finally-Anweisungen für Sie. Eine Verwendung wird hier genügen.
    • Habe es gerade ausprobiert in dem Fall, dass etwas sich geändert hatte, für C# 7, aber leider immer noch nur true, wenn der Text freigegeben ist, wie using (IDisposable conn = new SqlConnection(), comm = new SqlCommand()) Sie nicht using (SqlConnection conn = new SqlConnection(), SqlCommand comm = new SqlCommand()) so drei using Aussagen sind in diesem Fall verpflichtet.
    • Danke @AdamHouldsworth, ich wusste nicht, dass. Die Verwendung ist nicht ganz so cool wie ich dachte. Ich Stimme drei aufzuhalten der Benutzung ist die beste Antwort. Könnten Sie erklären, Sie alle als IDisposable, dann niedergeschlagen in den block, aber das ist sehr hässlich. Sie könnte erklären Sie vor der Verwendung, aber das ist gefährlich, weil die Variablen, die wäre noch im Rahmen, nachdem Sie bereits entsorgt wurden. Fühlen Sie sich frei, um downvote mein Kommentar oben, SO lässt mich nicht 🙂

    InformationsquelleAutor Adam Houldsworth
  2. 4

    Als ich wissen, Sie brauchen, um die Verbindung zu schließen, indem Sie eigene

    Beste Weg das zu tun ist

    using(SqlConnection con = new SqlConnection())
{
   //you code 
}

    dies wird schließen Sie die Verbindung automatisch

    using-block in C# kommt sehr handlich, während Sie sich mit Einweg-Objekte. Einweg-Objekte sind jene Objekte, die explizit die Ressourcen freizugeben, die Sie verwenden, wenn Sie aufgerufen zu entsorgen. Wie wir wissen .Net garbage collection ist nicht-deterministisch, so kann man nicht Vorhersagen, Wann genau wird das Objekt von der garbage Collection freigegeben.

    Lesen Sie diesen Beitrag für weitere details : Verständnis 'mit' block in C#

    InformationsquelleAutor Pranay Rana
Schreibe einen Kommentar