การแยกการเข้าถึงข้อมูลใน ASP.NET MVC

ฉันต้องการตรวจสอบให้แน่ใจว่าฉันปฏิบัติตามมาตรฐานอุตสาหกรรมและแนวปฏิบัติที่ดีที่สุดกับการแตกที่แท้จริงครั้งแรกของฉันที่ MVC ในกรณีนี้คือ ASP.NET MVC โดยใช้ C #

ฉันจะใช้ Entity Framework 4.1 สำหรับโมเดลของฉันโดยมีวัตถุเป็นรหัสแรก (ฐานข้อมูลมีอยู่แล้ว) ดังนั้นจะมีวัตถุ DBContext สำหรับดึงข้อมูลจากฐานข้อมูล

ในการสาธิตที่ฉันได้อ่านผ่านเว็บไซต์ asp.net ผู้ควบคุมจะมีรหัสการเข้าถึงข้อมูลอยู่ สิ่งนี้ดูไม่ถูกต้องสำหรับฉันโดยเฉพาะอย่างยิ่งเมื่อทำตามแนวทางปฏิบัติของ DRY (อย่าทำซ้ำตัวเอง)

ตัวอย่างเช่นสมมติว่าฉันกำลังเขียนแอปพลิเคชันเว็บที่จะใช้ในห้องสมุดสาธารณะและฉันมีตัวควบคุมสำหรับการสร้างอัปเดตและลบหนังสือในแคตตาล็อก

การกระทำหลายอย่างอาจใช้ ISBN และต้องการส่งคืนวัตถุ "หนังสือ" (โปรดทราบว่านี่อาจไม่ใช่รหัสที่ถูกต้อง 100%):

public class BookController : Controller
{
    LibraryDBContext _db = new LibraryDBContext();

    public ActionResult Details(String ISBNtoGet)
    {
        Book currentBook = _db.Books.Single(b => b.ISBN == ISBNtoGet);
        return View(currentBook);
    }

    public ActionResult Edit(String ISBNtoGet)
    {
        Book currentBook = _db.Books.Single(b => b.ISBN == ISBNtoGet);
        return View(currentBook);
    }
}

แต่ควรที่จริงผมมีวิธีการในวัตถุบริบทฐานข้อมูลของฉันจะกลับหนังสือเล่มหนึ่งหรือไม่? ดูเหมือนจะเป็นการแยกทางที่ดีกว่าสำหรับฉันและช่วยส่งเสริม DRY เพราะฉันอาจต้องรับหนังสือวัตถุโดย ISBN จากที่อื่นในเว็บแอปพลิเคชันของฉัน

public partial class LibraryDBContext: DBContext
{
    public Book GetBookByISBN(String ISBNtoGet)
    {
        return Books.Single(b => b.ISBN == ISBNtoGet);
    }
}

public class BookController : Controller
{
    LibraryDBContext _db = new LibraryDBContext();

    public ActionResult Details(String ISBNtoGet)
    {
        return View(_db.GetBookByISBN(ISBNtoGet));
    }

    public ActionResult Edit(ByVal ISBNtoGet as String)
    {
        return View(_db.GetBookByISBN(ISBNtoGet));
    }
}

นี่เป็นชุดของกฎที่ถูกต้องที่จะปฏิบัติตามในการเข้ารหัสแอปพลิเคชันของฉันหรือไม่

หรือฉันคิดว่าคำถามเชิงอัตวิสัยจะเป็นดังนี้: "นี่เป็นวิธีที่ถูกต้องหรือไม่"

โดยทั่วไปคุณต้องการให้ผู้ควบคุมของคุณทำบางสิ่ง:

1. จัดการคำขอที่เข้ามา
2. มอบหมายการประมวลผลไปยังวัตถุทางธุรกิจบางอย่าง
3. ส่งผลลัพธ์ของการประมวลผลทางธุรกิจไปยังมุมมองที่เหมาะสมสำหรับการเรนเดอร์

