這是一個很好的工廠候選人嗎？

Question

我想創建一個投票系統，在多個領域對象可以進行表決：

• 日曆事件
• 評論
• 用戶

所以我想我會創建一個Voteable接口，用於以下項目：

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

我認爲這vote方法將代理信息庫方法，是這樣的：

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

    } 
} 

目前，該庫將是一個數據庫。這個數據庫將對每種類型的投票鏈接表：

• eventVote
• commentVote
• userVote

所以，這基本上意味着，每個域對象需要另一個表來投的票。這會成爲工廠的一個很好的候選人嗎？ A VotingRepositoryFactory在這種情況下？換句話說是這樣的：

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()); 
     } 
    } 
} 

然後，域對象中嘗試所有這些，從（在這種情況下，例如評論），我會是這個樣子：

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

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

這是否合理？

Answer 1

是的版本庫和工廠都有意義。

有關工廠幾點意見：

我會刪除switch ($type)，併爲每種類型的可投票對象的方法。所以不是

VotingRepositoryFactory::createVotingRepository('comment');

我寧願

VotingRepositoryFactory::createCommentVotingRepository();

的原因是很容易忘記一個新的情況下添加到交換機上，同時（我不知道PHP，但）編譯語言會告訴你何時缺少一個被調用的方法。還要記住你可以將哪些字符串發送到工廠方法中，因爲$ type很難，而大多數智能IDE會告訴你類/對象上存在哪些方法。

另一個想法是添加一個可以被稱爲VotingRepositoryFactory::Instance->createCommentVotingRepository();的單身人士。那麼「實例」可以是一個DatabaseVotingRepositoryFactory或一個FakeVotingRepositoryFactory（用於單元測試）或VotingRepositoryFactory的任何其他實現。這樣你可以輕鬆地替換VotingRepositoryFactory的實現，如果你想編寫單元測試或切換到其他存儲系統。

只是一些想法..

Answer 2

是的，它的確如此。

：]

Answer 3

哦是的，它使。 +1