DataAnnotations [Требуется] Атрибут не вызывает исключений быть выброшен

Question

У меня есть приложение Asp.NET MVC, в котором я использую аннотации данных для добавления проверки в некоторых областях:

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

Однако это не кажется работать так, как я ожидал. В принципе Если на странице есть какие-либо другие ошибки, которые я вручную проверяю и выкидываю новое RuleViolation(), то требуемое нарушение показано в резюме проверки. Если требуемое нарушение является единственной ошибкой, то оно не отображается.

Мой контроллер имеет этот код в нем:

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

Но учитывая, что требуемое нарушение не бросать, я никогда не хожу сюда.

Нужно ли мне что-то делать, что я не знаю об ошибках, вызванных нарушением DataAnnotation?

Спасибо за любую помощь

Edit:

Вот действие контроллера:

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

Где Обновление:

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: Окончательный код, который работает и проверяет как и ожидалось, с помощью dataannotations. Благодаря кобыле.

[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

Интересно, как никто не обратил на это (jfar был близок, но его формулировка была от так chrisp_68 вероятно, не понял, что jfar имел в виду модель государственных нарушений), но вам не хватает этого из контроллера:

 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 

Так что ModelState.IsValid проверить, что вам нужно, потому что DataAnnotations не будет выполнять никаких исключений метания сами по себе.

РЕДАКТИРОВАТЬ: На самом деле, DataAnnotations не делают исключение, потому что это было бы глупо, если бы это произошло, потому что это нарушило бы выполнение вашего приложения, которое, конечно, вы не хотите. Вы хотите вернуться к одному и тому же представлению и дать пользователю возможность исправить свои ошибки.

С другой стороны, вы все равно можете иметь try..catch блок внутри, если (ModelState.IsValid), чтобы поймать РЕАЛЬНОГО исключение как отказ писать на диск или не хранить в базе данных или вставки аннулирует в Столбцы БД, где не допускаются никакие нули, и т. Д.

HTH

Answer 2

В MVC2 ничего не было, что вызвало бы исключение из-за поля [Обязательное].

Вы получаете типовые нарушения состояния, вот и все. Довольно-таки исключения исключаются только из ручного вызова ModelBinder, когда вы пытаетесь связать «Joel Atwood» с полем DateTime.

Answer 3

На класса собственности есть у положить аннотацию данных, пространство имен должно быть «YouProject.Model»