ไม่ควรมีใด ๆการเข้าถึงข้อมูลหรือเหตุผลทางธุรกิจที่ซับซ้อนในการควบคุม

[ในแอพที่ง่ายที่สุดคุณอาจหนีไปกับการกระทำ CRUD ข้อมูลพื้นฐานในคอนโทรลเลอร์ของคุณ แต่เมื่อคุณเริ่มเพิ่มการโทรแบบรับสายและรับสายแบบธรรมดาคุณจะต้องแยกการประมวลผลของคุณออกเป็นคลาสแยกต่างหาก ]

ตัวควบคุมของคุณจะขึ้นอยู่กับ 'บริการ' เพื่อทำงานการประมวลผลจริง ในคลาสบริการของคุณคุณอาจทำงานโดยตรงกับแหล่งข้อมูลของคุณ (ในกรณีของคุณ DbContext) แต่อีกครั้งหากคุณพบว่าตัวเองเขียนกฎธุรกิจจำนวนมากนอกเหนือจากการเข้าถึงข้อมูลคุณอาจต้องการแยกธุรกิจของคุณออก ตรรกะจากการเข้าถึงข้อมูลของคุณ

ณ จุดนี้คุณอาจจะมีคลาสที่ไม่ทำอะไรเลยนอกจากการเข้าถึงข้อมูล บางครั้งสิ่งนี้เรียกว่า Repository แต่ไม่สำคัญว่าชื่อคืออะไร ประเด็นก็คือรหัสทั้งหมดสำหรับการรับข้อมูลเข้าและออกจากฐานข้อมูลนั้นอยู่ในที่เดียว

สำหรับโครงการ MVC ทุกครั้งที่ฉันทำงานฉันมักจะลงเอยด้วยโครงสร้างที่ชอบ:

ตัวควบคุม

public class BookController : Controller
{
    ILibraryService _libraryService;

    public BookController(ILibraryService libraryService)
    {
        _libraryService = libraryService;
    }

    public ActionResult Details(String isbn)
    {
        Book currentBook = _libraryService.RetrieveBookByISBN(isbn);
        return View(ConvertToBookViewModel(currentBook));
    }

    public ActionResult DoSomethingComplexWithBook(ComplexBookActionRequest request)
    {
        var responseViewModel = _libraryService.ProcessTheComplexStuff(request);
        return View(responseViewModel);
    }
}

ธุรกิจบริการ

public class LibraryService : ILibraryService
{
     IBookRepository _bookRepository;
     ICustomerRepository _customerRepository;

     public LibraryService(IBookRepository bookRepository, 
                           ICustomerRepository _customerRepository )
     {
          _bookRepository = bookRepository;
          _customerRepository = customerRepository;
     }

     public Book RetrieveBookByISBN(string isbn)
     {
          return _bookRepository.GetBookByISBN(isbn);
     }

     public ComplexBookActionResult ProcessTheComplexStuff(ComplexBookActionRequest request)
     {
          // Possibly some business logic here

          Book book = _bookRepository.GetBookByISBN(request.Isbn);
          Customer customer = _customerRepository.GetCustomerById(request.CustomerId);

          // Probably more business logic here

          _libraryRepository.Save(book);

          return complexBusinessActionResult;

     } 
}

กรุ

public class BookRepository : IBookRepository
{
     LibraryDBContext _db = new LibraryDBContext();

     public Book GetBookByIsbn(string isbn)
     {
         return _db.Books.Single(b => b.ISBN == isbn);
     }

     // And the rest of the data access
}

นี่คือวิธีที่ฉันทำแม้ว่าฉันจะฉีดผู้ให้บริการข้อมูลเป็นอินเทอร์เฟซบริการข้อมูลทั่วไปเพื่อให้ฉันสามารถสลับการใช้งานได้

เท่าที่ฉันรู้คอนโทรลเลอร์มีจุดประสงค์เพื่อให้คุณรับข้อมูลดำเนินการใด ๆ และส่งผ่านข้อมูลไปยังมุมมอง