Рекомендации по валидации для Model и ViewModel

Question

У меня есть отдельные классы модели и модели представления.В тех случаях, когда классы viewmodel выполняют только проверку уровня пользовательского интерфейса (см .: Validation: Model или ViewModel ).

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

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

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

//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 с бизнес-объектом.

Есть предложения?

Answer 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]+@((\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]+@((\w+\-+)|(\w+\.))*\w{1,63}\.[a-zA-Z]{2,6}$");
        }

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

   [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 = "test@test.com";
            Assert.AreEqual("test@test.com", 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 = "test@test.com" };
            x.Email = "test@test.com";
            Assert.AreEqual("test@test.com", x.Email);
        }
    }

Answer 2

Я думаю, что лучше использовать аннотации данных, посмотрите на

Проверка ASP.NET MVC

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

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