Валидация лучшей практики для модели и ViewModel

У меня есть отдельные модели и классы viewmodel. Если классы viewmodel проверяют только уровень UI (см. Validation: Model или ViewModel).

Я могу проверить действие post в контроллере, что модель (vewmodel) действительна.

Спросите: Как проверить модель (основной объект с аннотациями данных).

Я не разрабатываю viewmodel с использованием объекта модели. Просто дублируйте свойства и добавьте все свойства, которые могут потребоваться в этом конкретном представлении.

//Model Class
public class User
{
    [Required]
    public string Email {get; set;}

    [Required]
    public DateTime Created {get; set;}
}

//ViewModel Class
public class UserViewModel
{
    [Required]
    public string Email {get; set;}

    [Required]
    public string LivesIn {get; set;}
}

//Post action
public ActionResult(UserViewModel uvm)
{
    if( ModelState.IsValid)
        //means user entered data correctly and is validated

    User u = new User() {Email = uvm.Email, Created = DateTime.Now};
    //How do I validate "u"?

    return View();
}

Должно сделать что-то вроде этого:

var results = new List<ValidationResult>();
var context = new ValidationContext(u, null, null);
var r = Validator.TryValidateObject(u, context, results);

То, что я думаю, заключается в добавлении этой техники валидации в базовый класс (бизнес-объекта) и ее проверки при сопоставлении из класса viewmodel в бизнес-объект.

Любые предложения?

Ответы

Ответ 1

1) Используйте свободную проверку модели, которая извлекает информацию от пользователя. он более гибкий, чем аннотация данных и проще тестировать.

2) Возможно, вы захотите изучить automapper, используя automapper, вам не нужно писать x.name = y.name.

3) Для вашей модели базы данных я буду придерживаться аннотаций данных.

Все ниже описано на основе новой информации

Во-первых, и все, что вы должны сделать валидацией в обоих местах, как вы сделали сейчас, для фактической проверки модели, вот как я это сделаю. Отказ от ответственности: это не идеальный способ

Сначала и все обновите UserViewModel до

public class UserViewModel
    {
        [Required()]
        [RegularExpression(@"^(([A-Za-z0-9]+_+)|([A-Za-z0-9]+\-+)|([A-Za-z0-9]+\.+)|([A-Za-z0-9]+\++))*[A-Za-z0-9][email protected]((\w+\-+)|(\w+\.))*\w{1,63}\.[a-zA-Z]{2,6}$")]
        public String Email { get; set; }
    }

Затем обновите метод действия до

        // Post action
        [HttpPost]
        public ActionResult register (UserViewModel uvm)
        {
            // This validates the UserViewModel
            if (ModelState.IsValid)
            {

                try
                {
                    // You should delegate this task to a service but to keep it simple we do it here
                    User u = new User() { Email = uvm.Email, Created = DateTime.Now };
                    RedirectToAction("Index"); // On success you go to other page right?
                }
                catch (Exception x)
                {
                    ModelState.AddModelError("RegistrationError", x); // Replace x with your error message
                }

            }       

            // Return your UserViewModel to the view if something happened               
            return View(uvm);
        }

Теперь для модели пользователя становится сложно, и у вас есть много возможных решений. Решение, которое я придумал (возможно, не самый лучший), следующее:

public class User
    {
        private string email;
        private DateTime created;

        public string Email
        {
            get
            {
                return email;
            }
            set
            {
                email = ValidateEmail(value);
            }
        }

        private string ValidateEmail(string value)
        {
            if (!validEmail(value))
                throw new NotSupportedException("Not a valid email address");     

            return value;
        }

        private bool validEmail(string value)
        {
            return Regex.IsMatch(value, @"^(([A-Za-z0-9]+_+)|([A-Za-z0-9]+\-+)|([A-Za-z0-9]+\.+)|([A-Za-z0-9]+\++))*[A-Za-z0-9][email protected]((\w+\-+)|(\w+\.))*\w{1,63}\.[a-zA-Z]{2,6}$");
        }

Последние несколько unit test, чтобы проверить мой собственный код:

   [TestClass()]
    public class UserTest
    {

        /// <summary>
        /// If the email is valid it is stored in the private container
        /// </summary>
        [TestMethod()]
        public void UserEmailGetsValidated()
        {
            User x = new User();
            x.Email = "[email protected]";
            Assert.AreEqual("[email protected]", x.Email);
        }

        /// <summary>
        /// If the email is invalid it is not stored and an error is thrown in this application
        /// </summary>
        [TestMethod()]
        [ExpectedException(typeof(NotSupportedException))]
        public void UserEmailPropertyThrowsErrorWhenInvalidEmail()    
       {
           User x = new User();
           x.Email = "blah blah blah";
           Assert.AreNotEqual("blah blah blah", x.Email);
       }


        /// <summary>
        /// Clears an assumption that on object creation the email is validated when its set
        /// </summary>
        [TestMethod()]
        public void UserGetsValidatedOnConstructionOfObject()
        {
            User x = new User() { Email = "[email protected]" };
            x.Email = "[email protected]";
            Assert.AreEqual("[email protected]", x.Email);
        }
    }

Ответ 2

Я думаю, лучше использовать Data Annotations посмотрите на это

ASP.NET MVC Valdation

и для проверки на стороне сервера вы можете использовать плавную проверку

и посмотрите на это question