コードの重複を避けるためにパラメータの妥当性検査を行うべき場所

Question

ベストプラクティス/推奨事項がここにありますか？ XMLElementを操作する関数を提供するクラスがあるとします。

class Foo 
{ 
    private $data; 

    public function setData(\SimpleXMLElement $data): void 
    { 
     $this->data = $data; 
    } 

    public function getElement():string 
    { 
     return $this->data->element; 
    } 
} 

クラスはそうのような他のクラスで使用されます。

class Bar 
{ 
    public function fooBar(string $data) 
    { 
     $xml = new \SimpleXMLElement($data); 
     $foo = new Foo(); 

     $foo->setData($xml); 
     echo $foo->getElement(); 
    } 
} 

setData()とfooBar()の両方がそれらのパラメータを検証するべきか？

fooBar()がそのパラメータを検証しても問題ありませんか？

または、foobar()を呼び出す関数がそのパラメータを検証する必要がありますか？

Answer 1

実際には、アーキテクチャの仕様によって異なります。検証は、実際にスローEmailAddressクラスのコンストラクタで起こるこの場合

$bob = new User; 
$bob->setEmail(new EmailAddress($param)); 

：基本的には利用されている

class User { 
    // some code 
    public function setEmail(EmailAddress $email) { .... 
} 

：最近、私はこのようになりますよう、アプローチを好む始めています例外として、有効な電子メールアドレスではないものでインスタンス化しようとしている場合です。

私はそれを「価値オブジェクト」と総称していると思いますが、それについて私を引用しません。私は、DDDの全体的な疑問があるにも関わらず、DDDアプローチを一般に普及させている人々からこれを借用してきました（それは少し混乱したような気がします。