MVC ActionFilterAttribute DbContext Disposed 解决方案

Question

我目前在我的 MVC 项目中有以下 ActionFilterAttribute。它适用于第一个请求，但后续请求会返回 DbContext 已释放的消息。

public class PermissionFilter : ActionFilterAttribute
{
    private readonly ApplicationGroupManager _groupManager = new ApplicationGroupManager();
    private readonly ActionPermissionManager _permissionManager = new ActionPermissionManager();

    public override void OnActionExecuting(ActionExecutingContext filterContext)
    {
        var request = filterContext.HttpContext.Request;
        var response = filterContext.HttpContext.Response;
        if (request.IsAjaxRequest())
        {
            #region Preventing caching of ajax request in IE browser

            response.Cache.SetExpires(DateTime.UtcNow.AddDays(-1));
            response.Cache.SetValidUntilExpires(false);
            response.Cache.SetCacheability(HttpCacheability.NoCache);
            response.Cache.SetNoStore();

            #endregion Preventing caching of ajax request in IE browser
        }
        var currentAreaName = filterContext.RequestContext.RouteData.DataTokens["area"];
        var currentActionName = filterContext.ActionDescriptor.ActionName;
        var currentControllerName = filterContext.ActionDescriptor.ControllerDescriptor.ControllerName;
        var userId = HttpContext.Current.User.Identity.GetUserId<int>();

        if (!_groupManager.UserHasAdministratorAccess(userId))
        {
            if (!_permissionManager.HasPermission((currentAreaName == null ? String.Empty : currentAreaName.ToString()), currentControllerName, currentActionName, userId))
            {
                filterContext.Result = new RedirectToRouteResult(new RouteValueDictionary { { "controller", "Account" }, { "action", "Login" } });
            }
        }

        base.OnActionExecuting(filterContext);
    }
}

我已经阅读了足够多的内容，意识到这是 MVC3 中引入的以下更改的问题

重大更改：在以前版本的 ASP.NET MVC 中，操作过滤器 除少数情况外，按请求创建。这种行为从来没有 一个有保证的行为，但仅仅是一个实现细节和 过滤器的合同是认为它们是无国籍的。在 ASP.NET MVC 3 中， 过滤器被更积极地缓存。因此，任何自定义操作 不正确地存储实例状态的过滤器可能会被破坏。

我不确定如何最好地解决此问题。我考虑将我的两个私有只读字段移动到 OnActionExecuting 部分，我相信这可以解决问题，但我担心多线程以及该实现是否存在问题。

似乎很多人使用 Castle Windsor 或 Ninject 解决了这个问题，但这些超出了我的专业水平，即使在完成了 Windsor 教程 (https://github.com/castleproject/Windsor/blob/master/docs/mvc-tutorial-intro.md) 之后，我也无法理解我到底需要做什么.

Answer 1

问题在于您初始化并保留的两个字段：

private readonly ApplicationGroupManager _groupManager = new ApplicationGroupManager();
private readonly ActionPermissionManager _permissionManager = new ActionPermissionManager();

大概其中至少有一个具有 DbContext 实例，该实例也在构造函数或字段/属性初始化程序中初始化。我显然不知道这些类的内部工作原理，但看起来它们没有被任何其他方法使用。它们确实应该在方法而不是类字段中声明：

public override void OnActionExecuting(ActionExecutingContext filterContext)
{
    ApplicationGroupManager groupManager = new ApplicationGroupManager();
    ActionPermissionManager permissionManager = new ActionPermissionManager();

    // The rest of your code here
}

这也将解决您看到的错误。由于每个请求都会重新初始化 Manager 实例（及其 DbContext 成员），因此您不必担心代码会尝试重用已处理的实例。

Answer 2

另一种方法是创建一个代理过滤器类，它将实例化并连接实际的过滤器。

public class PermissionFilterProxy : ActionFilterAttribute
    {
        public override void OnActionExecuting(ActionExecutingContext filterContext)
        {
            // create and wire the actual filter
            // so in this way its lifetime is handled by us
            PermissionFilter permissionFilter = new PermissionFilter();
            permissionFilter.OnActionExecuting(filterContext);
            base.OnActionExecuting(filterContext);

        }
    }

现在您将配置（添加）此代理，而不是 PermissionFilter。