Model和ViewModel的验证最佳实践

时间:2011-06-24 14:51:05

标签: c# asp.net-mvc-2 model viewmodel asp.net-mvc-2-validation

我有单独的模型和视图模型类。 viewmodel类只进行UI级别验证(参见:Validation: Model or ViewModel)。

我可以在控制器中验证模型(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类映射到业务实体时验证它。

有什么建议吗?

2 个答案:

答案 0 :(得分:10)

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

答案 1 :(得分:3)

我认为最好使用数据注释 看看这个

ASP.NET MVC Valdation

对于服务器端验证,您可以使用fluent validation

并查看此question