ASP.NET MVC - Làm thế nào để Bảo tồn lỗi ModelState trên RedirectToAction?

Tôi có hai phương pháp hành động sau (đơn giản hóa cho câu hỏi):

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

Vì vậy, nếu quá trình xác nhận vượt qua, tôi sẽ chuyển hướng đến một trang khác (xác nhận).

Nếu xảy ra lỗi, tôi cần hiển thị cùng một trang có lỗi.

Nếu tôi làm vậy return View(), lỗi sẽ được hiển thị, nhưng nếu tôi làm như vậy return RedirectToAction(như trên), nó sẽ mất các lỗi Model.

Tôi không ngạc nhiên về vấn đề này, chỉ tự hỏi làm thế nào các bạn xử lý điều này?

Tất nhiên, tôi có thể chỉ trả lại cùng một Chế độ xem thay vì chuyển hướng, nhưng tôi có logic trong phương thức "Tạo" điền dữ liệu chế độ xem, mà tôi sẽ phải sao chép.

Bất kỳ đề xuất?

Bạn cần có cùng một trường hợp Reviewvề HttpGethành động của mình . Để làm điều đó, bạn nên lưu một đối tượng Review reviewtrong biến tạm thời trên HttpPosthành động của bạn và sau đó khôi phục nó khi HttpGethành động.

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

Nếu bạn muốn điều này hoạt động ngay cả khi trình duyệt được làm mới sau lần thực hiện đầu tiên của HttpGethành động, bạn có thể làm như sau:

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

Nếu không, đối tượng nút refresh reviewsẽ trống vì sẽ không có bất kỳ dữ liệu nào trong đó TempData["Review"].

Tôi đã phải tự mình giải quyết vấn đề này hôm nay và bắt gặp câu hỏi này.

Một số câu trả lời hữu ích (sử dụng TempData), nhưng không thực sự trả lời câu hỏi trong tầm tay.

Lời khuyên tốt nhất tôi tìm thấy là trên bài đăng blog này:

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

Về cơ bản, sử dụng TempData để lưu và khôi phục đối tượng ModelState. Tuy nhiên, sẽ gọn gàng hơn rất nhiều nếu bạn tóm tắt điều này thành các thuộc tính.

Ví dụ

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

Sau đó, theo ví dụ của bạn, bạn có thể lưu / khôi phục ModelState như sau:

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

Nếu bạn cũng muốn chuyển mô hình đó trong TempData (như bigb đề xuất) thì bạn vẫn có thể làm điều đó.

Tại sao không tạo một hàm private với logic trong phương thức "Create" và gọi phương thức này từ cả phương thức Get và Post và chỉ cần trả về View ().

tôi có thể sử dụng TempData["Errors"]

TempData được chuyển qua các hành động bảo quản dữ liệu 1 lần.

Tôi khuyên bạn nên trả lại chế độ xem và tránh trùng lặp thông qua một thuộc tính trên hành động. Đây là một ví dụ về cách điền để xem dữ liệu. Bạn có thể làm điều gì đó tương tự với logic phương thức tạo của bạn.

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
}

Đây là một ví dụ:

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

Tôi có một phương pháp thêm trạng thái mô hình vào dữ liệu tạm thời. Sau đó, tôi có một phương thức trong bộ điều khiển cơ sở của mình để kiểm tra dữ liệu tạm thời cho bất kỳ lỗi nào. Nếu có chúng, nó sẽ thêm chúng trở lại ModelState.

Kịch bản của tôi phức tạp hơn một chút vì tôi đang sử dụng mẫu PRG nên ViewModel ("SummaryVM") của tôi ở trong TempData và màn hình Tóm tắt của tôi hiển thị nó. Có một biểu mẫu nhỏ trên trang này để ĐĂNG một số thông tin cho một Hành động khác. Sự phức tạp đến từ yêu cầu người dùng chỉnh sửa một số trường trong SummaryVM trên trang này.

Summary.cshtml có tóm tắt xác thực sẽ bắt lỗi ModelState mà chúng tôi sẽ tạo.

@Html.ValidationSummary()

Biểu mẫu của tôi bây giờ cần ĐĂNG lên hành động HttpPost cho Summary (). Tôi có một ViewModel rất nhỏ khác để đại diện cho các trường đã chỉnh sửa và việc lập mô hình sẽ giúp tôi nhận được những điều này.

Hình thức mới:

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

và hành động ...

[HttpPost]
public ActionResult Summary(EditedItemsVM vm)

Tại đây, tôi thực hiện một số xác thực và tôi phát hiện một số đầu vào không hợp lệ, vì vậy tôi cần quay lại trang Tóm tắt có lỗi. Đối với điều này, tôi sử dụng TempData, nó sẽ tồn tại khi chuyển hướng. Nếu không có vấn đề gì với dữ liệu, tôi thay thế đối tượng SummaryVM bằng một bản sao (nhưng với các trường đã chỉnh sửa thì tất nhiên đã thay đổi) rồi thực hiện 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");

Hành động của bộ điều khiển Tóm tắt, nơi tất cả điều này bắt đầu, tìm kiếm bất kỳ lỗi nào trong dữ liệu tạm thời và thêm chúng vào 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);
            }
        }

Microsoft đã loại bỏ khả năng lưu trữ các kiểu dữ liệu phức tạp trong TempData, do đó các câu trả lời trước đó không còn hoạt động; bạn chỉ có thể lưu trữ các loại đơn giản như chuỗi. Tôi đã thay đổi câu trả lời của @ asgeo1 để hoạt động như mong đợi.

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

Từ đây, bạn có thể chỉ cần thêm chú thích dữ liệu bắt buộc vào phương thức bộ điều khiển nếu cần.

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


[SetTempDataModelStateAttribute]
[HttpPost]
public async Task<IActionResult> MethodName()
{
    ModelState.AddModelError("KEY HERE", "ERROR HERE");
}

Tôi muốn thêm một phương thức vào ViewModel của mình để điền các giá trị mặc định:

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...
    }
}

Sau đó, tôi gọi nó khi nào tôi cần dữ liệu gốc như thế này:

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

Tôi chỉ cung cấp mã mẫu ở đây Trong viewModel của bạn, bạn có thể thêm một thuộc tính kiểu "ModelStateDictionary" là

public ModelStateDictionary ModelStateErrors { get; set; }

và trong phần hướng dẫn hành động POST của bạn, bạn có thể viết mã trực tiếp như

model.ModelStateErrors = ModelState; 

và sau đó gán mô hình này cho Tempdata như bên dưới

TempData["Model"] = model;

và khi bạn chuyển hướng đến phương thức hành động của bộ điều khiển khác thì trong bộ điều khiển, bạn phải đọc giá trị 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);
    }
}

Đó là nó. Bạn không cần phải viết các bộ lọc hành động cho việc này. Điều này đơn giản như đoạn mã trên nếu bạn muốn nhận lỗi trạng thái Mô hình sang một chế độ xem khác của bộ điều khiển khác.