Ist es in Ordnung, ein Entity Framework DataContext-Objekt zu erstellen und es in jeder meiner CRUD-Methoden in einem using-Block zu entsorgen?

Ich erstelle eine wpf-Anwendung, die die folgenden Funktionen implementiert:

1. Nehmen Sie Benutzereingaben und lesen Sie Daten aus Datenbanken
2. Führen Sie einige Berechnungen durch
3. Präsentieren Sie es dem Benutzer in mehreren Arten von Ansichten und schreiben Sie Änderungen zurück in db

Vorgeschlagene Architektur: Datenbank -> Entity Framework -> Repository -> Geschäftslogik -> Datendienst -> ViewModel

Gründe für die Verwendung dieser Architektur: Mehrere in der Anwendung vorhandene Szenarien (mehrere Ansichten) und mehrere Datenbanken. Daher bin ich bereit, das Repository in der Mitte für die Abstraktion zu verwenden.

Eine Einschränkung ist, dass der Kontext langlebig ist, wenn das Repository implementiert wird. Um dies zu überwinden, ist es in Ordnung, in jeder der Rohmethoden einen Kontext zu erstellen und diese in einem using () -Block zu entsorgen.

Sie können gerne alternative Ansätze vorschlagen.

Verwenden Sie ein DbContext-Objekt pro Datenzugriff oder Transaktion.

DbContextist ein leichtes Objekt; Es kann einmal pro Geschäftsvorfall verwendet werden. Wenn Sie DbContexteinen Singleton erstellen und in der gesamten Anwendung wiederverwenden, kann dies zu anderen Problemen führen, z. B. zu Problemen mit Parallelität und Speicherverlust.

DbContextimplementiert im Wesentlichen eine Arbeitseinheit. Behandle es entsprechend.

Entsorgen Sie keine DbContext-Objekte.

Obwohl die DbContextGeräte IDisposable, sollten Sie sie nicht manuell entsorgen oder in eine usingAnweisung einschließen. DbContextverwaltet sein eigenes Leben; Wenn Ihre Datenzugriffsanforderung abgeschlossen ist, DbContextwird die Datenbankverbindung automatisch für Sie geschlossen.

Um zu verstehen, warum dies der Fall ist, überlegen Sie, was passiert, wenn Sie eine Linq-Anweisung für eine Entitätssammlung von a ausführen DbContext. Wenn Sie einen Lazy-Laden Rückkehr IQueryableaus der Datenzugriffsmethode, stehen Sie eine Pipeline bis der , bis der Client nicht tatsächlich ausgeführt wird , einige Daten von ihm zwingt (durch den Aufruf FirstOrDefault(), ToList()oder über sie iterieren).

Weiterführende
Literatur Muss ich Dispose () für meine DbContext-Objekte immer aufrufen?
Warum sollten Sie nicht Singleton Datacontexts in Entity Framework verwenden ,
Rückkehr IEnumerable<T>gegen IQueryable<T>
Rückkehr sollte Repositorys IQueryable?

Idealerweise sollte der Kontext für eine einzelne Transaktion initialisiert und beendet werden. In Ihrem Fall sollte der Kontext in Business Logic instanziiert und zum Lesen / Schreiben von Daten an das Repository weitergeleitet werden.

Wenn Sie DbContext für jede Methode in Ihrer Anwendung aufrufen, tritt ein Speicherverlust auf. Verwenden Sie eine einzelne Instanz des DbContext. Siehe den Kommentar im folgenden Beispiel:

public bool IsInStock(int _ProductId)
{
  var result = false;

  try
  {
    using (var dataService = new StoreDbDataService()) // NB: This line on each method will eventually cause memory leak.
    {
      result = dataService.IsInStock(_ProductId);
    }
  }
  catch (Exception ex)
  {
    Log.LogException(ex);
  }

  return result;
}