ASP.NET MVC中使用通用仓储与工作单元时DbContext已释放错误排查

兄弟，这个问题我之前踩过坑！核心原因是MigrateDatabaseToLatestVersion初始化器在应用启动时会创建一个独立的DbContext实例来执行数据库迁移，如果你的通用仓储/工作单元的DbContext生命周期管理没做好，就会出现上下文被提前释放的冲突。下面给你几个具体的解决思路：

1. 确保DbContext是请求级生命周期 #

ASP.NET MVC中，DbContext最佳实践是每个HTTP请求对应一个实例，请求结束后再释放。如果你的工作单元是手动管理DbContext，或者依赖注入配置不对，很容易和初始化器的实例搞混：

• 如果你用Autofac这类DI容器，把DbContext注册为InstancePerHttpRequest；
• 要是用原生依赖注入（旧版MVC可能需要扩展），也要确保上下文是请求范围内的，避免全局单例或者过早释放。

2. 手动触发迁移，隔离初始化器的DbContext #

不要让初始化器自动执行迁移，而是手动创建临时DbContext来完成初始化，避免和业务代码的上下文冲突。修改Global.asax的Application_Start代码：

protected void Application_Start()
{
    // 注册迁移初始化器
    Database.SetInitializer(new MigrateDatabaseToLatestVersion<YourDbContext, Configuration>());
    
    // 手动用临时上下文执行迁移，执行完自动释放
    using (var tempContext = new YourDbContext())
    {
        // force: false 表示只在需要时执行迁移，不会重复触发
        tempContext.Database.Initialize(force: false);
    }

    // 其他初始化代码（路由、过滤器等）...
}

这样初始化器用的是独立的临时上下文，执行完就释放，不会影响后续业务请求的DbContext。

3. 检查工作单元与仓储的实现 #

确保工作单元持有唯一的DbContext实例，并且在请求结束时才释放：

• 工作单元应该通过构造函数注入DbContext，而不是自己new；
• 工作单元要实现IDisposable，在Dispose方法中正确释放DbContext；
• 控制器通过DI注入工作单元，并且控制器本身如果需要的话，也要实现IDisposable来释放工作单元（如果DI容器没自动处理的话）。

举个工作单元的示例代码：

public class UnitOfWork : IUnitOfWork, IDisposable
{
    private readonly YourDbContext _dbContext;
    private bool _isDisposed = false;

    // 通过构造函数注入DbContext
    public UnitOfWork(YourDbContext dbContext)
    {
        _dbContext = dbContext;
    }

    // 获取通用仓储的方法
    public GenericRepository<TEntity> GetRepository<TEntity>() where TEntity : class
    {
        return new GenericRepository<TEntity>(_dbContext);
    }

    public void SaveChanges()
    {
        _dbContext.SaveChanges();
    }

    protected virtual void Dispose(bool disposing)
    {
        if (!_isDisposed)
        {
            if (disposing)
            {
                // 释放DbContext
                _dbContext.Dispose();
            }
            _isDisposed = true;
        }
    }

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

4. 禁止仓储手动创建DbContext #

通用仓储必须从工作单元获取DbContext实例，绝对不能自己new一个！否则会出现多个上下文实例，其中一个被释放后，关联的实体操作就会抛出“已释放”错误。比如你的GenericRepository应该是这样的：

public class GenericRepository<TEntity> where TEntity : class
{
    protected readonly YourDbContext _context;
    protected readonly DbSet<TEntity> _dbSet;

    // 通过构造函数注入DbContext（由工作单元传入）
    public GenericRepository(YourDbContext context)
    {
        _context = context;
        _dbSet = context.Set<TEntity>();
    }

    // 仓储方法...
}

按照上面的步骤调整后，初始化器和业务代码的DbContext就完全隔离了，不会再出现上下文提前释放的问题。

内容的提问来源于stack exchange，提问作者soheil taghizadeh