Question

在我们的Web应用程序中，需要从数据库中的各种表中获取数据。今天，您可能会发现5个或6个数据库查询是针对单个请求串行执行的。这些查询都不依赖于来自另一个的数据，因此它们是并行执行的完美候选者。问题是众所周知的DbConcurrencyException，当针对相同的上下文执行多个查询时会抛出该问题。

我们通常每个请求使用一个上下文，然后有一个存储库类，以便我们可以在各个项目中重用查询。然后，当处理控制器时，我们会在请求结束时处理上下文。

以下是使用并行性的示例，但仍然存在问题！

var fileTask = new Repository().GetFile(id);
var filesTask = new Repository().GetAllFiles();
var productsTask = AllProducts();
var versionsTask = new Repository().GetVersions();
var termsTask = new Repository().GetTerms();

await Task.WhenAll(fileTask, filesTask, productsTask, versionsTask, termsTask);

每个存储库都在内部创建自己的上下文，但就像现在一样，它们并没有被处理掉。这是一个问题。我知道我可以在我创建的每个存储库上调用Dispose，但这会使代码快速混乱。我可以为每个使用自己的上下文的查询创建一个包装器函数，但这感觉很麻烦，并且不是解决问题的长期解决方案。

解决此问题的最佳方法是什么？我希望客户端/消费者不必担心在并行执行多个查询的情况下处理每个存储库/上下文。

我现在唯一的想法是遵循类似于工厂模式的方法，除了我的工厂将跟踪它创建的所有对象。一旦我知道我的查询完成并且工厂可以在内部处理每个存储库/上下文，我就可以处理工厂。

我很惊讶地看到关于并行性和实体框架的这种小讨论，所以希望来自社区的更多想法会出现。

修改

以下是我们的存储库的简单示例：

public class Repository : IDisposable {
    public Repository() {
        this.context = new Context();
        this.context.Configuration.LazyLoadingEnabled = false;
    }

    public async Task<File> GetFile(int id) {
        return await this.context.Files.FirstOrDefaultAsync(f => f.Id == id);
    }

    private bool disposed = false;

    protected virtual void Dispose(bool disposing) {
        if (!this.disposed) {
            if (disposing) {
                context.Dispose();
            }
        }
        this.disposed = true;
    }

    public void Dispose() {
        Dispose(true);
        GC.SuppressFinalize(this);
    }
}

如您所见，每个存储库都有自己的上下文。这意味着需要处理每个存储库。在我上面给出的示例中，这意味着我需要4次调用Dispose()。

我对工厂解决问题的想法如下：

public class RepositoryFactory : IDisposable {
    private List<IRepository> repositories;

    public RepositoryFactory() {
        this.repositories = new List<IRepository>();
    }

    public IRepository CreateRepository() {
        var repo = new Repository();
        this.repositories.Add(repo);
        return repo;            
    }

    #region Dispose
    private bool disposed = false;

    protected virtual void Dispose(bool disposing) {
        if (!this.disposed) {
            if (disposing) {
                foreach (var repo in repositories) {
                    repo.Dispose();
                }
            }
        }
        this.disposed = true;
    }

    public void Dispose() {
        Dispose(true);
        GC.SuppressFinalize(this);
    }
    #endregion
}

这个工厂将负责创建我的存储库的实例，但它也会跟踪它创建的所有实例。一旦处理完这个单一的工厂类，它就会负责处理它创建的每个存储库。

Answer 1

您可以允许客户端通过将某种可选（默认为false）Repository位传递给构造函数来配置autodispose的处置行为。实现看起来像这样：

public class Repository : IDisposable
{
    private readonly bool _autodispose = false;
    private readonly Lazy<Context> _context = new Lazy<Context>(CreateContext);

    public Repository(bool autodispose = false) {
        _autodispose = autodispose;
    }

    public Task<File> GetFile(int id) {
        // public query methods are still one-liners
        return WithContext(c => c.Files.FirstOrDefaultAsync(f => f.Id == id));
    }

    private async Task<T> WithContext<T>(Func<Context, Task<T>> func) {
        if (_autodispose) {
            using (var c = CreateContext()) {
                return await func(c);
            }
        }
        else {
            return await func(_context.Value);
        }
    }

    private static Context CreateContext() {
        var c = new Context();
        c.Configuration.LazyLoadingEnabled = false;
        return c;
    }

    public void Dispose() {
        if (_context.IsValueCreated)
            _context.Value.Dispose();
    }
}

注意：为了说明，我保持处理逻辑简单;您可能需要重新使用disposed位。

您的查询方法仍然是简单的单行，客户端可以根据需要非常轻松地配置处理行为，甚至可以在自动处理情况下重用Repository实例：

var repo = new Repository(autodispose: true);
var fileTask = repo.GetFile(id);
var filesTask = repo.GetAllFiles();
var productsTask = AllProducts();
var versionsTask = repo.GetVersions();
var termsTask = repo.GetTerms();

await Task.WhenAll(fileTask, filesTask, productsTask, versionsTask, termsTask);

并行性和实体框架

1 个答案: