Ist das ein guter Kandidat für eine Fabrik?

Question

Ich möchte ein Voting-System zu schaffen, in denen mehrere Domänenobjekte, über die abgestimmt werden können:

• ein Kalenderereignis
• Kommentar
• ein Benutzer

So würde ich dachte, ich erstellen eine Voteable Schnittstelle für diese Artikel:

interface Voteable 
{ 
    public function vote(User $user, $value); 
} 

Ich dachte, die vote Methode würde Proxy eine Repository-Methode, so etwas wie:

class VotingRepository 
{ 
    public function castVote(Voteable $item, User $user, $value) 
    { 
     // save the these values, along with the value 
     $itemId = $item->getId(); 
     $userId = $user->getId(); 

    } 
} 

Vorerst wird das Repository eine Datenbank sein. Diese Datenbank wird Tabellen für jede Art von Stimme verbindet:

• eventVote
• commentVote
• userVote

Also, das bedeutet im Wesentlichen, dass jede Domain-Objekt eine andere Tabelle, die die Stimmabgabe muss . Wäre das ein guter Kandidat für eine Fabrik? A VotingRepositoryFactory in diesem Fall? Mit anderen Worten so etwas wie:

class VotingRepositoryFactory 
{ 
    createVotingRepository($type) 
    { 
     switch($type) 
     { 
      case 'event': 
       // create a voting repository with EventVote table 
       return new VotingRepository(new EventVoteTable()); 
      case 'comment': 
       // create a voting repository with CommentVote table 
       return new VotingRepository(new CommentVoteTable()); 
      case 'user': 
       // create a voting repository with UserVote table 
       return new VotingRepository(new UserVoteTable()); 
     } 
    } 
} 

Dann alles zusammen zu binden, von innerhalb der Domain-Objekte (Kommentar in diesem Fall zum Beispiel), würde ich so etwas wie folgt aussehen:

class Comment implements Voteable 
{ 
    public function construct() 
    { 
     $this->_repository = VotingRepositoryFactory::createVotingRepository('comment'); 
    } 

    public function vote(User $user, $value) 
    { 
     $this->_repository->castVote($this, $user, $value); 
    } 
} 

Ist diese Sinn ergeben?

Answer 1

ja sowohl das Repository und die Fabrik sinnvoll.

ein paar Kommentare über das Werk:

ich die switch ($type) entfernen würde und für jede Art von Votable Objektmethoden erstellen. so statt

VotingRepositoryFactory::createVotingRepository('comment');

ich würde es vorziehen,

VotingRepositoryFactory::createCommentVotingRepository();

der Grund dafür ist, dass es einfach zu vergessen, einen neuen Fall an den Schalter hinzuzufügen, während (Ich bin nicht sicher über php, aber) kompilierte Sprachen sagen Ihnen, wenn eine aufgerufene Methode fehlt. Denken Sie auch daran, welche Zeichenfolgen Sie an die Factory-Methode senden können, da $ type schwer ist, während die meisten intelligenten IDEs Ihnen sagen, welche Methoden für eine Klasse/ein Objekt existieren.

eine andere Idee wäre, ein Singleton hinzuzufügen, das wie VotingRepositoryFactory::Instance->createCommentVotingRepository(); aufgerufen werden könnte.Die "Instanz" könnte dann eine DatabaseVotingRepositoryFactory oder eine FakeVotingRepositoryFactory (für Komponententests) oder eine beliebige andere Implementierung einer VotingRepositoryFactory sein. Auf diese Weise können Sie die Implementierung der VotingRepositoryFactory problemlos ersetzen, wenn Sie Komponententests schreiben oder zu einem anderen Speichersystem wechseln möchten.

nur ein paar Ideen ..

Answer 2

Ja, tut es.

:]

Answer 3

Oh ja, es macht. +1