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

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

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

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

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

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