Mithilfe von C#'s 'mit' - Anweisung mit einem benutzerdefinierten Objekt-Funktion, Muss ich IDisposable implementieren?

Ich habe eine sqlConnection-manager-Klasse in etwa so:

public class SQLConn {
  public string connStr = System.Configuration.ConfigurationSettings.AppSettings["ConnectionString"];

  private SqlConnection sqlConn;

  public SqlConnection Connection()
  {
      sqlConn = new SqlConnection(connStr);

      return sqlConn;
  }

  public void Open()
  {
        sqlConn .Open();
  }
}

Wenn ich eine Funktion mit der 'mit' - Anweisung wie:

var conn = new SQLConn();

using (conn.Connection()) 
{ 
    String query = "Select * from table";
    objSql = new SqlCommand(query, conn.Connection());      

    conn.Open(); 
    DoSomething(); 
}

Macht die using-Anweisung entsorgen Sie die Verbindung automatisch, da conn.Connection() gibt eine SqlConnection-Objekt? Oder muss ich implementieren IDisposable und eine benutzerdefinierte Dispose-Methode auf die SqlConn Klasse?

Ist dies auch eine gute Möglichkeit überhaupt? Ich arbeite mit legacy-code und ich bin nicht in der Lage zu verwenden ein ORM noch aber ist es ein Weg, um dies zu vereinfachen, bestehende Muster zu verwalten/erstellen von SQL-verbindungen?

InformationsquelleAutor zulkamal | 2009-05-27
  1. 12

    Den using - Anweisung wird die Letzte Art des Ausdrucks - D. H. was auch immer zurück .Connection(); wenn diese etwas zurückgibt, wird IDisposable, dann sind Sie OK.

    Dem compiler sagen, wenn Sie es falsch ;-p (es wird nicht lassen Sie using auf etwas, das nicht IDisposable).

    Sollte man wohl aufpassen, wo Sie erstellen zwei Anschlüsse:

    using (var c = conn.Connection()) //<==edit
{ 
    String query = "Select * from table";
    objSql = new SqlCommand(query, c); //<==edit

    c.Open(); 
    DoSomething(); 
}

    und möglicherweise:

    public SqlConnection Connection()
{
  if(sqlConn == null) sqlConn = new SqlConnection(connStr); //<== edit
  return sqlConn;
}
    InformationsquelleAutor Marc Gravell
  2. 5

    Es funktionieren wird, aber nach der using {} Sie mit der linken, eine sqlConn, dass intern hält Entsorgt SqlConnection. Nicht wirklich nützlich situation

    InformationsquelleAutor Henk Holterman
  3. 1

    Dein code ist falsch!

    sollte so etwas wie dieses:

    Dim conn as New SQLConn();
Dim sqlConnection New SQLConnection();

sqlConnection = conn.Connection();

