Question

我对＆＃34; Preserve ModelState Errors Across RedirectToAction?＆＃34;实施了第二次回复。涉及使用两个自定义ActionFilterAttributes的问题。我喜欢这个解决方案，它通过向需要该功能的方法添加属性来保持代码清洁。

在大多数情况下，该解决方案效果很好，但我遇到了重复部分视图的问题。基本上我有部分视图使用它自己的模型，与父视图使用的模型分开。

主视图中我的代码的简化版本：

@for (int i = 0; i < Model.Addresses.Count; i++)
{
        address = (Address)Model.Addresses[i];
        @Html.Partial("_AddressModal", address);
}

部分视图＆＃34; _AddressModal＆＃34;：

@model Acme.Domain.Models.Address
[...]
@Html.TextBoxFor(model => model.Address1, new { @class = "form-control" } )
[...]

当不使用自定义ActionFilterAttributes时，一切都按预期工作。每次执行部分视图时，lamba表达式都是＆＃34; model =＆gt; model.Address1＆＃34;从ModelState中提取正确的值。

问题是当我获得重定向并使用自定义ActionFilterAttributes时。核心问题是，不仅更新了一个Address实例的ModelState，而且部分视图构建的所有地址的ModelState都被覆盖，因此它们包含相同的值，而不是正确的实例值。

我的问题是如何修改自定义ActionFilterAttributes，以便它只更新受影响的一个Address实例的ModelState，而不是所有ModelStates？我想避免向使用该属性的方法添加任何内容，以保持干净的实现。

以下是其他问题的自定义ActionFilterAttributes代码：

public class SetTempDataModelStateAttribute : ActionFilterAttribute
{
    public override void OnActionExecuted(ActionExecutedContext filterContext)
    {
        base.OnActionExecuted(filterContext);         
        filterContext.Controller.TempData["ModelState"] = 
           filterContext.Controller.ViewData.ModelState;
    }
}

public class RestoreModelStateFromTempDataAttribute : ActionFilterAttribute
{
    public override void OnActionExecuting(ActionExecutingContext filterContext)
    {
        base.OnActionExecuting(filterContext);
        if (filterContext.Controller.TempData.ContainsKey("ModelState"))
        {
            filterContext.Controller.ViewData.ModelState.Merge(
                (ModelStateDictionary)filterContext.Controller.TempData["ModelState"]);
        }
    }
}

Answer 1

查看this implementation（ben foster）是否有效：我大量使用它，从来没有遇到过问题。

您是否正确设置了属性？ RestoreModelStateFromTempDataAttribute行动上的get和SetTempDataModelState行动中的post

