ASP.NET MVC - 如何在RedirectToAction中保留ModelState错误?

问题描述 投票:78回答:9

我有以下两种操作方法(简化问题):

[HttpGet]
public ActionResult Create(string uniqueUri)
{
   // get some stuff based on uniqueuri, set in ViewData.  
   return View();
}

[HttpPost]
public ActionResult Create(Review review)
{
   // validate review
   if (validatedOk)
   {
      return RedirectToAction("Details", new { postId = review.PostId});
   }  
   else
   {
      ModelState.AddModelError("ReviewErrors", "some error occured");
      return RedirectToAction("Create", new { uniqueUri = Request.RequestContext.RouteData.Values["uniqueUri"]});
   }   
}

因此,如果验证通过,我将重定向到另一页(确认)。

如果发生错误,我需要显示包含错误的同一页面。

如果我做return View(),则会显示错误,但如果我执行return RedirectToAction(如上所述),则会丢失模型错误。

我对这个问题并不感到惊讶,只是想知道你们是如何处理这个问题的?

我当然可以返回相同的View而不是重定向,但我在“Create”方法中有逻辑,它填充了视图数据,我必须复制它。

有什么建议?

asp.net-mvc error-handling modelstate redirecttoaction http-redirect
9个回答
45
投票

您需要在Review动作上使用相同的HttpGet实例。要做到这一点,你应该在Review review动作的临时变量中保存一个对象HttpPost,然后在HttpGet动作上恢复它。

[HttpGet]
public ActionResult Create(string uniqueUri)
{
   //Restore
   Review review = TempData["Review"] as Review;            

   // get some stuff based on uniqueuri, set in ViewData.  
   return View(review);
}
[HttpPost]
public ActionResult Create(Review review)
{
   //Save you object
   TempData["Review"] = review;

   // validate review
   if (validatedOk)
   {
      return RedirectToAction("Details", new { postId = review.PostId});
   }  
   else
   {
      ModelState.AddModelError("ReviewErrors", "some error occured");
      return RedirectToAction("Create", new { uniqueUri = Request.RequestContext.RouteData.Values["uniqueUri"]});
   }   
}

另外我会建议,如果你想在第一次执行HttpGet动作后按下浏览器刷新按钮时也能正常工作,你可能会这样做

  Review review = TempData["Review"] as Review;  
  TempData["Review"] = review;

否则刷新按钮对象review将为空,因为TempData["Review"]中没有任何数据。


71
投票

我今天必须自己解决这个问题,并遇到了这个问题。

一些答案很有用(使用TempData),但并没有真正回答手头的问题。

我发现的最佳建议是在这篇博客文章中:

http://www.jefclaes.be/2012/06/persisting-model-state-when-using-prg.html

基本上,使用TempData来保存和恢复ModelState对象。但是,如果将其抽象为属性,则会更加清晰。

EG

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"]);
        }
    }
}

然后根据您的示例,您可以保存/恢复ModelState,如下所示:

[HttpGet]
[RestoreModelStateFromTempData]
public ActionResult Create(string uniqueUri)
{
    // get some stuff based on uniqueuri, set in ViewData.  
    return View();
}

[HttpPost]
[SetTempDataModelState]
public ActionResult Create(Review review)
{
    // validate review
    if (validatedOk)
    {
        return RedirectToAction("Details", new { postId = review.PostId});
    }  
    else
    {
        ModelState.AddModelError("ReviewErrors", "some error occured");
        return RedirectToAction("Create", new { uniqueUri = Request.RequestContext.RouteData.Values["uniqueUri"]});
    }   
}

如果您还想在TempData中传递模型(建议使用bigb),那么您仍然可以这样做。


7
投票

为什么不使用“Create”方法中的逻辑创建一个私有函数,并从Get和Post方法中调用此方法,然后返回View()。


4
投票

我可以使用TempData["Errors"]

TempData通过保留数据1次的动作传递。


4
投票

我建议你返回视图,并通过动作上的属性避免重复。以下是填充查看数据的示例。您可以使用create方法逻辑执行类似的操作。

public class GetStuffBasedOnUniqueUriAttribute : ActionFilterAttribute
{
    public override void OnActionExecuting(ActionExecutingContext filterContext)
    {
        var filter = new GetStuffBasedOnUniqueUriFilter();

        filter.OnActionExecuting(filterContext);
    }
}


public class GetStuffBasedOnUniqueUriFilter : IActionFilter
{
    #region IActionFilter Members

    public void OnActionExecuted(ActionExecutedContext filterContext)
    {

    }

    public void OnActionExecuting(ActionExecutingContext filterContext)
    {
        filterContext.Controller.ViewData["somekey"] = filterContext.RouteData.Values["uniqueUri"];
    }

    #endregion
}

这是一个例子:

[HttpGet, GetStuffBasedOnUniqueUri]
public ActionResult Create()
{
    return View();
}

[HttpPost, GetStuffBasedOnUniqueUri]
public ActionResult Create(Review review)
{
    // validate review
    if (validatedOk)
    {
        return RedirectToAction("Details", new { postId = review.PostId });
    }

    ModelState.AddModelError("ReviewErrors", "some error occured");
    return View(review);
}

2
投票

我有一个方法,可以将模型状态添加到临时数据。然后我在我的基本控制器中有一个方法来检查临时数据是否有任何错误。如果有,则将它们添加回ModelState。


0
投票

