IEnumerable vs ToList 直接在视图中迭代

Question

如果我只需要使用该结果一次，即在视图中进行迭代和显示。我一般做第一个，是不是比较明智​​？

IEnumerable<model> result = _someService.GetValues();
return View(result);

或

IList<model> result = _someService.GetValues().ToList();
return View(result);

我签入了分析器，并且使用 EF6 仅针对该数据查询 db 一次。

清除它是一个普通的 SELECT 并且两者都有数据并且没有相邻的列表可以启动延迟加载。

注意：我知道 IEnumerable 有一个 GetEnumerator 方法，它在每次调用它时都会执行，它会执行数据，而 ToList() 只执行一次。

Answer 1

见Programmers.SE: Why should I use List<T> over IEnumerable<T>?:

有时对 linq 查询执行 ToList() 对确保查询按您期望的时间和顺序执行很重要。

这里正是如此。

如果_someService.GetValues() 返回一个IEnumerable<model>，这是一个通过延迟执行的实体框架查询，则在您的控制器中调用ToList() 将在那里执行查询：在您的控制器中，您可以在其中处理可能发生的异常。例如查询超时。

如果您不调用.ToList()，则会在您的视图中在@foreach (var item in Model) 处引发相同的异常。你不想这样。

另外，如果您使用延迟加载，上下文将在查询结果最终出现在您的视图中并再次引发异常时被释放，因此在这种情况下您将必须具体化通过调用.ToList()，查询结果在您的控制器中。

---

当谈到IEnumerable<T> 与IList<T> 时，人们会一头雾水。他们通常互相模仿一些一般性建议（找到here, amongst others），但他们不明白该建议是针对API 设计。您不是在设计 API，而是在实现具有非常特定要求的应用程序：视图可能永远不会抛出异常。为了满足这个要求，你必须在你的控制器中执行你的查询，因此在那里调用.ToList()。

提出上述问题的用户就在the comment they posted：

应执行所有查询，以便模型完成并为视图加载做好准备。 IE。视图应该接收所有内容，而不是查询数据库。

Razor 将您的模型渲染到您的视图可能没有任何副作用。执行查询是一种副作用，因此在渲染视图时可能不会发生这种情况。

即使在您的控制器中调用.ToList()，您在视图中的@model 指令仍然可以而且应该仍然是IEnumerable<T>！