以下是需要的4个类（导出，导入，传输和验证）ModelState

 [AttributeUsage(AttributeTargets.Class | AttributeTargets.Method, AllowMultiple = false, Inherited = true)]
    public class ExportModelStateToTempDataAttribute : ModelStateTempDataTransfer
    {
        public override void OnActionExecuted(ActionExecutedContext filterContext)
        {
            // Only copy when ModelState is invalid and we're performing a Redirect (i.e. PRG)
            if (!filterContext.Controller.ViewData.ModelState.IsValid &&
                (filterContext.Result is RedirectResult || filterContext.Result is RedirectToRouteResult)) 
            {
                ExportModelStateToTempData(filterContext);
            }

            base.OnActionExecuted(filterContext);
        }
    }


 /// <summary>
    /// An Action Filter for importing ModelState from TempData.
    /// You need to decorate your GET actions with this when using the <see cref="ValidateModelStateAttribute"/>.
    /// </summary>
    /// <remarks>
    /// Useful when following the PRG (Post, Redirect, Get) pattern.
    /// </remarks>
    [AttributeUsage(AttributeTargets.Class | AttributeTargets.Method, AllowMultiple = false, Inherited = true)]
    public class ImportModelStateFromTempDataAttribute : ModelStateTempDataTransfer
    {
        public override void OnActionExecuted(ActionExecutedContext filterContext)
        {
            // Only copy from TempData if we are rendering a View/Partial
            if (filterContext.Result is ViewResult)
            {
                ImportModelStateFromTempData(filterContext);
            }
            else 
            {
                // remove it
                RemoveModelStateFromTempData(filterContext);
            }

            base.OnActionExecuted(filterContext);
        }
    }

 [AttributeUsage(AttributeTargets.Class | AttributeTargets.Method, AllowMultiple = false, Inherited = true)]
    public abstract class ModelStateTempDataTransfer : ActionFilterAttribute
    {
        protected static readonly string Key = typeof(ModelStateTempDataTransfer).FullName;

        /// <summary>
        /// Exports the current ModelState to TempData (available on the next request).
        /// </summary>       
        protected static void ExportModelStateToTempData(ControllerContext context)
        {
            context.Controller.TempData[Key] = context.Controller.ViewData.ModelState;
        }

        /// <summary>
        /// Populates the current ModelState with the values in TempData
        /// </summary>
        protected static void ImportModelStateFromTempData(ControllerContext context)
        {
            var prevModelState = context.Controller.TempData[Key] as ModelStateDictionary;
            context.Controller.ViewData.ModelState.Merge(prevModelState);
        }

        /// <summary>
        /// Removes ModelState from TempData
        /// </summary>
        protected static void RemoveModelStateFromTempData(ControllerContext context)
        {
            context.Controller.TempData[Key] = null;
        }
    }

  /// <summary>
    /// An ActionFilter for automatically validating ModelState before a controller action is executed.
    /// Performs a Redirect if ModelState is invalid. Assumes the <see cref="ImportModelStateFromTempDataAttribute"/> is used on the GET action.
    /// </summary>
    [AttributeUsage(AttributeTargets.Class | AttributeTargets.Method, AllowMultiple = false, Inherited = true)]
    public class ValidateModelStateAttribute : ModelStateTempDataTransfer
    {
        public override void OnActionExecuting(ActionExecutingContext filterContext)
        {
            if (!filterContext.Controller.ViewData.ModelState.IsValid)
            {
                if (filterContext.HttpContext.Request.IsAjaxRequest())
                {
                    ProcessAjax(filterContext);
                }
                else
                {
                    ProcessNormal(filterContext);
                }
            }

            base.OnActionExecuting(filterContext);
        }

        protected virtual void ProcessNormal(ActionExecutingContext filterContext)
        {
            // Export ModelState to TempData so it's available on next request
            ExportModelStateToTempData(filterContext);

            // redirect back to GET action
            filterContext.Result = new RedirectToRouteResult(filterContext.RouteData.Values);
        }

        protected virtual void ProcessAjax(ActionExecutingContext filterContext)
        {
            var errors = filterContext.Controller.ViewData.ModelState.ToSerializableDictionary();
            var json = new JavaScriptSerializer().Serialize(errors);

            // send 400 status code (Bad Request)
            filterContext.Result = new HttpStatusCodeResult((int)HttpStatusCode.BadRequest, json);
        }
    }

修改

这是普通（非动作过滤器）PRG模式：

[HttpGet] public async Task<ActionResult> Edit(Guid id) { var calendarEvent = await calendarService.FindByIdAsync(id); if (calendarEvent == null) return this.RedirectToAction<CalendarController>(c => c.Index()); var model = new CalendarEditViewModel(calendarEvent); ViewData.Model = model; return View(); } [HttpPost] public async Task<ActionResult> Edit(Guid id, CalendarEventBindingModel binding) { if (!ModelState.IsValid) return await Edit(id); var calendarEvent = await calendarService.FindByIdAsync(id); if (calendarEvent != null) { CalendarEvent model = calendarService.Update(calendarEvent, binding); await context.SaveChangesAsync(); } return this.RedirectToAction<CalendarController>(c => c.Index()); }

使用动作过滤器（或其目的）要避免的是删除每个帖子Action上的ModelState.IsValid检查，因此相同（使用动作过滤器）将是：