using (sqlConnection) 
{ 
    String query = "Select * from table";
    objSql = new SqlCommand(query, sqlConnection);      

    conn.Open(); 
    DoSomething(); 
}

    Diese Weise die using-Anweisung wird über die Verbindung am Ende.

    • Warum schaffen Sie zwei SqlConnections? Mindestens einer von Ihnen wird nicht entsorgt werden
    • +1 Das problem nicht mit dem schreiben using (conn.Connection()), sondern auch durch die Tatsache, dass die Connection() - Methode erstellt ein neues Objekt jedes mal, wenn es aufgerufen wird, und in dem original-Beitrag, die Verbindung, die unter der "mithilfe" ist nicht eine andere Instanz als die Verbindung tatsächlich genutzt wird, innerhalb des Blocks.
    • Sie wahrscheinlich nicht brauchen, die "New SQLConnection", dann
    • Ja, wir brauchen nicht neue SqlConnection - sorry dafür (ich nutze C# und nicht nur verstehen, VB überhaupt) @Noam Gal - Mein Punkt war, dass der bereitgestellte code ist nutzlos, da die SQLConn Klasse zurück nicht einem einzelnen SqlConnection-Instanz. Also den code in die Frage, die wir haben, eine Verbindung (die ist entsorgt, indem Sie die using-Anweisung) und eine andere, die ist vom code verwendet und nicht entsorgt am Ende...
    InformationsquelleAutor Pavel Nikolov
  4. 1

    Zu klären, was gesagt wird über:

    Jedes Objekt, das Sie verwenden müssen, um mit der Verwendung entsorgt werden sollte am Ende der using-Anweisung. Der compiler muss daher sicherstellen, dass Ihre Art die IDisposable-Schnittstelle implementiert, wenn es sieht mit auf das Typ-Objekt, oder es wird dich nicht gehen lassen.

    InformationsquelleAutor Yohanis
  5. 1

    beantworten deine headline-Frage, Sie muss IDisposable implementieren Sie in der Klasse, deren Objekt Sie sind mit "mit". Andernfalls erhalten Sie eine compile-time-Fehler.

    Dann, ja, "mit" entledigen sich Ihrer SqlConnection am Ende des Blocks. Denken Sie an "mit" als "try-finally": es gibt einen impliziten Aufruf von Dispose() in der "finally" - block.

    Schließlich, sauberer code wäre:

    using( SqlConnection = new SqlConnection( connStr ) {
    //do something
}

    Zumindest die Leser des Codes nicht haben, um den mentalen Aufwand zu realisieren, da Henk Holterman wies darauf hin, dass Ihre SQLConn-Objekt hält eine Referenz auf eine entsorgt-Verbindung.

    InformationsquelleAutor azheglov
  6. 0

    Nein musst du nicht, solange das zurückgegebene Objekt ist IDisposable.

    Das zurückgegebene Objekt muss nicht zu implementieren IDisposable, aber dann die using-block würde keinem Zweck dienen.

    • Die Letzte Art der Ausdruck nicht brauchen zu implementieren IDisposable: Fehler 1 'Bar': Typ verwendet, in einer Anweisung verwenden, müssen implizit konvertierbar " - System.IDisposable
    • Hmmm, interessant, ist das veränderte Verhalten von früheren Versionen des Compilers, oder habe ich nur miss verstanden es alle zusammen?
    • Ich bin mir nicht bewusst, eine änderung (aber ich habe nicht geschaut bei der 1.2 Spezifikation in letzter Zeit). Sind Sie vielleicht denken foreach? wo ist der enumerator könnte werden Einweg - (und wenn dem so ist, wird entsorgt), aber muss nicht sein? Oder vielleicht die Tatsache, dass der zurückgegebene Wert darf null sein?
    • In der Tat, ich bezweifle, dass es geändert - da würde deutlich, dass eine bedeutende änderung, die gestoppt werden einige vorhandenen code aus kompilieren; das C# - team versuchen sehr hart, um zu vermeiden, dass...
    • Wissen Sie, ich bin wahrscheinlich denken foreach 🙂 Verdammt Reflektor versteckt alle details lässt Sie vergessen!
    InformationsquelleAutor leppie
  7. 0

    Vom MSDN:

    Dem Objekt bereitgestellt, um die Verwendung
    Anweisung implementieren muss
    IDisposable-Schnittstelle.

    Haben Sie nicht rufen Sie die Dispose () - obwohl, die using-Anweisung implizit tut dies für Sie.

    InformationsquelleAutor annakata
  8. 0

    Scheint es, wird die Verbindung ordnungsgemäß geschlossen, aber dies wird nicht empfohlen:

    Können Sie die Ressource instanziieren
    Objekt und übergeben Sie die variable
    die using-Anweisung, aber dies ist nicht eine
    best-practice. In diesem Fall, die
    Objekt bleibt im Rahmen der Kontrolle
    Blätter der using-block verwenden, obwohl es
    wird wahrscheinlich keinen Zugriff mehr auf
    seine nicht verwaltete Ressourcen. In anderen
    Worten, es wird nicht mehr voll
    initialisiert. Wenn Sie versuchen, verwenden Sie die
    Objekt außerhalb des using-Blocks, Sie
    Risiko verursachen eine Ausnahme zu sein
    geworfen. Aus diesem Grund, es ist
    in der Regel besser zu instanziieren
    Objekt in der using-Anweisung und
    limit dem Umfang nach auf den using-block.

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

    InformationsquelleAutor Alexander Prokofyev
Schreibe einen Kommentar