Comment dois-je entretenir ModelState erreurs lors de l'utilisation de RedirectToAction?

J'ai un code qui permet d'économiser un billet dans notre système. Si il y a une erreur, il fait un RedirectToAction. Le problème est que je ne semble pas avoir mes erreurs dans la nouvelle action. Comment puis-je résoudre ce problème? 

 ModelState.AddModelError("_FORM", "Unable to save ticket");
 ModelState.AddModelError("_FORM", "Phone number was invalid.");
 ModelState.AddModelError("_FORM", "Lane number is required.");
 return RedirectToAction("CreateStep", "Ticket");

Je sais que certains ont proposé d'utiliser TempData, mais comment aurais-je obtenir chaque erreur de la ModelState?

Grâce.

InformationsquelleAutor Mike Roosa | 2009-03-18
  1. 41

    Le PRG modèle est ok, mais je l'ai fait:

    Contrôleur de Base:

    protected override void OnActionExecuted(ActionExecutedContext filterContext)
{
    if (TempData["ModelState"] != null && !ModelState.Equals(TempData["ModelState"]))
        ModelState.Merge((ModelStateDictionary)TempData["ModelState"]);

    base.OnActionExecuted(filterContext);
}

    Action (je suis en utilisant xVal):

    try
{
    user.Login();
    AuthenticationManager.SignIn(user);
}
catch (RulesException rex)
{
    //on bad login
    rex.AddModelStateErrors(ModelState, "user");
    TempData["ModelState"] = ModelState;
    return Redirect(Request.UrlReferrer.ToString());
}

    L'action déclenche une exception, ajoute le ModelState à TempData et la renvoie au référent. Étant donné que l'action est pris, OnActionExecuted est toujours exécuté, mais la première fois autour de la ModelState est le même que TempData["ModelState"], de sorte que vous ne voulez pas fusionner avec vous-même. Lors de la redirection d'action est exécutée, OnActionExecuted déclenche à nouveau. Cette fois, si il n'y a rien dans TempData["ModelState"], il se confond avec le présent de l'action ModelState.

    Vous pourrait l'étendre à plusieurs modèles en utilisant TempData["ModelState.utilisateur"] = ModelState et puis la fusion de tous les TempData objet qui commence par ModelState.

    • J'aime cette... Semble un moyen pratique pour faire ce travail. Merci!!!!
    • Je l'aime. J'ai étendu votre code pour mon utilisation. Au lieu de votre RulesException.AddModelStateErrors(), je l'ai fait dans l'autre sens, j'ai étendu ModelState d'avoir deux méthodes d'extension; UpdateModel et RestoreModel. +1 pour vous.
    InformationsquelleAutor
  2. 29

    Je sais que ce fil est vieux, mais ce blog ASP.NET les Meilleures Pratiques a quelques excellentes suggestions.

    #13 sur la page traite à l'aide de 2 filtres d'actions de sauvegarder et de restaurer ModelState entre les redirections.

    C'est le modèle que je propose, et je l'aime.

    Voici l'exemple simplifié:

    [ImportModelStateFromTempData]
public ActionResult Dashboard()
{
    return View();
}

[AcceptVerbs(HttpVerbs.Post), ExportModelStateToTempData]
public ActionResult Dashboard(string data)
{
    if (ValidateData(data))
    {
        try
        {
            _service.Submit(data);
        }
        catch (Exception e)
        {
            ModelState.AddModelError(ModelStateException, e);
        }
    }

    return RedirectToAction("Dashboard");
}
    • L'âge de la question ou les réponses vraiment n'a pas d'importance - même ici, en 2014, ils peuvent aider 🙂 Et merci pour le lien, vraiment du bon matos!
    • Le [ImportModelStateFromTempData] et [ExportModelStateToTempData] attributs sont également expliqué ici: Comment faire pour enregistrer le ModelState dans la session suivant les bonnes pratiques
    • Votre lien (qui est omniprésent à chaque discussion de PRG dans MVC, j'ai vu) est bizarrement morts. Il ressemble à Rashid blog a été retiré de la ASP.NET blog de l'équipe.
    • Le lien est cassé 🙁
    • Trouvé (sans doute) la (même) lien asp.net MVC Meilleures Pratiques (Partie 1) - voir n ° 13. Utilisation PRG Motif de la Modification des Données
    InformationsquelleAutor Scott Rippey
  3. 4

    ce blog explique comment vous pourriez mettre en œuvre le PRG-Pattern MVC
    http://blog.simonlovely.com/archive/2008/11/26/post-redirect-get-pattern-in-mvc.aspx

    hth

    • Ce lien est rompu
    InformationsquelleAutor marc.d
  4. 3

    Utiliser le TempData[] Collection

    La tempdata est stockée à partir de l'un demande à l'autre, alors son parti.

    InformationsquelleAutor AndreasN
  5. 0

    Ce que j'ai fait pour maintenir mon ModelState peu importe où je vais avec des redirections est la suivante:

    1. Dans votre modèle, ajouter:

      public ModelStateDictionary modelstate { get; set; }

    2. Dans votre modèle du constructeur, ajouter:

      this.modelstate = new System.Web.Mvc.ModelStateDictionary();

    3. Échantillon Post avec mon modèle, Modèles.ContactInformation:

      [HttpPost]
[ValidateAntiForgeryToken]
public ActionResult contact(Models.ContactInformation con)
{
  if (string.IsNullOrEmpty(con.SelectedAgencySelectorType))
  {
    ModelState.AddModelError("", "You did not select an agency type.");
  }

  con.modelstate = ModelState;
  TempData["contact"] = con;
  if (!ModelState.IsValid) return RedirectToAction("contactinformation", "reports");

    //do stuff

    return RedirectToAction("contactinformation", "reports");
}

    4. Alors maintenant, vos tempdata a votre modèle et modelstate comme est.

    5. Ce qui suit est mon point de vue, qui est agnostique à l'état de quoi que ce soit, sauf s'il a quelque chose. Voici le code:

      [HttpGet]
public ActionResult contactinformation()
{
    //try cast to model
    var m = new Models.ContactInformation();
    if (TempData["contact"] is Models.ContactInformation) m = (Models.ContactInformation)TempData["contact"];

    //restore modelstate if needed
    if (!m.modelstate.IsValid)
    {
        foreach (ModelState item in m.modelstate.Values)
        {
            foreach (ModelError err in item.Errors)
            {
                ModelState.AddModelError("", err.ErrorMessage.ToString());
            }
        }
    }

    return View(m);
}
    InformationsquelleAutor Rudy Hinojosa