如何使用 RedirectToAction 维护 ModelState？

如果在我的 ModelState中有一个错误而不丢失我的 ModelState信息，我怎样才能返回一个不同操作的结果或者将用户移动到一个不同的操作？

场景是: Delete操作接受来自我的 Index操作/视图呈现的 DELETE 表单的 POST。如果在 Delete中有一个错误，我希望将用户移回到 Index Action/View，并显示在 ViewData.ModelState中由 Delete动作存储的错误。在 ASP.NET MVC 中如何做到这一点？

[AcceptVerbs(HttpVerbs.Post | HttpVerbs.Delete)]
public ActionResult Delete([ModelBinder(typeof(RdfUriBinder))] RdfUri graphUri)
{
if (!ModelState.IsValid)
return Index(); //this needs to be replaced with something that works :)


return RedirectToAction("Index");
}
41741 次浏览
小开

试试吧

return View("Index");

而不是

return Index();
小开
最佳答案

将视图数据存储在 TempData中，并在 Index操作中检索它(如果存在的话)。

   ...
if (!ModelState.IsValid)
TempData["ViewData"] = ViewData;


RedirectToAction( "Index" );
}


public ActionResult Index()
{
if (TempData["ViewData"] != null)
{
ViewData = (ViewDataDictionary)TempData["ViewData"];
}


...
}

[编辑]我检查了 MVC 的在线源代码，看起来控制器中的 ViewData是可以设置的，所以把所有的 ViewData，包括 ModelState，转移到 Index 操作可能是最简单的。

小开

请注意，尽管在大多数情况下，tvanfoson 的解决方案并不总是有效，但它应该是正常的。

这个特定解决方案的问题在于，如果您已经有了任何 ViewData 或 ModelState，那么您最终会用前一个请求的状态覆盖它们。例如，新请求可能有一些与传递给操作的无效参数相关的模型状态错误，但是这些错误最终会被隐藏，因为它们被覆盖了。

另一种情况是，如果您的 Action Filter 初始化了一些 ViewData 或 ModelState 错误，那么它可能不会像预期的那样工作。同样，它们会被代码覆盖。

我们正在寻找一些 ASP.NET MVC 的解决方案，它们可以让你更容易地从两个请求中合并状态，所以请继续关注。

谢谢, 艾伦

小开

使用动作过滤器(PRG 模式)(和使用属性一样简单)

提到了 给你和 给你。

小开

如果这对我使用@bob 推荐的 PRG 解决方案的人有用的话:

见第13项-> 链接。

在执行 RedirectToAction("Action")操作时，我遇到了额外的问题，即在控制器操作中从 TempData 手动编写和检查/加载 VeiwBag 传递到 View 的消息。为了简化(并使其可维护) ，我稍微扩展了这种方法来检查和存储/加载其他数据。我的行动方式是这样的:

 [AcceptVerbs(HttpVerbs.Post)]
[ExportModelStateToTempData]
public ActionResult ChangePassword(ProfileViewModel pVM) {
bool result = MyChangePasswordCode(pVM.ChangePasswordViewModel);
if (result) {
ViewBag.Message = "Password change success";
else {
ModelState.AddModelError("ChangePassword", "Some password error");
}
return RedirectToAction("Index");
}

我的索引行动:

[ImportModelStateFromTempData]
public ActionResult Index() {
ProfileViewModel pVM = new ProfileViewModel { //setup }
return View(pVM);
}

操作过滤器中的代码:

// Following best practices as listed here for storing / restoring model data:
// http://weblogs.asp.net/rashid/archive/2009/04/01/asp-net-mvc-best-practices-part-1.aspx#prg
public abstract class ModelStateTempDataTransfer : ActionFilterAttribute {
protected static readonly string Key = typeof(ModelStateTempDataTransfer).FullName;
}

:

public class ExportModelStateToTempData : ModelStateTempDataTransfer {
public override void OnActionExecuted(ActionExecutedContext filterContext) {
//Only export when ModelState is not valid
if (!filterContext.Controller.ViewData.ModelState.IsValid) {
//Export if we are redirecting
if ((filterContext.Result is RedirectResult) || (filterContext.Result is RedirectToRouteResult)) {
filterContext.Controller.TempData[Key] = filterContext.Controller.ViewData.ModelState;
}
}
// Added to pull message from ViewBag
if (!string.IsNullOrEmpty(filterContext.Controller.ViewBag.Message)) {
filterContext.Controller.TempData["Message"] = filterContext.Controller.ViewBag.Message;
}


base.OnActionExecuted(filterContext);
}
}

:

public class ImportModelStateFromTempData : ModelStateTempDataTransfer {
public override void OnActionExecuted(ActionExecutedContext filterContext) {
ModelStateDictionary modelState = filterContext.Controller.TempData[Key] as ModelStateDictionary;


if (modelState != null) {
//Only Import if we are viewing
if (filterContext.Result is ViewResult) {
filterContext.Controller.ViewData.ModelState.Merge(modelState);
} else {
//Otherwise remove it.
filterContext.Controller.TempData.Remove(Key);
}
}
// Restore Viewbag message
if (!string.IsNullOrEmpty((string)filterContext.Controller.TempData["Message"])) {
filterContext.Controller.ViewBag.Message = filterContext.Controller.TempData["Message"];
}


base.OnActionExecuted(filterContext);
}
}

我意识到我在这里所做的改变是对 ModelState 已经通过@the link@bob 提供的代码所做的一个非常明显的扩展——但是在我想到用这种方式处理它之前，我不得不在这个线程上磕磕绊绊。

小开

请不要为了这个答案刺我，这是一个合理的建议。

使用 AJAX

用于管理 ModelState 的代码很复杂，并且(可能?)表明代码中存在其他问题。

你可以很容易地滚动你自己的 AJAX javascript 代码:

Https://gist.github.com/jesslilly/5f646ef29367ad2b0228e1fa76d6bdcc#file-ajaxform

(function ($) {


$(function () {


// For forms marked with data-ajax="#container",
// on submit,
// post the form data via AJAX
// and if #container is specified, replace the #container with the response.
var postAjaxForm = function (event) {


event.preventDefault(); // Prevent the actual submit of the form.


var $this = $(this);
var containerId = $this.attr("data-ajax");
var $container = $(containerId);
var url = $this.attr('action');


console.log("Post ajax form to " + url + " and replace html in " + containerId);


$.ajax({
type: "POST",
url: url,
data: $this.serialize()
})
.done(function (result) {
if ($container) {
$container.html(result);
// re-apply this event since it would have been lost by the form getting recreated above.
var $newForm = $container.find("[data-ajax]");
$newForm.submit(postAjaxForm);
$newForm.trigger("data-ajax-done");
}
})
.fail(function (error) {
alert(error);
});
};
$("[data-ajax]").submit(postAjaxForm);
});


})(jQuery);

提交答案