Apakah OK untuk membuat objek Entity Framework DataContext dan membuangnya di blok menggunakan di setiap metode CRUD saya?

Saya sedang membangun aplikasi WPF yang mengimplementasikan fitur-fitur berikut:

1. Ambil input pengguna dan baca data dari basis data
2. melakukan beberapa perhitungan di atasnya
3. Perlihatkan kepada pengguna dalam berbagai jenis tampilan dan tulis perubahan kembali ke db

Arsitektur yang diajukan: Database -> Kerangka Entitas -> Repositori -> Logika Bisnis -> Layanan Data -> ViewModel

Alasan untuk menggunakan arsitektur ini: Beberapa skenario hadir dalam aplikasi (Multiple view) dan beberapa database. Karenanya, saya bersedia menggunakan repositori di tengah untuk abstraksi.

Satu peringatan adalah bahwa konteksnya akan berumur panjang jika repositori diimplementasikan. Untuk mengatasinya, apakah boleh membuat konteks dan membuangnya dalam blok using () di masing-masing metode crud.?

jangan ragu untuk menyarankan pendekatan alternatif.

Gunakan satu objek DbContext per akses data atau transaksi.

DbContextadalah benda yang ringan; itu dirancang untuk digunakan sekali per transaksi bisnis. Membuat DbContextSingleton Anda dan menggunakannya kembali di seluruh aplikasi dapat menyebabkan masalah lain, seperti masalah konkurensi dan kebocoran memori.

DbContextpada dasarnya mengimplementasikan Satuan Kerja. Perlakukan sesuai.

Jangan buang objek DbContext.

Meskipun DbContextimplementasinya IDisposable, Anda tidak harus membuangnya secara manual, atau membungkusnya dalam usingpernyataan. DbContextmengelola hidupnya sendiri; ketika permintaan akses data Anda selesai, DbContextsecara otomatis akan menutup koneksi database untuk Anda.

Untuk memahami mengapa hal ini terjadi, pertimbangkan apa yang terjadi ketika Anda menjalankan pernyataan Linq pada koleksi entitas dari a DbContext. Jika Anda mengembalikan pemuatan malas IQueryabledari metode akses data Anda, Anda berdiri pipa yang sebenarnya tidak dieksekusi sampai klien memaksa beberapa data dari itu (dengan menelepon FirstOrDefault(), ToList()atau beralih di atasnya).

Bacaan Lebih Lanjut
Apakah saya harus selalu memanggil Buang () pada objek DbContext saya?
Mengapa Anda tidak harus menggunakan Singleton DataContexts di Entity Framework
Returning IEnumerable<T>vs. IQueryable<T>
Haruskah Repositori kembali IQueryable?

Idealnya konteks harus diinisialisasi dan diakhiri untuk satu transaksi. Dalam kasus Anda konteksnya harus dipakai di Business Logic dan diteruskan ke Repositori untuk membaca / menulis data.

Jika Anda memanggil DbContext pada setiap metode dalam aplikasi Anda, Anda akan mengalami kebocoran memori. Gunakan satu instance dari DbContext. Lihat komentar pada contoh di bawah ini:

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;
}