EF "There is already an open DataReader associated with this Command which must be closed first."

Question

现在公司用的是mvc EF,数据库查询都是通过DBContext，每次都要 this._ArtWorkDemandRepository.Entities.tolist(),才能取查询别的，这样合理吗？感觉好麻烦，第一次使用MVC EF ,不是特别懂，

Answer 1

在使用 MVC EF 架构的应用程序中，通常需要通过 DbContext 来执行数据库查询和持久化操作。如果你的代码中的每次查询都需要显式地调用 .ToList() 方法，那么可能会导致性能问题和代码冗余。

这是因为 .ToList() 方法将会立即从数据库中加载所有匹配查询条件的记录，转换为 List 集合返回给调用方。如果数据量较大，这将会导致性能瓶颈，降低应用程序的响应速度。此外，如果你只需要访问查询结果的一小部分，那么将所有记录加载到内存中是不必要的，会浪费系统资源。

为了避免这个问题，你可以使用延迟加载 (Lazy Loading) 或显示加载 (Eager Loading) 等技术来优化查询性能。延迟加载可以让你在需要访问特定属性或导航属性时才加载相关的数据，而显示加载则允许你在需要时一次性加载所有相关数据。

下面是一个使用显示加载的示例代码：

var artworkDemand = this._ArtWorkDemandRepository
                        .Include(a => a.Artwork)
                        .Include(a => a.Artist)
                        .SingleOrDefault(a => a.Id == id);

在这个示例中，我们使用了 .Include() 方法来显式地加载 Artwork 和 Artist 导航属性的相关数据。在调用 SingleOrDefault() 方法时，EF 将会一次性加载所有相关数据，并返回查询结果的单个实体。

总之，使用 MVC EF 架构进行数据库查询是非常常见和合理的做法。但是，你需要了解如何使用 EF 提供的各种技术来优化查询性能，以避免出现性能问题和冗余代码。