不使用 ServiceLocator 进行验证

Question

我一次又一次地思考对需要访问某些上下文的 POCO 对象执行验证的最佳方法（例如 NH 中的 ISession，IRepository）。

我仍然可以看到的唯一选项是使用服务定位器，所以我的验证看起来像：

public User : ICanValidate {
    public User() {} // We need this constructor (so no context known)

    public virtual string Username { get; set; }

    public IEnumerable<ValidationError> Validate() {
        if (ServiceLocator.GetService<IUserRepository>().FindUserByUsername(Username) != null)
            yield return new ValidationError("Username", "User already exists.")
    }
}

我已经使用控制反转和依赖注入并且真的不喜欢ServiceLocator 由于以下事实：

• 更难维护隐式依赖关系。
• 测试代码更困难。
• 潜在的线程问题。
• 仅显式依赖于 ServiceLocator。
• 代码变得更难理解。
• 测试时需要注册ServiceLocator接口。

但另一方面，对于普通的 POCO 对象，我没有看到任何其他方法可以像上面那样在没有 ServiceLocator 且仅使用 IoC/DI 的情况下执行验证。

目前我在服务层执行此类验证。因此，每当参与者尝试更改用户名（当然可能是不同的用户名）时，服务都会执行此验证。一个明显的缺点是每个使用 User 的服务都必须执行此检查（即使是一次调用）。

所以问题是：有没有办法在上述情况下使用 DI/IoC？

谢谢，
德米特里。

Answer 1

存储库通常处于比它们获取/存储的域对象更高的抽象级别。如果您发现域对象依赖于存储库，则表明上游存在设计问题。

你实际上拥有的是循环依赖。 IUserRepository 依赖于 User，User 依赖于 IUserRepository。从技术上讲，这是可行的，因为如果两个对象位于同一个程序集中，它就会编译，但作为一般设计，它会给您带来麻烦。可能有各种各样的对象想要处理User，但对它来自的IUserRepository一无所知。

我对您的建议是不要将其作为 User 的“验证”属性。验证应该由存储库本身执行，或者更好的是，如果存储库在尝试保存时用户名已经存在，则只需让存储库引发异常。

这个建议还有一个次要原因。这个原因就是并发性。即使您验证用户名并发现它不已经存在，1秒后当您尝试保存该用户时也可能不存在。因此，无论如何，您都需要处理异常情况（尝试插入已经存在的用户名）。鉴于此，您不妨推迟到最后一刻，因为您无法事先做出保证。

域对象应该没有依赖关系；如果它们自我验证，那么验证应该仅取决于正在验证的实际对象，而不是数据库中的其他数据。重复用户名约束实际上是数据约束，而不是域约束。

摘要：将此特定验证移到 User 类之外。它不属于那里；这就是为什么您发现自己使用这种特定的反模式。

Answer 2

只是补充一下阿罗诺特所说的内容。这种设计存在一个更大的问题，因为领域模型验证应该只验证模型固有的属性，而不是在更大系统的上下文中。此类固有属性的一些示例包括用户名长度、可接受的字符、名字和姓氏均已归档等要求。

您正在执行的验证是系统范围的验证，属于服务/存储库。如果使用领域驱动设计来设计，该系统将如下所示：

public class User : ICanValidate {
    public User() {} 

    public virtual string Username { get; set; }

    public IEnumerable<ValidationError> Validate() {
        if (!string.IsNullOrEmpty(this.UserName))
          yield return new ValidationError("Username must not be empty");
    }
}

public class UserRepository : IUserRepository {
}

public static class UserService { 
  readonly IUserRepository Repository;

  static UserService() {
    this.Repository = ServiceLocator.GetService<IUserRepository>();
  }

  public static IEnumerable<ValidationError> Validate(User user) {
      if (Repository.FindUserByUsername(user.Username) != null)
          yield return new ValidationError("Username", "User already exists.")
  }
}