web-dev-qa-db-ja.com

アクションフィルターのセッション変数

セッション変数IDが設定されているかどうかをチェックするアクションフィルターがあります。開発目的で、チェックの前にこの変数を手動で設定しました。

public class MyActionFilterAttribute : ActionFilterAttribute
    {
        public override void OnActionExecuted(ActionExecutedContext context)
        {

            context.HttpContext.Session.Add("ID", 123123);

            int ID = (int)context.HttpContext.Session.Contents["ID"];
            var rd = context.HttpContext.Request.RequestContext.RouteData;

            TED _db = new TED();

            //if not in DB
            if (_db.Users.Find(ID) == null && rd.GetRequiredString("action") != "NoAccess")
            {
                RouteValueDictionary redirectTargetDictionary = new RouteValueDictionary();
                redirectTargetDictionary.Add("action", "NoAccess");
                redirectTargetDictionary.Add("controller", "Home");
                redirectTargetDictionary.Add("area", "");

                context.Result = new RedirectToRouteResult(redirectTargetDictionary);
            }

            base.OnActionExecuted(context);
        }
    }

私が理解している限り、このコードはページの前に実行されます。これはSession["ID"]は常に設定されます。一貫してテストしている場合はサイトは正常に動作しますが、しばらくそのままにしておいてテストを続行しようとすると、サイトが壊れているようです。ここに私が得るエラーがあります:

int UserID = (int)Session.Contents["ID"];
System.NullReferenceException: Object reference not set to an instance of an object.

最初はセッションが単に期限切れになる可能性があると思っていましたが、ページをロードする前にSession["ID"]が設定されているはずです。ここの問題は何ですか?

19
Jeff

アクションフィルターのon OnActionExecutedメソッドを実装し、アクションメソッドの後に実行する

OnActionExecutingメソッドを実装する必要があります

public class MyActionFilterAttribute : ActionFilterAttribute
    {
        public override void OnActionExecuting(ActionExecutingContext context)
        {

            context.HttpContext.Session.Add("ID", 123123);

            int ID = (int)context.HttpContext.Session.Contents["ID"];
            var rd = context.HttpContext.Request.RequestContext.RouteData;

            TED _db = new TED();

            //if not in DB
            if (_db.Users.Find(ID) == null && rd.GetRequiredString("action") != "NoAccess")
            {
                RouteValueDictionary redirectTargetDictionary = new RouteValueDictionary();
                redirectTargetDictionary.Add("action", "NoAccess");
                redirectTargetDictionary.Add("controller", "Home");
                redirectTargetDictionary.Add("area", "");

                context.Result = new RedirectToRouteResult(redirectTargetDictionary);
            }

            base.OnActionExecuting(context);
        }
    }
26
Dave Alperovich