Disposed DbContext in MVC Controller, welcher Weg "besser"?

Question

In MVC 5 werden die Gerüste Codes haben so etwas wie:

public class MyController : Controller 
{ 
    private MyContext db = new MyContext(); 

    protected override void Dispose(bool disposing) 
    { 
     if (disposing) 
     { 
      db.Dispose(); 
     } 
     base.Dispose(disposing); 
    } 

sonst, ich brauche

using (var db = new MyContext()) 
{...} 

in jeder Aktion haben.

Die Codes sehen gut aus, daher muss ich nicht in jeder Aktion verwenden. Wird dies jedoch von den Programmierern bevorzugt, oder hat ein solcher Stil einen Vorteil gegenüber der Verwendung in jeder Aktion, die den dbcontext verwenden muss?

Answer 1

Beide Lösungen sind gut - beide Lösung wird Db-Kontext verfügen. Aber meiner Meinung nach wird die zweite Option besser sein - Sie erstellen den db-Kontext genau dort, wo Sie müssen.

Aber was ist, wenn eine andere Klasse (einige Service-Klasse) auch Db-Kontext verwendet. Es empfiehlt sich, einen db-Kontext für die gesamte Webanforderung zu verwenden. In diesem Fall sollten Sie den zuvor erstellten db-Kontext an alle Klassen übergeben, die den db-Kontext verwenden, um das Erstellen eines neuen db-Kontexts in allen Klassen zu verhindern. Also werde ich die Verwendung von IoC-Containern in Betracht ziehen. IoC-Container löst Ihre Abhängigkeiten und verwaltet auch die Objektlebensdauer. Bellow

ich einige IoC Container aufgelistet:

• Castle Windsor
• Spring Framework
• StructureMap

Answer 2

Eine Using-Anweisung ruft die Dispose() -Methode am Ende des Using-Blocks automatisch auf. Die Using-Anweisung ruft die Dispose() -Methode auf, selbst wenn ein Fehler im Code aufgetreten ist.

Answer 3

In Bezug auf Best Practices, sollten Sie unbedingt die Vorlage eingerüstete Sachen verwenden und nicht mit Zohan mit dem Muster using(){}, es sei denn, Sie haben einen wirklich guten zwingenden Grund. Beide Lösungen bringen das gleiche Ergebnis, aber beide sind keine guten Lösungen. Der Grund, warum die Vorlage eine einzige DbContext hat, ist es einfacher, Test zu machen - heres ein Beispiel:

public class SomeController : Controller 
{ 
    private ApplicationDbContext db; 

    public AccountController() 
    { 
     db = new ApplicationDbContext(); 
    } 

    public AccountController(ApplicationDbContext context) 
    { 
     db = context; 
    } 
} 

Der erste Konstruktor ohne Argumente ist, dass die in der Produktion verwendet wird, und erstellt automatisch eine neue db Kontext basiert auf die App-Konfigurationsdatei Die zweite Option ermöglicht es Ihnen, einen gespiegelten Db-Kontext zu injizieren, wenn Sie Komponententests durchführen.

Am Ende des Tages geht es bei dieser Frage und meiner Antwort nicht wirklich darum, db-Kontexte zu entsorgen - es geht darum, warum die Code-Template-Designer den Ansatz gewählt haben und warum er Ihnen hilft. Sie sollten read more on Einheit testen.

Answer 4

Für einfache Szenarien Aufruf von Dispose (oder Verwendung von) ist überhaupt nicht notwendig "Das Standardverhalten von DbContext ist, dass die zugrunde liegende Verbindung automatisch geöffnet wird, sobald sie benötigt wird und geschlossen, wenn sie nicht mehr benötigt wird." Von hier: http://blog.jongallant.com/2012/10/do-i-have-to-call-dispose-on-dbcontext/