【发布时间】:2016-11-07 19:54:05
【问题描述】:
假设我们有一个构建文章的系统。文章有一些组件验证器,清洁器,存储......在客户端构建文章我必须实例化每个组件:
$title = 'title';
$description = 'Description in html';
//Cleaner just clean some things from each field.
$cleaner = new Cleaner();
//Validator throw exception if something is not correct
$validator = new Validator();
// Storage save files and article itself
$storage = new Storage();
//Dom Class get some files from description field
$dom = new Dom();
$files = $dom->getFiles($description);
$storage->files($files);
$article = new ArticleBuilder();
$article->addTitle($validator->title($cleaner->title($title)));
$article->addDescription($validator->description($cleaner->description($description)));
$article->add....
没有这些,就不可能构建文章。
我的问题是: 我可以像这样使用工厂模式来创建所有这些:
class ArticleFactory
{
private $article;
public function __construct()
{
$this->article = new ArticleBuilder();
}
public function setTitle(string $title)
{
$title = ($this->validator())->title($title);
$title = ($this->cleaner())->title($title);
$this->article->addTitle($title);
}
public function setDescription(string $des)
{
$des = ($this->validator())->title($des);
$des = ($this->cleaner())->title($des);
$this->article->addDescription($des);
}
public function getArticle(): ArticleBuilder
{
return $this->article;
}
public function getFiles($description)
{
return ($this->dom())->getFiles($description);
}
public function storeFile($files)
{
($this->storage())->files($files);
}
public function validator(): ValidatorInterface
{
return new Validator();
}
public function cleaner(): CleanerInterface
{
return new Cleaner();
}
public function storage(): StorageInterface
{
return new Storage();
}
public function dom(): DomInterface
{
return new Dom();
}
}
在客户端用上面的工厂创建文章更方便:
$myTitle = 'my title';
$myDes = 'mty description';
$article = new ArticleFactory();
$article->setTitle($myTitle);
$article->setDescription($myDes);
$files = $article->getFiles($description);
$article->storeFile($files);
这是否违反任何 SOLID 原则?
有没有更好的方法?
【问题讨论】:
-
我可以使用工厂模式来创建所有这些当然可以。问题是它会有多大用处。为此,我们需要询问我们的水晶球,因为您提供了零上下文。
-
这个我不太明白。您有一个
ArticleFactory,但您返回的是Validator、Cleaner和Storage类型。这些是Article的派生词还是依赖项? -
我已经更新了我的回答。如果你想要更多的东西,请告诉我。谢谢
-
您的设计确实很奇怪,但您必须解释整个问题域,以便我们提供更好的解决方案。
标签: oop design-patterns factory-pattern solid-principles