将数据添加到对象 - 验证“然后”添加，还是在添加时验证？

Question

举个例子，

class Cup {}

class MyObject {
    protected $cups = array();

    public function addCup(Cup $cup) {
        if (!in_array($cup, $this->getCups())) {
            array_push($this->cups, $cup);
        }
        return $this;
    }

    public function addCups(array $cups) {
        // Add cups logic, see below
    }

    public function getCups() {
        return $this->cups;
    }

    public function setCups(array $cups) {
        // Set cups logic, see below
    }
}

在这个类上，我们可以使用 $myObject->addCup() 添加一个 Cup，但我们也可以使用 $myObject 添加多个杯子->addCups()，或使用 $myObject->setCups() 覆盖任何现有杯子。

我的问题是，在 addCups() 和 setCups() 方法中，您是否在之前验证所有传递的数据是否有效/em> 添加它们，或者您在添加它们时验证？

场景 1，在添加之前验证：

public function addCups(array $cups) {
    foreach($cups as $cup) {
        if (!($cup instanceof Cup)) {
            throw new InvalidArgumentException();
        }
    }
    foreach($cups as $cup) {
        $this->addCup($cup);
    }
}

场景 2，在添加时验证：

public function addCups(array $cups) {
    foreach($cups as $cup) {
        if (!($cup instanceof Cup)) {
            throw new InvalidArgumentException();
        }
        $this->addCup($cup);
    }
}

我很高兴没有代码有很大差异，但它从根本上改变了对象添加/设置数据的方式。在场景 1 中，只有在添加的所有数据都有效的情况下，您才能向对象添加新数据，而在场景 2 中，数据实际上会添加到错误点。

我个人一直在使用场景 1，但我忍不住觉得你也可以使用场景 2，因为如果你抛出了异常，你就不应该继续执行？

我很想知道是否有具体的设计模式可以遵循，或者是否有人们的意见。

谢谢。

Answer 1

首先检查所有项目。

这允许事务行为，这通常是一件好事。这确保了行为易于理解和调试。

就性能而言，它也是更可取的。

但是，如果您的目标是“尽可能多地完成工作”，请检查每一项并在可能的情况下添加它 - 如果一项失败，只需跳到下一项。不过，这是一个罕见的用例。

Answer 2

我想说这取决于您是否想将添加“杯子”视为交易。当一个或多个 Cup 失败时，您是否希望添加所有 Cup 都失败，请使用方法 1。否则，请使用方法 2。

据我所知，这两种方法都是可行的，并且都有自己的用例。