[HttpGet, ImportModelStateFromTempData] public async Task<ActionResult> Edit(Guid id) { var calendarEvent = await calendarService.FindByIdAsync(id); if (calendarEvent == null) return this.RedirectToAction<CalendarController>(c => c.Index()); var model = new CalendarEditViewModel(calendarEvent); ViewData.Model = model; return View(); } // ActionResult changed to RedirectToRouteResult [HttpPost, ValidateModelState] public async Task<RedirectToRouteResult> Edit(Guid id, CalendarEventBindingModel binding) { // removed ModelState.IsValid check var calendarEvent = await calendarService.FindByIdAsync(id); if (calendarEvent != null) { CalendarEvent model = calendarService.Update(calendarEvent, binding); await context.SaveChangesAsync(); } return this.RedirectToAction<CalendarController>(c => c.Index()); }

这里没有更多的事情发生。因此，如果您只使用ExportModelState操作过滤器，您将最终得到如下的后期操作：

[HttpPost, ExportModelStateToTempData] public async Task<RedirectToRouteResult> Edit(Guid id, CalendarEventBindingModel binding) { if (!ModelState.IsValid) return RedirectToAction("Edit", new { id }); var calendarEvent = await calendarService.FindByIdAsync(id); if (calendarEvent != null) { CalendarEvent model = calendarService.Update(calendarEvent, binding); await context.SaveChangesAsync(); } return this.RedirectToAction<CalendarController>(c => c.Index()); }

这让我问你，为什么你首先要打扰ActionFilters？虽然我确实喜欢ValidateModelState模式（很多人没有），但是如果你在控制器中重定向我没有看到任何好处，除了一个场景，你有额外的模型状态错误，为了完整性让我给你一个例子：

[HttpPost, ValidateModelState, ExportModelStateToTempData] public async Task<RedirectToRouteResult> Edit(Guid id, CalendarEventBindingModel binding) { var calendarEvent = await calendarService.FindByIdAsync(id); if (!(calendarEvent.DateStart > DateTime.UtcNow.AddDays(7)) && binding.DateStart != calendarEvent.DateStart) { ModelState.AddModelError("id", "Sorry, Date start cannot be updated with less than 7 days of event."); return RedirectToAction("Edit", new { id }); } if (calendarEvent != null) { CalendarEvent model = calendarService.Update(calendarEvent, binding); await context.SaveChangesAsync(); } return this.RedirectToAction<CalendarController>(c => c.Index()); }

在上一个示例中，我同时使用了ValidateModelState和ExportModelState，这是因为ValidateModelState在ActionExecuting上运行，因此在进入方法正文之前进行验证，如果绑定有一些验证错误，它会自动重定向。然后我有一个不能在数据注释中的另一个检查，因为它处理加载一个实体并查看它是否有正确的要求（我知道这不是最好的例子，把它看作是在注册时提供的用户名是否可用），我知道远程数据注释，但并不涵盖所有情况）然后我只是根据除绑定之外的外部因素更新ModelState我自己的错误。由于ExportModelState在ActionExecuted上运行，我对ModelState的所有修改都会保留在TempData上，因此我会在HttpGet修改操作上进行修改。

我知道这一切可能让我们中的一些人感到困惑，关于如何在Controller / PRG端进行MVC没有很好的指示。我正在努力制作博客文章以涵盖所有场景和解决方案。这只是它的1％。

我希望至少我清除了POST-GET工作流程的几个关键点。如果这有点困扰，请告诉我。对不起，很长的帖子。

我还要注意PRG中有一个微妙的区别，返回一个返回RedirectToRouteResult的ActionResult。如果在使用ValidationError后使用RedirectToRouteResult刷新页面（F5），则错误将不会保留，并且您将获得一个干净的视图，就像您第一次输入一样。使用ActionResult，您可以刷新并查看包含错误的完全相同的页面。这与ActionResult或RedirectToRouteResult返回类型无关，因为在一种情况下，您始终在POST上重定向，而另一种情况仅在成功POST时重定向。 PRG并不建议在不成功的POST上进行重定向，但有些人更喜欢在每个帖子上进行重定向，这需要TempData传输。

条件ModelState合并

1 个答案: