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

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

在使用 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 提供的各种技术来优化查询性能，以避免出现性能问题和冗余代码。

在MVC EF项目中，使用DbContext是常见的方式，而在查询数据时，使用ToList()方法将查询结果转换为列表，也是一种常见的方式。然而，每次查询都使用ToList()可能会存在一些性能问题，特别是在查询结果较大时。

因此，是否合理取决于具体情况。如果您的查询结果只包含少量记录，则使用ToList()并不会对性能造成很大影响。但是，如果查询结果包含大量记录，则使用ToList()会导致内存占用过高，可能导致系统崩溃或变慢。在这种情况下，您应该考虑使用延迟加载或分页查询等技术来减少内存占用。

此外，建议在实现时使用“仓储模式”，将数据访问逻辑单独封装到仓储类中，从而避免在控制器等业务逻辑中直接使用DbContext。这种做法可以提高代码的可维护性和可扩展性。