我的场景有点复杂,因为我正在使用PRG模式,所以我的ViewModel(“SummaryVM”)在TempData中,我的摘要屏幕显示它。此页面上有一个小表单,可将​​某些信息发布到另一个操作。复杂性来自于用户要求在此页面上编辑SummaryVM中的某些字段。

Summary.cshtml具有验证摘要,它将捕获我们将创建的ModelState错误。

@Html.ValidationSummary()

我的表单现在需要POST到一个HttpPost动作的Summary()。我有另一个非常小的ViewModel来表示已编辑的字段,而modelbinding将会得到这些字段。

新表格:

@using (Html.BeginForm("Summary", "MyController", FormMethod.Post))
{
    @Html.Hidden("TelNo") @* // Javascript to update this *@

和行动......

[HttpPost]
public ActionResult Summary(EditedItemsVM vm)

在这里我做了一些验证,我检测到一些错误的输入,所以我需要返回带有错误的Summary页面。为此,我使用TempData,它将在重定向后继续使用。如果数据没有问题,我用一个副本替换SummaryVM对象(但当然改变了编辑的字段)然后执行RedirectToAction(“NextAction”);

// Telephone number wasn't in the right format
List<string> listOfErrors = new List<string>();
listOfErrors.Add("Telephone Number was not in the correct format. Value supplied was: " + vm.TelNo);
TempData["SummaryEditedErrors"] = listOfErrors;
return RedirectToAction("Summary");

所有这些开始的摘要控制器操作会查找tempdata中的任何错误,并将它们添加到模型状态。

[HttpGet]
[OutputCache(Duration = 0)]
public ActionResult Summary()
{
    // setup, including retrieval of the viewmodel from TempData...


    // And finally if we are coming back to this after a failed attempt to edit some of the fields on the page,
    // load the errors stored from TempData.
        List<string> editErrors = new List<string>();
        object errData = TempData["SummaryEditedErrors"];
        if (errData != null)
        {
            editErrors = (List<string>)errData;
            foreach(string err in editErrors)
            {
                // ValidationSummary() will see these
                ModelState.AddModelError("", err);
            }
        }

0
投票

我更喜欢在我的ViewModel中添加一个方法来填充默认值:

public class RegisterViewModel
{
    public string FirstName { get; set; }
    public IList<Gender> Genders { get; set; }
    //Some other properties here ....
    //...
    //...

    ViewModelType PopulateDefaultViewData()
    {
        this.FirstName = "No body";
        this.Genders = new List<Gender>()
        {
            Gender.Male,
            Gender.Female
        };

        //Maybe other assinments here for other properties...
    }
}

然后,当我需要这样的原始数据时,我会调用它:

    [HttpGet]
    public async Task<IActionResult> Register()
    {
        var vm = new RegisterViewModel().PopulateDefaultViewValues();
        return View(vm);
    }

    [HttpPost]
    public async Task<IActionResult> Register(RegisterViewModel vm)
    {
        if (!ModelState.IsValid)
        {
            return View(vm.PopulateDefaultViewValues());
        }

        var user = await userService.RegisterAsync(
            email: vm.Email,
            password: vm.Password,
            firstName: vm.FirstName,
            lastName: vm.LastName,
            gender: vm.Gender,
            birthdate: vm.Birthdate);

        return Json("Registered successfully!");
    }

0
投票

Microsoft删除了在TempData中存储复杂数据类型的功能,因此之前的答案不再有效;你只能存储像字符串这样的简单类型。我已经改变@ asgeo1的答案,按预期工作。

public class SetTempDataModelStateAttribute : ActionFilterAttribute
{
    public override void OnActionExecuted(ActionExecutedContext filterContext)
    {
        base.OnActionExecuted(filterContext);

        var controller = filterContext.Controller as Controller;
        var modelState = controller?.ViewData.ModelState;
        if (modelState != null)
        {
            var listError = modelState.Where(x => x.Value.Errors.Any())
                .ToDictionary(m => m.Key, m => m.Value.Errors
                .Select(s => s.ErrorMessage)
                .FirstOrDefault(s => s != null));
            controller.TempData["KEY HERE"] = JsonConvert.SerializeObject(listError);
        }
    }

public class RestoreModelStateFromTempDataAttribute : ActionFilterAttribute
{
    public override void OnActionExecuting(ActionExecutingContext filterContext)
    {
        base.OnActionExecuting(filterContext);

        var controller = filterContext.Controller as Controller;
        var tempData = controller?.TempData?.Keys;
        if (controller != null && tempData != null)
        {
            if (tempData.Contains("KEY HERE"))
            {
                var modelStateString = controller.TempData["KEY HERE"].ToString();
                var listError = JsonConvert.DeserializeObject<Dictionary<string, string>>(modelStateString);
                var modelState = new ModelStateDictionary();
                foreach (var item in listError)
                {
                        modelState.AddModelError(item.Key, item.Value ?? "");
                }

                controller.ViewData.ModelState.Merge(modelState);
            }
        }
    }        
}
}

从这里,您可以根据需要在控制器方法上简单地添加所需的数据注释。

[RestoreModelStateFromTempDataAttribute]
    [HttpGet]
    public async Task<IActionResult> MethodName()
    {
    }

[SetTempDataModelStateAttribute]
    [HttpPost]
    public async Task<IActionResult> MethodName()
    {
            ModelState.AddModelError("KEY HERE", "ERROR HERE");
    }
© www.soinside.com 2019 - 2024. All rights reserved.