🔥
フレームワークへの依存度を下げるFLUDパターン (4.5)
目次
- フレームワークへの依存度を下げるFLUDパターン (1) 概要
- フレームワークへの依存度を下げるFLUDパターン (2) Newsの表示
- フレームワークへの依存度を下げるFLUDパターン (3) 個別の記事の表示
- フレームワークへの依存度を下げるFLUDパターン (4) 記事の作成
- フレームワークへの依存度を下げるFLUDパターン (4.5) リファクタリング
- フレームワークへの依存度を下げるFLUDパターン (5) まとめ
フレームワークへの依存度を下げるFLUDパターン (4)
の続きです。
CodeIgniter4のチュートリアルのコードをFLUDパターンに変更しています。
重複したコードが見られますので、FLUDパターンとは直接関係ないですが、リファクタリングしておきます。
リファクタリング
NewsとNewsDto
News
と NewsDto
に重複したコードがありますので、トレイト化します。
--- a/packages/news/src/Domain/News/News.php
+++ b/packages/news/src/Domain/News/News.php
@@ -4,10 +4,12 @@ declare(strict_types=1);
namespace Acme\News\Domain\News;
-use LogicException;
+use Acme\Shared\GetterTrait;
class News
{
+ use GetterTrait;
+
private $id;
private $title;
private $slug;
@@ -24,22 +26,4 @@ class News
$this->slug = $slug;
$this->body = $body;
}
-
- public function __isset(string $name)
- {
- if (! property_exists($this, $name)) {
- throw new LogicException('No such property: ' . $name);
- }
-
- return isset($this->$name);
- }
-
- public function __get(string $name)
- {
- if (! property_exists($this, $name)) {
- throw new LogicException('No such property: ' . $name);
- }
-
- return $this->$name;
- }
}
--- a/packages/news/src/UseCase/News/NewsDto.php
+++ b/packages/news/src/UseCase/News/NewsDto.php
@@ -4,10 +4,12 @@ declare(strict_types=1);
namespace Acme\News\UseCase\News;
-use LogicException;
+use Acme\Shared\GetterTrait;
class NewsDto
{
+ use GetterTrait;
+
private $id;
private $title;
private $slug;
@@ -24,22 +26,4 @@ class NewsDto
$this->slug = $slug;
$this->body = $body;
}
-
- public function __isset(string $name)
- {
- if (! property_exists($this, $name)) {
- throw new LogicException('No such property: ' . $name);
- }
-
- return isset($this->$name);
- }
-
- public function __get(string $name)
- {
- if (! property_exists($this, $name)) {
- throw new LogicException('No such property: ' . $name);
- }
-
- return $this->$name;
- }
}
GetterTrait
は複数のパッケージで使いそうですので、Sharedパッケージに配置します。
packages/shared/src/GetterTrait.php
<?php
declare(strict_types=1);
namespace Acme\Shared;
use LogicException;
trait GetterTrait
{
public function __isset(string $name)
{
if (! property_exists($this, $name)) {
throw new LogicException('No such property: ' . $name);
}
return isset($this->$name);
}
public function __get(string $name)
{
if (! property_exists($this, $name)) {
throw new LogicException('No such property: ' . $name);
}
return $this->$name;
}
}
UseCase
ユースケースクラスにも重複がありますので、抽象クラスを抽出します。
--- a/packages/news/src/UseCase/News/CreateNewsUseCase.php
+++ b/packages/news/src/UseCase/News/CreateNewsUseCase.php
@@ -5,20 +5,9 @@ declare(strict_types=1);
namespace Acme\News\UseCase\News;
use Acme\News\Domain\News\News;
-use Acme\News\Domain\News\NewsRepositoryInterface;
-class CreateNewsUseCase
+class CreateNewsUseCase extends AbstractNewsUserCase
{
- /**
- * @var NewsRepositoryInterface
- */
- private $newsRepository;
-
- public function __construct(NewsRepositoryInterface $newsRepositry)
- {
- $this->newsRepository = $newsRepositry;
- }
-
public function run(string $title, string $slug, string $body): void
{
$news = new News(
--- a/packages/news/src/UseCase/News/GetNewsItemUseCase.php
+++ b/packages/news/src/UseCase/News/GetNewsItemUseCase.php
@@ -5,20 +5,9 @@ declare(strict_types=1);
namespace Acme\News\UseCase\News;
use Acme\News\Domain\News\News;
-use Acme\News\Domain\News\NewsRepositoryInterface;
-class GetNewsItemUseCase
+class GetNewsItemUseCase extends AbstractNewsUserCase
{
- /**
- * @var NewsRepositoryInterface
- */
- private $newsRepository;
-
- public function __construct(NewsRepositoryInterface $newsRepositry)
- {
- $this->newsRepository = $newsRepositry;
- }
-
public function run(string $slug): ?NewsDto
{
/** @var News|null $news */
--- a/packages/news/src/UseCase/News/GetNewsListUseCase.php
+++ b/packages/news/src/UseCase/News/GetNewsListUseCase.php
@@ -5,20 +5,9 @@ declare(strict_types=1);
namespace Acme\News\UseCase\News;
use Acme\News\Domain\News\News;
-use Acme\News\Domain\News\NewsRepositoryInterface;
-class GetNewsListUseCase
+class GetNewsListUseCase extends AbstractNewsUserCase
{
- /**
- * @var NewsRepositoryInterface
- */
- private $newsRepository;
-
- public function __construct(NewsRepositoryInterface $newsRepositry)
- {
- $this->newsRepository = $newsRepositry;
- }
-
/**
* @return NewsDto[]
*/
packages/news/src/UseCase/News/AbstractNewsUserCase.php
<?php
declare(strict_types=1);
namespace Acme\News\UseCase\News;
use Acme\News\Domain\News\NewsRepositoryInterface;
abstract class AbstractNewsUserCase
{
/**
* @var NewsRepositoryInterface
*/
protected $newsRepository;
public function __construct(NewsRepositoryInterface $newsRepositry)
{
$this->newsRepository = $newsRepositry;
}
}
Newsコントローラ
重複したコードをコンストラクタに抽出します。
--- a/app/Controllers/News.php
+++ b/app/Controllers/News.php
@@ -10,10 +10,19 @@ use Acme\News\UseCase\News\CreateNewsUseCase;
class News extends BaseController
{
+ /**
+ * @var NewsRepository
+ */
+ private $repository;
+
+ public function __construct()
+ {
+ $this->repository = model(NewsRepository::class);
+ }
+
public function index()
{
- $repository = model(NewsRepository::class);
- $useCase = new GetNewsListUseCase($repository);
+ $useCase = new GetNewsListUseCase($this->repository);
/** @var NewsDto[] $newsList */
$newsList = $useCase->run();
@@ -30,8 +39,7 @@ class News extends BaseController
public function view($slug = null)
{
- $repository = model(NewsRepository::class);
- $useCase = new GetNewsItemUseCase($repository);
+ $useCase = new GetNewsItemUseCase($this->repository);
$data['news'] = $useCase->run($slug);
@@ -48,8 +56,7 @@ class News extends BaseController
public function create()
{
- $repository = model(NewsRepository::class);
- $useCase = new CreateNewsUseCase($repository);
+ $useCase = new CreateNewsUseCase($this->repository);
if ($this->request->getMethod() === 'post' && $this->validate([
'title' => 'required|min_length[3]|max_length[255]',
ディレクトリ構成
現状のディレクトリ構成は以下になります。
app/
├── Config
│ ├── Autoload.php
├── Controllers
│ ├── News.php
├── Models
│ └── NewsRepository.php
└── Views
└── news
├── create.php
├── overview.php
├── success.php
└── view.php
packages/
├── news ... Newsパッケージ
│ └── src
│ ├── Domain ... ドメイン層
│ │ └── News
│ │ ├── News.php
│ │ └── NewsRepositoryInterface.php
│ └── UseCase ... ユースケース層
│ └── News
│ ├── AbstractNewsUserCase.php ← 追加
│ ├── CreateNewsUseCase.php
│ ├── GetNewsItemUseCase.php
│ ├── GetNewsListUseCase.php
│ └── NewsDto.php
└── shared ... Sharedパッケージ
└── src
└── GetterTrait.php ← 追加
app/
以下のファイルはモデルのファイル名を変更しただけで配置はそのままにしています。
packages/
以下にドメイン層とユースケース層のコードが追加されています。
フレームワークへの依存度を下げるFLUDパターン (5)
へ続く。
Discussion