验证域对象设置器中的表单输入？

Question

自从我开始学习 MVC 以来，我一直在我的控制器中验证我的表单数据，这是我在浏览 CodeIgniters 代码时养成的习惯，但我了解到它执行某些操作的方式并不是最好的，它只是完成工作。

所有表单数据都应该由域对象验证吗？如果是这样，它应该像这样在二传手中完成

public function setFirstName($firstName) {

    // Check if the field was required
    if(!$firstName) {
        throw new InvalidArgumentException('The "First name" field is required');
    }

    // Check the length of the data
    // Check the format 
    // Etc etc
}

另外，例如，如果我正在处理基本用户注册，我的 User 类没有 $confirmPassword 属性，所以我不会这样做

$user->setConfirmPassword($confirmPassword);。

检查输入的两个密码是否相等的一种方法是设置$password 并执行类似的操作

$user->setPassword($password);
if(!$user->matchPassword($confirmPassword)) {
    throw new PasswordsNotEqualException('Some message');
}

我认为这将在服务层中完成？

任何可以帮助我朝着正确方向前进的建议都会很棒。谢谢。

Answer 1

所有表单数据都应该由域对象验证吗？如果是这样 是否应该像这样在设置器中完成

IMO 您应该只允许创建有效对象，而实现这一点的最佳方法是在创建对象的方法中进行这些检查。

假设用户的名字不能更改，您将在创建用户时验证它。这样，您就可以忘记 setter，因为您不再需要它了。

在某些情况下，您可能希望更改属性，并且您也需要验证它们（因为如果是这种情况，更改可能会导致从有效对象变为无效对象）。

检查输入的两个密码是否相等的一种方法是 设置 $password 并执行类似...

你可以用同样的方式处理这个问题：有一个 Password 对象，它在创建时检查密码和确认。如果您有一个有效的 Password 实例，则可以在知道它通过了您指定的所有验证后使用它。

参考文献

这些设计原则（从一开始就完整和有效的对象等）来自 Hernan Wilkinson 的“Patagonia 背后的设计原则”。请务必查看ESUG 2010 Video 和presentation slides。

我最近回答了另一个关于验证属性的问题，我认为您可能会派上用场：https://stackoverflow.com/a/14867390/146124

干杯！

Answer 2

TL;DR

不，setter 不应该验证数据。而 nick2083 是完全错误的。

加长版...

根据 Tim Howard 提供的定义 ^[source]，domain objects 可以验证它们包含的域信息的状态。这基本上说明，要让您真正拥有一个域对象，该对象需要能够验证自己。

何时验证

你基本上必须选择：

• 在每个 setter 中验证
• 有一种方法可以验证整个对象

如果验证是 setter 的一部分，则存在一个主要缺点：setter 的顺序很重要。

_{示例：假设您正在制作一个涉及人寿保险的应用程序。很有可能，当触发保单时（被保险人死亡），您将拥有一个域对象，其中包含被保险人和获得保费的人。您必须确保收件人和被保险人不是同一个人。但是没有规则可以控制执行 setter 的顺序。}

当您在域对象中有两个或多个参数时，必须相互验证，实现变得有点模糊。最可行的解决方案是检查何时分配了所有参数，但此时您已经失去了 in-setter 验证的好处：代码的执行已经移过了无效数据的来源。

如果参数 B 很大，那么 21 并且 C 已经设置，那么域对象的有效状态没有设置参数 A，您将如何处理？

_{结论： setter 中的验证只是可行的解决方案，当您拥有非常简单的域对象且没有纠结的验证规则时。}