ASP.NET MVC - Как сохранить ошибки ModelState через RedirectToAction? - PullRequest
Новая
       23

ASP.NET MVC - Как сохранить ошибки ModelState через RedirectToAction?

78 голосов
/ 10 января 2011

У меня есть два следующих метода действий (упрощенный для вопроса): 

[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 (как указано выше), она теряет ошибки модели.

Меня не удивляет проблема, просто интересно, как вы, ребята, справляетесь с этим?

Конечно, я мог бы просто вернуть тот же самый вид вместо перенаправления, но у меня есть логика в "Создать"метод, который заполняет данные представления, который я должен был бы дублировать.

Есть предложения?

Ответы [ 9 ]

71 голосов
/ 19 августа 2012

Мне пришлось сегодня решить эту проблему самому, и наткнулся на этот вопрос.

Некоторые ответы полезны (с использованием TempData), но на самом деле не отвечают на данный вопрос.

Лучший совет, который я нашел, был в этом сообщении в блоге:

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

В основном используйте TempData для сохранения и восстановления объекта ModelState. Однако, если вы абстрагируете это в атрибуты, будет намного чище.

1012 * Е.Г. * 

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), вы все равно можете это сделать.

45 голосов
/ 10 января 2011

Вам необходимо иметь тот же экземпляр 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"].

7 голосов
/ 10 января 2011

Почему бы не создать приватную функцию с логикой в ​​методе "Создать" и вызвать этот метод как из метода Get, так и из метода Post, а просто вернуть View ().

4 голосов
/ 10 января 2011

Я предлагаю вам вернуть представление и избегать дублирования через атрибут в действии.Вот пример заполнения для просмотра данных.Вы можете сделать что-то подобное с вашей логикой метода создания. 

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);
}
4 голосов
/ 10 января 2011

Я мог бы использовать TempData["Errors"]

TempData передается через действия, сохраняя данные 1 раз.

2 голосов
/ 10 января 2011

У меня есть метод, который добавляет состояние модели к временным данным. Затем в моем базовом контроллере есть метод, который проверяет временные данные на наличие ошибок. Если он есть, он добавляет их обратно в ModelState.

0 голосов
/ 07 сентября 2018

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");
    }
0 голосов
/ 23 декабря 2017

Я предпочитаю добавить метод к моей 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 голосов
/ 10 декабря 2015

Мой сценарий немного сложнее, так как я использую шаблон PRG, поэтому моя ViewModel ("SummaryVM") находится в TempData, и мой экран Summary отображает его.На этой странице есть небольшая форма для размещения некоторой информации в другом действии.Сложность возникла из-за того, что пользователь должен отредактировать некоторые поля в SummaryVM на этой странице.

Summary.cshtml содержит сводку проверки, которая будет перехватывать ошибки ModelState, которые мы создадим. 

@Html.ValidationSummary()

Моя форма теперь должна POST к действию HttpPost для Summary ().У меня есть еще одна очень маленькая ViewModel для представления отредактированных полей, и привязка к модели получит их мне.

Новая форма: 

@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");

Действие контроллера 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);
            }
        }
...