DataAnnotations [Obligatoire] attribut ne provoque pas d'exception à jeter

Question

J'ai une application MVC Asp.NET dans lequel j'utilise des annotations de données pour ajouter la validation à certains champs:

[Required] 
    [DisplayName("Course Name")] 
    string Name { get; set; } 

Toutefois, cela ne semble pas travailler comme je m'y attendais. Fondamentalement Si la page contient d'autres erreurs que je vérifie manuellement et lancez un nouveau RuleViolation(), alors la violation requise est montrée dans le résumé de validation. Si la violation requise est la seule erreur, elle n'est pas affichée.

Mon contrôleur a ce code dans le:

 catch (Exception ex) 
     { 
      ModelState.AddModelError("", ex.Message); 
      ModelState.AddRuleViolations(courseViewModel.Course.GetRuleViolations()); 
      return View(courseViewModel); 
     } 

Mais étant donné que la violation requise est de ne pas jeter, je ne vais jamais ici. Dois-je faire quelque chose que je ne connais pas pour intercepter les erreurs provoquées par la violation de DataAnnotation?

Merci pour toute aide

Edit:

Voici l'action du contrôleur:

[HttpPost] 
    [ValidateInput(true)] 
    public ActionResult Edit(int id, CourseViewModel courseViewModel) 
    { 

     var oldCourse = _eCaddyRepository.GetCourse(id); 

     if (courseViewModel.Course == null) 
     { 
      return View("NotFound", string.Format("Course {0} Not Found", id)); 
     } 

     try 
     { 
      courseViewModel.Update(oldCourse); 
      _eCaddyRepository.SubmitChanges(); 

      return RedirectToAction("Index", "Course"); 
     } 
     catch (Exception ex) 
     { 
      ModelState.AddModelError("", ex.Message); 
      ModelState.AddRuleViolations(courseViewModel.Course.GetRuleViolations()); 
      return View(courseViewModel); 
     } 
    } 

Où mise à jour est:

public class CourseViewModel : BaseViewModel 
{ 
    public Course Course { get; set; } 

    public void Update(Course oldCourse) 
    { 
     oldCourse.Name = this.Course.Name != null ? this.Course.Name.Trim() : string.Empty; 
     oldCourse.Postcode = this.Course.Postcode != null ? this.Course.Postcode.Trim() : string.Empty; 

     for (var i = 0; i < 18; i++) 
     { 
      oldCourse.Holes[i].Par = this.Course.Holes[i].Par; 
      oldCourse.Holes[i].StrokeIndex = this.Course.Holes[i].StrokeIndex; 
     } 
    } 
} 

EDIT: Code final qui fonctionne et valide comme prévu en utilisant des notifications de données. Merci à mare.

[HttpPost] 
    [ValidateInput(true)] 
    public ActionResult Edit(int id, CourseViewModel courseViewModel) 
    { 
     var oldCourse = _eCaddyRepository.GetCourse(id); 

     if (courseViewModel.Course == null) 
     { 
      return View("NotFound", string.Format("Course {0} Not Found", id)); 
     } 

     if (ModelState.IsValid) 
     { 
      try 
      { 
       courseViewModel.Update(oldCourse); 
       _eCaddyRepository.SubmitChanges(); 
       return RedirectToAction("Index", "Course"); 
      } 
      catch (Exception ex) 
      { 
       ModelState.AddModelError("", ex.Message); 
      } 
     } 

     // Return Model with errors 
     ModelState.AddRuleViolations(courseViewModel.Course.GetRuleViolations()); 
     return View(courseViewModel); 
    } 

Answer 1

Je me demande comment personne ne fait autre ceci (jfar était proche, mais son libellé était de si chrisp_68 probablement ne comprenaient pas ce jfar voulu dire des violations d'état du modèle), mais il vous manque ce de votre contrôleur:

 if (ModelState.IsValid) // this check for model validity, not try..catch block 
     { 
      // do your stuff here, update to the datastore and return another view 
     } 

     // you can add additional Model State errors here manually if you wish 
     // with AddModelError() like you do it now 
     return View(editing); // return the same view with errors 

il est donc ModelState.IsValid vérifier que vous avez besoin, car DataAnnotations n'effectue pas lancer d'exception par eux-mêmes. EDIT: En fait, pour être précis, DataAnnotations ne fait pas de lancement d'exception car cela aurait un sens, car cela briserait l'exécution de votre application, ce que bien sûr vous ne voulez pas. Vous voulez revenir à la même vue et donner à l'utilisateur une chance de corriger ses erreurs.

D'autre part, vous pouvez toujours avoir try..catch bloc à l'intérieur du si (ModelState.IsValid) pour attraper le REAL exception comme défaut d'écrire sur le disque ou de ne pas stocker la base de données ou d'insérer dans nulls Colonnes DB où aucun null n'est autorisé, etc.

HTH

Answer 2

Il n'y a rien dans MVC2 qui déclencherait une exception en raison d'un champ [Obligatoire].

Vous obtenez des violations d'état du modèle, c'est tout. Assez sûr que les exceptions sont seulement lancées à partir d'un appel manuel à ModelBinder lorsque vous essayez de lier "Joel Atwood" à un champ DateTime.

Answer 3

Sur la propriété de la classe il y a u mettre l'annotation de données, espace de noms doit être « YouProject.Model »