对我的域对象或视图模型进行验证？它应该去哪里？

Question

我试图让我的服务层摆脱 ASP.NET MVC 依赖项，但我遇到了问题。我想使用 Compare，但它是 asp.net mvc 库的一部分。那我该怎么办呢？

我有一个域类（稍后与 Fluent nhibernate 一起使用）

public class User()
{
   [Required(ErrorMessage = "Required")]
   [Compare()] // can't do this because not my User domain needs to know about mvc and I   rather it not
   public virtual string Email (get; set;}
   public virtual string ConfirmEmail (get; set;} // not mapped with fluent nhibernate
}

public class UserViewModel()
{
   public User User {get; set;}
}


public ActionMethod method()
{
  if(!this.ModelState.isValid)
  {

   }
}

我制作了这个域对象，所以我发现将所有这些属性移动到 UsersViewAModel 中只是为了对它们进行验证，然后将所有这些属性返回到中是毫无意义的将用于提交的域模型（nhibernate commit）

Answer 1

您可能应该为您的模型类使用“伙伴类”来保存验证属性：

// Note the partial keyword below:  the full definition of MyModel 
// can be put in a file in the model
[MetadataType(typeof(MyModelMetadata))]
public partial class MyModel {}

public class MyModelMetadata
{
    //properties with validation attributes (dataannotations)  go here...
}

验证属性是一项不能在 MVC 关注点之间清晰划分的项目：
它们与视图中的客户端验证相关，与域中的服务器验证相关，在数据存储到数据库并因此需要验证时。

因此，@jfar 说它们应该进入 ViewModel，这与说它们应该进入域的人一样错误。

事实上，恕我直言，将验证属性放在 ViewModel 中是完全不正确的。 ViewModel 应该是发送到视图的数据的容器。它不应该移植自己的任何逻辑。该逻辑（在本例中为验证属性）应在 ViewModel 外部定义，而伙伴类是最佳选择，特别是在您使用自动化 ORM 时，但不仅限于这种特殊情况。

这个问题说明了这样一个事实：并非每个问题都完全映射到 MVC，因此对验证属性应该去哪里的问题采取所有斯大林主义的态度是完全错误的。

我的建议：将它们放在一个地方，与所有其他问题分开。做到这一点的方法是使用伙伴类。

Answer 2

如果您试图分离关注点，那么将验证属性放在域模型上是最糟糕的事情。

将验证属性放在 ViewModel 上。

Answer 3

public class User()
    {
       public virtual string Email (get; set;}
       public virtual string ConfirmEmail (get; set;} // not mapped with fluent nhibernate
    }
public class UserModel()
{
   [Required(ErrorMessage = "Required")]
   [Compare()] // can't do this because not my User domain needs to know about mvc and I   rather it not
   public virtual string Email (get; set;}
   public virtual string ConfirmEmail (get; set;} // not mapped with fluent nhibernate
}

public class UserViewModel()
{
   public UserModel User {get; set;}
}


public ActionMethod method()
{
  if(!this.ModelState.isValid)
  {

   }
}

您可能想使用 AutoMapper 来解决从 User 填充 UserModel 的问题反之亦然。