存储库模式：好还是坏？

Question

在阅读ASP.NET MVC 3.0项目中的Repository Design模式后，我遇到了一个让我困惑的问题，并开始质疑模式的好处。也许有人可以帮我澄清这一点。

我有以下3个表：

• - 用户（parent）
• - UserLead（child of User）
• - UserLeadNotes（child of UserLead）

粗略地说，表和它们之间的关系是这样设计的：

• 用户[ID]
• UserLead[ID，用户ID（外键）
• UserLeadNotes[Id，UserLeadId（外键）]

我有一个BaseRepository，其中包含基本的CRUD方法。 (IEnumerable<TEntity> Get(), TEntity GetByID(), Insert(), Delete(), etc…)

随着到位，我也有一个Service Layer如果在需要的时候并调用适当的存储库。

简而言之，应用程序显示一个UserLead并允许用户添加/删除该UserLead的注释（在UserLeadNotes表中存储/删除）。添加/删除笔记通过Ajax调用完成。注释的删除沿着我希望删除的注释的“id”传递。

现在...之前，我确实删除的笔记，我需要确保的音符真正属于自己的登录用户当前（User.Identity.Name aka UserId）

考虑我的方法deleteUserNote(int noteId)只接收一个“noteId”参数和考虑我UserLeadNotes表不对UserId外键，我需要做的，就是增加一个

。包括（“UserLead'）

我_userLeadNoteRepository里面有类似：

UserLeadNote userLeadNote = _userLeadNoteRepository 
       .AllIncluding(p => p.UserLead) 
       .FirstOrDefault(p => p.UserLeadNoteID.Equals(noteId) 
        && p.UserLead.UserID.Equals(User.Identity.Name)); 

那我就删，我只是找到了UserLeadNote对象：

_userLeadNoteRepository.Delete(userLeadNote); 

---

问题：

按照上面的方法，这将迫使我为我的UserLeadNotes创建一个存储库，b)由于.Include（）找到的userLeadNote对象将具有一个名为UserLead的属性，其中包含UserLead表中定义的所有字段。

如果我的UserLead表碰巧有14个字段，其中的10个字段是ntext，这意味着我将会加载/保存太多的信息！

如果我是使用，而不是Lambda表达式的，一个LINQ声明绕过库并直接使用我的上下文：

using(MyContext db = new MyContext()) 
      { 
       var query = 
        from uln in db.UserLeadNote 
        join ul in db.UserLead on uln.UserLeadID equals ul.UserLeadID 
        where uln.UserLeadNoteID == userLeadNoteID 
         && ul.UserID == User.Identity.Name 
        select uln; 

       var userLeadNote = query.FirstOrDefault(); 

       db.UserLeadNote.Remove(userLeadNote); 

       db.SaveChanges(); 
      } 

难道这种做法是因为它会没有更有效从UserLead表中带回这些不必要的字段？ （例如那10个字段）。

除非我没有正确使用存储库模式，或者这可能是一个实体框架问题，否则我想知道任何人的想法！

请记住，我知道有一个存储库模式的好处，但好奇，如果这是人们有或没有的某种问题。如果是这样，他们是否决定放弃模式，并使用Linq语句直接使用它们的上下文。

两个邪恶中最小的是什么？

Answer 1

总之，

阅读瑞安的后鲁昂的帖子，似乎创建一个每个实体的存储库可能不是合适的方法。如果我有一个孩子（或孩子的孩子）实体，我应该创建一个单独的父库，并通过孩子（或孩子的孩子）导航我的方式。

我遇到的问题是，当试图删除一个子实体（没有父母的外键）的孩子时，我不得不.Include(“”)其他“上”表。 （在我的情况中，只有一张桌子，父母，但可以很容易地删除一个孩子的孩子的孩子，并将不得不Include()许多“父母/上”表）。

最后，包括Parent表创建了一个属性，该属性包含Parent的所有字段，在我的情况下，这被认为是“获取”了我手头简单任务的太多信息。

这个所谓的问题，可以避免，如果我要放弃使用仓库模式，而是直接使用上下文。

对于某些人来说，这可能不是问题（或者他们根本不知道发生了什么），对于其他人来说，使用Pattern更重要。

我还没有决定完全删除存储库模式，因为我喜欢它带来的表。

感谢所有。

Answer 2

尝试看看这里：

asp.net mvc page is not showing properties from associated objects

有我的回答有关于这种通用仓库模式被滥用的解释/意见