ASP.NET MVC-RedirectToAction全体でModelStateエラーを保持する方法

次の2つのアクションメソッドがあります（質問では簡略化しています）。

[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（上記のように）実行すると、モデルエラーが失われます。

私はこの問題に驚いていない、あなたたちがこれをどのように処理するのかと思っているだけですか？

もちろん、リダイレクトの代わりに同じビューを返すだけでもかまいませんが、「Create」メソッドにロジックがあり、ビューデータを入力します。これを複製する必要があります。

助言がありますか？

あなたは、同じインスタンスを持っている必要があり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 your 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"]。

私は自分でこの問題を解決しなければならず、この質問に出くわしました。

いくつかの回答は（TempDataを使用して）役立ちますが、実際の質問には実際には回答しません。

私が見つけた最高のアドバイスはこのブログ投稿にありました：

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

基本的に、TempDataを使用してModelStateオブジェクトを保存および復元します。ただし、これを属性に抽象化すると、よりクリーンになります。

例えば

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

（bigbが示唆するように）TempDataでもモデルを渡したい場合は、それも可能です。

「Create」メソッドのロジックを使用してプライベート関数を作成し、GetメソッドとPostメソッドの両方からこのメソッドを呼び出して、単にView（）を返さないのはなぜですか。

私は使うことができた TempData["Errors"]

TempDataは、データを1回保存するアクション間で渡されます。

ビューを返し、アクションの属性による重複を避けることをお勧めします。以下は、データを表示するために入力する例です。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);
}

一時データにモデルの状態を追加するメソッドがあります。次に、一時データにエラーがないかどうかをチェックするメソッドをベースコントローラに配置します。それらがある場合は、ModelStateに追加し直します。

PRGパターンを使用しているため、私のシナリオは少し複雑になっているため、ViewModel（ "SummaryVM"）はTempDataにあり、Summary画面に表示されます。このページには、別のアクションに情報をPOSTするための小さなフォームがあります。複雑さは、ユーザーがこのページのSummaryVMの一部のフィールドを編集する必要があるために生じています。

Summary.cshtmlには、作成するModelStateエラーをキャッチする検証サマリーがあります。

@Html.ValidationSummary()

私のフォームは、Summary（）のHttpPostアクションにPOSTする必要があります。編集済みのフィールドを表す非常に小さなViewModelがもう1つあります。モデルバインディングを使用すると、これらを取得できます。

新しいフォーム：

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

そしてアクション...

[HttpPost]
public ActionResult Summary(EditedItemsVM vm)

ここで検証を行い、いくつかの不正な入力を検出したため、エラーのある[概要]ページに戻る必要があります。これには、リダイレクトを存続させる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内のエラーを探し、それらをmodelstateに追加します。

[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);
            }
        }

マイクロソフトは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");
}

デフォルト値を入力するメソッドを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!");
    }

ここではサンプルコードのみを示しています。viewModelでは、「ModelStateDictionary」タイプのプロパティを1つ追加できます。

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

それでおしまい。このためのアクションフィルターを作成する必要はありません。別のコントローラーの別のビューにモデル状態エラーを取得する場合、これは上記のコードと同じくらい簡単です。