PHP pseudo immutable objet utilisant clone

Question

Je souhaite créer un validateur personnalisé. Mais je ne veux pas appeler new à chaque fois, car cela rend plus difficile à tester. Je pense aussi App::make est un peu moche. Je veux juste Dependency Injecter mon validateur personnalisé et l'utiliser.

Mes pensées sur la façon d'y parvenir sont en utilisant clone. Appelant la méthode validate, définirait les propriétés sur le clone, plutôt que $ this. Ainsi, la version DI de la classe reste intacte. Il est également très facile de tester/simuler de cette façon.

Est-ce un modèle approprié/Dois-je simplement utiliser new?

Mon validateur personnalisé

class CustomValidator 
{ 
    protected $rules = [ 
     'key' => 'required', 
    ]; 

    protected $errors = []; 

    public function validate(array $event): CustomValidator 
    { 
     $v = Validator::make($event, $this->rules); 

     if ($v->fails()) { 
      $copy = clone $this; 
      $copy->errors = $v->errors()->toArray(); 

      return $copy; 
     } 

     return $this; 
    } 

    public function hasErrors(): bool 
    { 
     return !empty($this->errors); 
    } 

    public function getErrors(): array 
    { 
     return $this->errors; 
    } 
} 

Utilisation:

foreach ($events as $event) { 
    // customValidator is immutable. $v is cloned 
    $v = $this->customValidator->validate($event); 

    if ($v->hasErrors()) { 
     // do something with $v->errors(); 
     continue; 
    } 

    // No errors... Do something else 
} 

Answer 1

clone est tout à fait valable et est souvent utilisé pour immuabilité d'objets dans les API syntaxed couramment. Donc je ne vois pas de problème là-bas.

Je pense que vous aurez des commentaires sur semi-immuabilité bien, mais je pense personnellement que c'est une solution raisonnable à votre problème