这是一个很好的工厂候选人吗？

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