为什么不在视图中使用存储库

Question

我有一个部分视图，循环遍历其模型（事物列表）以显示 thing.Name 和三个整数值（它们是相关实体的计数）。

首先，我尝试放置：（伪剃刀）

foreach(thing in Model){
    @thing.Name : 
        @thing.related.entities.where(condition1).Count()
        @thing.related.entities.where(condition2).Count()
        @thing.related.entities.where(condition3).Count()
}

但是它真的很慢...所以我在 ThingRepository 中创建了一个函数，它可以更快地执行相同的查询，就像这样（伪代码）

function GetCountofRelatedEntities(relatedID,condition){
    return db.entities.where(relatedID==relatedID && condition).count()
}

并且速度更快，所以我想要来调用它。我想我应该从控制器调用它，但是我需要一个 ViewModel 来保留 (thing, int, int, int) 集合作为模型，或者我可以使用 ViewBag 来将结果传递给视图，但是，这是问题： 为什么不简单地从视图中使用存储库？视图中的这段代码有什么问题？ （伪剃刀）

@repo=new ThingRepository()
foreach(thing in Model){
    @thing.Name : 
        @repo.GetCountofRelatedEntities(thing.relatedID,condition1)
        @repo.GetCountofRelatedEntities(thing.relatedID,condition1)
        @repo.GetCountofRelatedEntities(thing.relatedID,condition1)
}

你能告诉我为什么我不应该在视图中实例化存储库吗？或者我可以做吗？

Answer 1

为什么不简单地从视图中使用存储库？

因为你违反了MVC设计模式。视图的职责不是获取数据。它是在视图模型的形式下显示从控制器传递给它的数据。就这么简单。

您可以在视图中调用存储库或任何您喜欢的内容，但不要再用 asp.net-mvc 标记您的问题，因为您不再执行任何 MVC。我的意思是你可以做任何你喜欢的事情 - 如果你愿意的话，你甚至可以在你的视图中编写 SQL 查询。

但 MVC 设计模式的全部要点是将数据检索逻辑与表示逻辑分开。

Answer 2

MVC 模式的一个目的是提供一种适合各种常见编程情况的结构。简单来说：

• 模型：描述应用程序的形状，即特定于您的业务的软件部分。
• 视图：向用户显示数据，并将用户事件传输到服务器。
• 控制器：充当视图和模型之间的中间人。

您提出的建议“有效”，因为它可以在您想要的页面上获取数据。从短期来看，它似乎可以节省您的时间和精力，因为您不必担心控制器、视图袋等。

但是，您正在以一种稍后可能会后悔的方式破坏 MVC 结构。例如，假设几周后您的老板来找您并说：“嘿，您知道您添加的用于显示实体列表的页面吗？我们需要对其进行一些过滤和排序。我昨天需要它。”

现在您面临一个选择：是否将此过滤逻辑添加到我的视图页面并在截止日期前完成，或者我是否花时间将数据访问代码移至控制器并重新设计我的视图，冒着错过视图的风险？截止日期并打破已经有效的方法？

您可能会采取简单的方法并将逻辑添加到视图中，但现在您的手上却出现了越来越混乱的情况。我们一直在使用带有 6,000 行代码隐藏文件的 VB6 和 Webforms 应用程序走这条路。相信我——你不会想去那里的。

另一个问题是 MVC 结构被编程社区很好地理解。如果其他人出现并尝试编写您的代码，您就会通过偏离传统方法来让他们变得更困难。

MVC 结构经过了时间的考验并且是健全的。在您完全理解其目的及其提供的好处之前，请尝试密切关注它。在你牢牢掌握规则之前，打破规则并不是一个好主意。

Answer 3

我的主要反对意见是关注点分离。一旦你开始从你的视图访问数据库，你的“视图”就不再只是一个视图了。将数据访问和视图完全分开确实非常方便。

为什么这种关注点分离很重要？它使得使用由这些干净的分离组成的系统变得更加容易。当您需要调整检索的数据时，您永远不需要对视图大惊小怪。只要视图获得正确的值，它就会正确显示。同样，如果您想更改值的显示方式，您可以修改视图，而不会破坏数据。

Answer 4

问题是您的 View 中不应该有任何逻辑，因为这不是 MVC 方法。

MVC 是关注点分离。

因此，您应该创建包含视图所需的所有数据的 ViewModel。