验证实体中的值 - 通过设置器或断言验证？答案

【问题标题】：Validating values in entity - by setter or assert validation?验证实体中的值 - 通过设置器或断言验证？
【发布时间】：2013-10-20 00:50:16
【问题描述】：

我对验证实体中的值的 OOP 策略有疑问。假设我有这样的实体：

/*
 * @ORM\Table()
 * @ORM\Entity(repositoryClass="My\PageBundle\Entity\PageRepository")
 */
class File {
    /*
     * @ORM\Column(name="id", type="integer")
     * @ORM\Id
     * @ORM\GeneratedValue(strategy="AUTO")
     */
    private $id;

    /*
     * @ORM\Column(name="type", type="string")
     */
    private $type;

    /*
     * @ORM\ManyToOne(targetEntity="File")
     * @ORM\JoinColumn(name="parent_id", referencedColumnName="id")
     */
    private $parent;
}

现在我只想让实体类型“父母”可以是父母。我可以通过两种方式做到这一点。

使用Symfony Callback validation：

if ($this->getParent() != null && $this->getParent()->getType() != 'group') $context->addViolationAt('parent', '无效的父级。', array(), null);

这很明显，但只有在我调用验证器时才有效，或者

将此逻辑放入 setter 中，就像在 Symfony book 中提出的那样

您应该检查生成的实体并根据自己的需要调整 getter/setter 逻辑

例如：

setParent(File $parent) {
    if ($parent->getType() != 'group')
        throw new \Exception('Invalid parent');
    $this->parent = $parent;
}

哪种方法更好？使用专门为此目的创建的验证约束或 getter 和 setter？如果使用验证 - 我应该始终使用默认的 getter 和 setter，还是我可以在其中做任何花哨（且有用）的事情（任何示例）？

【问题讨论】：

标签： validation symfony doctrine-orm entity setter

【解决方案1】：

我会使用 Symfony 回调验证，首先是因为它是用于验证实体的专用构造，并且因为您的示例更接近用于回调验证的组件的设计理念。

虽然 PHP OOP 书籍确实鼓励您不仅使用 getter 和 setter 来检索属性值，但我一直使用它们来确保字段的默认值和约束（即我有一个 $id 属性，它是 int 类型的，我想确保在设置时将 $id 的值转换为 int它，即使有人错误地发送了一个字符串/任何东西）。

除此之外，我还认为设置器的 try/catch 块很奇怪，尤其是当我知道我正在发送正确的参数类型时。

更新：刚刚注意到您问题的最后一部分。我使用我的 Symfony getter 和 setter 基本上只是为了检索/存储值。我倾向于为构造函数中的事物设置默认值。多年来，我已经看到一些代码，其中 getter 和 setter 做了一些非常疯狂的事情，这些确实不属于实体的范围，但更接近于服务或存储库。

我认为实体是允许我将数据库表映射到 PHP 类的结构——它们不需要是智能、特殊的或在该范围之外做任何事情。

我更喜欢让它们保持简单，并使用架构的其余部分来补充可能的操作。它是否提供了一些通用功能并且可以独立隔离？为此做一个服务。是否与操纵实体、检索某些信息或以特定方式处理它们有关？为此创建一个存储库。

【讨论】：