Apakah itu praktik buruk yang repositori panggilan pengontrol alih-alih layanan?

Apakah itu praktik buruk yang repositori panggilan pengontrol alih-alih layanan?

untuk menjelaskan lebih lanjut:

Saya mengetahui bahwa dalam desain yang baik pengendali memanggil layanan dan layanan menggunakan repositori.

tapi kadang-kadang di controller saya tidak punya / butuh logika dan hanya perlu mengambil dari db dan meneruskannya untuk melihatnya.

dan saya bisa melakukannya hanya dengan menelepon repositori - tidak perlu memanggil layanan - apakah itu praktik yang buruk?

Tidak, pikirkan seperti ini: repositori adalah layanan (juga).

Jika entitas yang Anda ambil melalui repositori menangani sebagian besar logika bisnis, maka tidak perlu untuk layanan lain. Hanya memiliki repositori sudah cukup.

Bahkan jika Anda memiliki beberapa layanan yang harus Anda lewati untuk memanipulasi entitas Anda. Ambil entitas dari repositori terlebih dahulu dan kemudian berikan ke layanan tersebut. Mampu membuang HTTP 404 bahkan sebelum mencoba sangat mudah.

Juga untuk skenario baca, Anda hanya perlu entitas untuk memproyeksikannya ke DTO / ViewModel. Memiliki lapisan layanan di antaranya kemudian sering menghasilkan banyak metode yang agak jelek.

Ini bukan praktik yang buruk bagi pengontrol untuk memanggil repositori secara langsung. "Layanan" hanyalah alat lain, jadi gunakan di tempat yang masuk akal.

NikolaiDante berkomentar:

... Pilih pola yang tepat untuk aplikasi yang tepat. Apa yang akan saya katakan adalah Anda harus membuat aplikasi Anda konsisten.

Saya tidak berpikir konsistensi adalah aspek yang paling penting. Kelas "service" dimaksudkan untuk merangkum beberapa logika level yang lebih tinggi sehingga controller tidak perlu mengimplementasikannya. Jika tidak ada "logika tingkat tinggi" yang diperlukan untuk operasi yang diberikan, langsung saja ke repositori.

Untuk mempromosikan Kepedulian Terpisah yang baik dan kemampuan testabilitas, repositori haruslah merupakan ketergantungan yang Anda berikan ke layanan melalui konstruktor:

IFooRepository repository = new FooRepository();
FooService service = new FooService(repository);

service.DoSomething(...);

Jika mencari catatan dalam database membutuhkan beberapa jenis permintaan parameter, sebuah kelas layanan mungkin merupakan tempat yang baik untuk mengambil dalam model tampilan Anda dan membangun kueri yang kemudian dijalankan oleh repositori.

Demikian pula, jika Anda memiliki model tampilan kompleks untuk formulir, kelas layanan dapat merangkum logika membuat, memperbarui, dan menghapus catatan dengan memanggil metode pada Model Domain / Entitas Anda, kemudian bertahan dengan menggunakan repositori.

Pergi ke arah yang berlawanan, jika controller Anda perlu mendapatkan catatan dengan ID-nya, kemudian mendelegasikan ke objek layanan untuk ini seperti memukul paku payung dengan palu godam - itu cara yang lebih dari yang Anda butuhkan.

Saya telah menemukan bahwa pengontrol berada pada posisi terbaik untuk menangani transaksi, atau objek Unit Kerja . Pengontrol atau objek Unit Kerja kemudian akan mendelegasikan ke objek layanan untuk operasi yang kompleks, atau langsung ke repositori untuk operasi sederhana (seperti menemukan catatan dengan Id).

public class ShoppingCartsController : Controller
{
    [HttpPost]
    public ActionResult Edit(int id, ShoppingCartForm model)
    {
        // Controller initiates a database session and transaction
        using (IStoreContext store = new StoreContext())
        {
            // Controller goes directly to a repository to find a record by Id
            ShoppingCart cart = store.ShoppingCarts.Find(id);

            // Controller creates the service, and passes the repository and/or
            // the current transaction
            ShoppingCartService service = new ShoppingCartService(store.ShoppingCarts);

            if (cart == null)
                return HttpNotFound();

            if (ModelState.IsValid)
            {
                // Controller delegates to a service object to manipulate the
                // Domain Model (ShoppingCart)
                service.UpdateShoppingCart(model, cart);

                // Controller decides to commit changes
                store.SaveChanges();

                return RedirectToAction("Index", "Home");
            }
            else
            {
                return View(model);
            }
        }
    }
}

Saya pikir campuran layanan dan bekerja dengan repositori secara langsung dapat diterima. Anda dapat lebih lanjut merangkum transaksi dalam objek Unit Kerja jika Anda merasa perlu.

Rincian tanggung jawab seperti ini:

• Pengontrol mengontrol aliran aplikasi
  • Mengembalikan "404 Tidak Ditemukan" jika keranjang belanja tidak ada dalam database
  • Merender ulang formulir dengan pesan validasi jika validasi gagal
  • Menghemat keranjang belanja jika semuanya memeriksa
• Pengontrol mendelegasikan ke kelas layanan untuk mengeksekusi logika bisnis pada Model Domain Anda (atau Entitas). Objek layanan tidak boleh menerapkan logika bisnis! Mereka menjalankan logika bisnis.
• Pengendali dapat mendelegasikan langsung ke repositori untuk operasi sederhana
• Objek layanan mengambil data dalam model tampilan, dan mendelegasikan ke Model Domain untuk menjalankan logika bisnis (mis. Objek layanan memanggil metode pada Model Domain sebelum memanggil metode pada repositori)
• Objek layanan mendelegasikan ke repositori untuk kegigihan data
• Pengendali harus:
  1. Kelola masa pakai suatu transaksi, atau
  2. Buat objek Unit Kerja untuk mengelola masa transaksi

Tergantung pada arsitektur Anda. Saya menggunakan Spring, dan transaksionalitas selalu dikelola oleh layanan.

Jika Anda memanggil repositori langsung untuk operasi tulis (atau layanan sederhana tanpa logika yang hanya mendelegasikan ke repositori), mungkin Anda menggunakan beberapa transaksi basis data untuk operasi yang harus dilakukan dalam satu. Itu akan menyebabkan data tidak koheren dalam database Anda. Sebagai aturan umum, operasi basis data harus berhasil, atau gagal, tetapi operasi setengah kerja adalah penyebab sakit kepala.

Untuk alasan itu saya berpikir bahwa memanggil repositori langsung dari pengontrol, atau menggunakan layanan pendelegasian sederhana, adalah praktik yang buruk. Anda mulai melakukannya hanya untuk membaca, dan segera Anda, atau satu atau teman Anda akan mulai melakukannya untuk operasi menulis.