您什么时候决定应该创建一个新的存储库？

Question

假设你有这两门课。

public class Author
{
public int ID {get; set;}
public string Name{get; set;}
public virtual ICollection<Book> Books { get; set;}
}

public class Book
{
public int ID {get; set;}
public string Name{get; set;}
public int AuthorId {get; set;}
public virtual Author Author{ get; set;}
}

如果我有一个 BooksController，其所有操作都会收到一个 AuthorId，

我会转到以下地址：

/作者/1/书籍

仅使用当前的 AuthorRepository 有意义吗？

public ActionResult Index(int AuthorId)
    {
    return View(_AuthorRepository.GetById(AuthorId).Books)
    }

或者为了保持存储库内的数据访问，我应该创建一个 BookRepository？

public ActionResult Index(int AuthorId)
    {
    return View(_BookRepository.GetByAuthorId(AuthorId))
    }

创建操作也会发生同样的情况。我很难决定哪一个更有意义。

[HttpPost]
public ActionResult Create(int AuthorId, BookViewModel book)
{
if(ModelState.IsValid)
{
Book b= new Book();
b= AutomapperMagicMethodThatGivesMeABookFromA(book); //
_AuthorRepository.FindById(AuthorId).Books.Add(b);
_AuthorRepository.SaveChanges();
return RedirectToAction("Index");
}
return View(book)
}

或者这种方法。

[HttpPost]
public ActionResult Create(int AuthorId, BookViewModel book)
{
if(ModelState.IsValid)
{
Book b= new Book();
b= AutomapperMagicMethodThatGivesMeABookFromA(book); //
b.AuthorId = AuthorId;
_BookRepository.Add(b);
_BookRepository.SaveChanges();
return RedirectToAction("Index");
}
return View(book)
}

我希望得到一些有关处理此类情况的建议。 请随意批评我的代码。

请帮忙提前致谢。

附：我正在使用 EF，如果它有什么区别的话。

Answer 1

我会在单元测试方面考虑它。哪种方法提供了更简单的方法来测试控制器和验证代码中的测试结果？

Answer 2

一般经验法则是每个聚合根创建一个存储库。

其中聚合根可以被认为是“父级”，并且“子级”不能没有父级而存在。

在您的示例中，作者是聚合根，因为没有作者就不可能存在书籍。 （FK证明了这一点）。

因此，您应该只有一个 LibraryRepository，它能够使用单个实体框架对象集与作者和书籍一起工作。

实际上，我们通过将 ObjectSet 输入到存储库上的通用类型来在存储库中强制执行这些聚合边界。

因此您可能会这样：

public class LibraryRepository : IRepository<Author>, GenericRepository<Author>
{
   public Author FindById(int id)
   {
      return _context.SingleOrDefault(x => x.AuthorId == id);
   }

   public Book FindById(int bookId)
   {
      return _context
         .Where(x => x.Books.Any(y => y.BookId == bookId))
         .Select(x => x.Books.SingleOrDefault(x => x.BookId == bookId))
         .SingleOrDefault();
   }
}

在上面的示例中，_context 是通用存储库中的一个受保护 属性，其类型为 ObjectSet (这是通用存储库中的 T）。

然而，看看找书是多么的混乱。如果您发现自己需要自己找到一个“子项”（例如，不首先检索父项），那么您也应该考虑创建一个图书存储库。

因此，它归结为两件事：

1. 实施聚合边界
2. 创建为您的应用程序服务的存储库

考虑用户界面，每个屏幕需要什么信息，它手头有什么信息来识别该信息（例如 URL）。