我有以下两种操作方法（简化问题）：
[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”方法中有逻辑，它填充了视图数据，我必须复制。
有什么建议吗？
答案 0 :(得分：71)
我今天必须自己解决这个问题，并遇到了这个问题。
有些答案很有用（使用TempData），但并没有真正回答手头的问题。
我发现的最佳建议是在这篇博客文章中：
http://www.jefclaes.be/2012/06/persisting-model-state-when-using-prg.html
基本上，使用TempData来保存和恢复ModelState对象。但是，如果将它抽象为属性，它会更加清晰。
E.g。
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），那么您仍然可以这样做。
答案 1 :(得分：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"]中没有任何数据。
答案 2 :(得分：7)
为什么不使用“Create”方法中的逻辑创建一个私有函数，并从Get和Post方法中调用此方法，然后返回View（）。
答案 3 :(得分：4)
我可以使用TempData["Errors"]
TempData通过保留数据1次的动作传递。
答案 4 :(得分：4)
我建议你返回视图，并通过动作上的属性避免重复。以下是填充查看数据的示例。您可以使用创建方法逻辑执行类似的操作。
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);
}
答案 5 :(得分：2)
我有一个方法可以将模型状态添加到临时数据中。然后我在我的基本控制器中有一个方法来检查临时数据是否有任何错误。如果有，则将它们添加回ModelState。
答案 6 :(得分：1)
public class RestoreModelStateFromTempDataAttribute : ActionFilterAttribute
{
public override void OnActionExecuting(ActionExecutingContext filterContext)
{
var controller = filterContext.Controller as Controller;
if (controller.TempData.ContainsKey("ModelState"))
{
var modelState = ModelStateHelpers.DeserialiseModelState(controller.TempData["ModelState"].ToString());
controller.ViewData.ModelState.Merge(modelState);
}
base.OnActionExecuting(filterContext);
}
}
public class SetTempDataModelStateAttribute : ActionFilterAttribute
{
public override void OnActionExecuted(ActionExecutedContext filterContext)
{
var controller = filterContext.Controller as Controller;
controller.TempData["ModelState"] = ModelStateHelpers.SerialiseModelState(controller.ViewData.ModelState);
base.OnActionExecuted(filterContext);
}
}
当我解决一些问题时，我遇到了很多不明显的障碍。我会一步一步地说明一切。 我的评论将部分复制当前分支的答案
答案 7 :(得分：0)
我的场景有点复杂，因为我正在使用PRG模式，所以我的ViewModel（＆＃34; SummaryVM＆＃34;）在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（＆＃34; NextAction＆＃34;）;
// 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);
}
}
答案 8 :(得分：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!");
}
答案 9 :(得分：0)
答案 10 :(得分：0)
我在这里只提供示例代码 在您的viewModel中，您可以添加一个类型为“ ModelStateDictionary”的属性作为
public ModelStateDictionary ModelStateErrors { get; set; }
在POST操作方法中，您可以像这样直接编写代码
model.ModelStateErrors = ModelState;
然后按如下所示将此模型分配给Tempdata
TempData["Model"] = model;
，当您重定向到其他控制器的操作方法时，则必须在控制器中读取Tempdata值
if (TempData["Model"] != null)
{
viewModel = TempData["Model"] as ViewModel; //Your viewmodel class Type
if(viewModel.ModelStateErrors != null && viewModel.ModelStateErrors.Count>0)
{
this.ViewData.ModelState.Merge(viewModel.ModelStateErrors);
}
}
就是这样。您不必为此编写动作过滤器。如果要将模型状态错误发送到另一个控制器的另一个视图，这和上面的代码一样简单。