4

自从我开始学习 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');
}

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

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

4

2 回答 2

3

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

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

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

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

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

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

参考

这些设计原则（从一开始就完整和有效的对象等）来自 Hernan Wilkinson 的“Patagonia 背后的设计原则”。请务必查看ESUG 2010 视频和演示幻灯片。

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

干杯!

于 2013-02-23T14:24:44.907 回答
3

TL;博士

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

加长版...

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

何时验证

你基本上必须选择：

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

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

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

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

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

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

于 2013-02-23T17:40:55.893 回答