Wie behalte ich ModelState-Fehler bei der Verwendung von RedirectToAction?

Ich habe einige code, der spart ein ticket in unserem system. Wenn es einen Fehler macht RedirectToAction. Das problem ist, dass ich nicht scheinen, um meine Fehler in der neuen Aktion. Wie kann ich dieses Problem beheben? 

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

Ich weiß, einige haben vorgeschlagen, TempData, aber wie bekomme ich die einzelnen Fehler aus der ModelState?

Dank.

InformationsquelleAutor der Frage Mike Roosa | 2009-03-18

  1. 41

    PRG pattern ist ok, aber ich habe dieses:

    Basis-controller:

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

    base.OnActionExecuted(filterContext);
}

    Aktion (ich bin mit 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());
}

    Die Aktion eine Ausnahme wirft, ergänzt der ModelState zu TempData und leitet zurück auf die referrer. Da die Aktion gefangen ist, OnActionExecuted wird weiterhin ausgeführt, aber das erste mal rund um den ModelState ist das gleiche wie TempData["ModelState"], so dass Sie nicht wollen, zu verschmelzen mit sich selbst. Wenn der redirect action ausgeführt wird, OnActionExecuted feuert wieder. Dieses mal, wenn es etwas gibt, in TempData["ModelState"], es verschmilzt mit dieser Aktion ModelState.

    Könnte man es erweitern auf mehrere Modelle durch die Verwendung von TempData["ModelState.user"] = ModelState und dann verschmelzen alle TempData-Objekt, das beginnt mit ModelState.

    InformationsquelleAutor der Antwort

  2. 28

    Ich weiß, dieser thread ist alt, aber dieses blog über ASP.NET Best Practices hat einige gute Vorschläge.

    #13 auf der Seite befasst sich mit der Verwendung von 2-Action-Filter zum speichern und wiederherstellen von ModelState zwischen umleitet.

    Dies ist das Muster, dass meine Arbeit verwendet, und ich Liebe es.

    Hier das vereinfachte Beispiel:

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

    InformationsquelleAutor der Antwort Scott Rippey

  3. 4

    dieser blog-post beschreibt, wie man sich die Umsetzung der PRG-Pattern im MVC
    http://blog.simonlovely.com/archive/2008/11/26/post-redirect-get-pattern-in-mvc.aspx

    hth

    InformationsquelleAutor der Antwort marc.d

  4. 3

    Verwenden Sie den TempData[] Sammlung

    Den tempdata gespeichert ist, von einer Anforderung zur nächsten, dann ist es vorbei.

    InformationsquelleAutor der Antwort AndreasN

  5. 0

    Was ich habe zu halten mein ModelState, egal, wo ich mit leitet ist die folgende:

    1. In Ihrem Modell hinzufügen:

      public ModelStateDictionary modelstate { get; set; }

    2. In Ihrem Modell Konstruktor hinzufügen:

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

    3. Beispiel-Post mit meinem Modell, das sogenannte Modelle.Kontaktdaten:

      [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. So, jetzt Ihr tempdata hat Ihr Modell und modelstate.

    5. Das folgende ist meine Ansicht, die nicht wissen, der Zustand des nichts, es sei denn, es hat etwas. Hier ist der 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 der Antwort Rudy Hinojosa

  6. -3

    Ich glaube, Sie verlieren Ihre Modell-Zustand, wenn Sie tun eine Umleitung. Vielleicht könnten Sie schreiben Sie Ihre Logik, um so etwas wie:

    public ActionResult Save()
{
  //your code...
  if(saveSucceeded)
  {
    return View("Saved");
  }
  else
  {
    return View("Create");
  }
}

    Und der übliche Weg, um Ihre Fehlermeldung:

    <%= Html.ValidationMessage("property_name") %>

    InformationsquelleAutor der Antwort Thomas Eyde

Schreibe einen Kommentar