Shogo Kawase / @shogogg 我々はなぜ「層」を分けるのか 〜「関心の分離」と「抽象化」で手に入れる変更に強いシンプルな設計〜 

自己紹介 河瀨 翔吾 / Shogo Kawase 　　　　　　 エンジニアリングマネージャー I LOVE... 妻 / 型安全 / アジャイル / ももいろクローバーZ F1 / マリオカート / ACE COMBAT shogogg shogogg 

おしながき 1 「層」についてのおさらい 2 「層」が実現する関心の分離と抽象化 3 「層」を形骸化させるアンチパターン 4 まとめ 

おしながき 1 「層」についてのおさらい 2 「層」が実現する関心の分離と抽象化 3 「層」を形骸化させるアンチパターン 4 まとめ 

いろいろな「層」の分け方 ● MVC ● 3層アーキテクチャ ● オニオンアーキテクチャ ● クリーンアーキテクチャ（？） 

'; while ($row = mysql_fetch_assoc($result)) { echo "

{$row['name']} - ¥{$row['price']}

"; } echo ''; ?> いにしえの PHP 

MVC の流行 ① リクエスト Controller View Model ② 処理依頼 ③ 処理結果 User ④ パラメータ ⑤ 表示 

The Clean Architecture 引用元：Robert C. Martin「The Clean Architecture」 

おしながき 1 「層」についてのおさらい 2 「層」が実現する関心の分離と抽象化 3 「層」を形骸化させるアンチパターン 4 まとめ 

Mr. Logic Mr. Logic 

Mr. Logic Mr. Logic はとってもいそがしい 顧客 いそがしい…… チャット Email 郵送 FAX 電話 

アシスタントがいれば…… 顧客 Assistant Mr. Logic 楽になった！ チャット Email 郵送 FAX 電話 チャット 

Mr. Logic Mr. Logic はそれでもいそがしい Assistant チャット Cloud Service B Cloud Service A Cloud Service C CSV JSON SQL 面倒だな… 

Mr. Logic 専門家がいれば…… Cloud Service B Cloud Service A Cloud Service C CSV SQL 楽になった！ Sheet JSON Admin 

Mr. Logic こうして Mr. Logic は救われた Assistant チャット Sheet Admin 

関心の分離 Controller Infrastructure Cloud Service A CSV JSON SQL User チャット Email 郵送 FAX 電話 Cloud Service B Cloud Service C リクエストに基づく入出力 DB・API・外部モジュール・レガシーコードとのやり取り Service 

Service 抽象化？ Controller Infrastructure ユーザーからJSONで受け取った データをDBに保存するよ！ 

Service 抽象化できてない…… Controller Infrastructure ユーザーからJSONで受け取った データをDBに保存するよ！ 

Service 抽象化すると？ Controller Infrastructure どっかから受け取ったデータを なんらかの形で保存しておくよ！ 

おしながき 1 「層」についてのおさらい 3 「層」を形骸化させるアンチパターン 4 まとめ 2 「層」が実現する関心の分離と抽象化 

アンチパターン① HTTPの都合に振り回されるサービス 

問題のあるコード // Controller public function index(Request $request) { return $this->service->findUsers($request->context(), $request->all()); } // Service public function findUsers(Context $context, array $params) { // ロジック内で「 HTTP の都合（文字列）」を解釈させられている $isActive = isset($params['is_active']) && $params['is_active'] === 'true'; $role = UserRole::tryFrom((int)$params['role']); $dateFrom = isset($params['date_from']) ? Carbon::parse($params['date_from']) : null; // 本来の Service の責務：コンテキストに基づいて条件の絞り込み $tenantId = $context->isAdmin() ? ($params['tenant_id'] ?? null) : $context->tenantId(); return $this->userRepository->search($tenantId, $role, $dateFrom, $isActive); } 

問題のあるコード // Controller public function index(Request $request) { return $this->service->findUsers($request->context(), $request->all()); } // Service public function findUsers(Context $context, array $params) { // ロジック内で「 HTTP の都合（文字列）」を解釈させられている $isActive = isset($params['is_active']) && $params['is_active'] === 'true'; $role = UserRole::tryFrom((int)$params['role']); $dateFrom = isset($params['date_from']) ? Carbon::parse($params['date_from']) : null; // 本来の Service の責務：コンテキストに基づいて条件の絞り込み $tenantId = $context->isAdmin() ? ($params['tenant_id'] ?? null) : $context->tenantId(); return $this->userRepository->search($tenantId, $role, $dateFrom, $isActive); } Controllerが入力値をServiceに丸投げ 

問題のあるコード // Controller public function index(Request $request) { return $this->service->findUsers($request->context(), $request->all()); } // Service public function findUsers(Context $context, array $params) { // ロジック内で「 HTTP の都合（文字列）」を解釈させられている $isActive = isset($params['is_active']) && $params['is_active'] === 'true'; $role = UserRole::tryFrom((int)$params['role']); $dateFrom = isset($params['date_from']) ? Carbon::parse($params['date_from']) : null; // 本来の Service の責務：コンテキストに基づいて条件の絞り込み $tenantId = $context->isAdmin() ? ($params['tenant_id'] ?? null) : $context->tenantId(); return $this->userRepository->search($tenantId, $role, $dateFrom, $isActive); } Service 内で変換・判定が必要に…… 

問題のあるコード // Controller public function index(Request $request) { return $this->service->findUsers($request->context(), $request->all()); } // Service public function findUsers(Context $context, array $params) { // ロジック内で「 HTTP の都合（文字列）」を解釈させられている $isActive = isset($params['is_active']) && $params['is_active'] === 'true'; $role = UserRole::tryFrom((int)$params['role']); $dateFrom = isset($params['date_from']) ? Carbon::parse($params['date_from']) : null; // 本来の Service の責務：コンテキストに基づいて条件の絞り込み $tenantId = $context->isAdmin() ? ($params['tenant_id'] ?? null) : $context->tenantId(); return $this->userRepository->search($tenantId, $role, $dateFrom, $isActive); } 本来やりたいことはこれだけ 

改善版 // Controller public function index(FindUsersRequest $request) { return $this->service->findUsers( $request->context(), $request->toCriteria(), ); } // Service public function findUsers(Context $context, UserSearchCriteria $criteria) { $tenantId = $context->isAdmin() ? $criteria->targetTenantId : $context->tenantId(); return $this->userRepository->search($tenantId, $criteria); } 

改善版 // Controller public function index(FindUsersRequest $request) { return $this->service->findUsers( $request->context(), $request->toCriteria(), ); } // Service public function findUsers(Context $context, UserSearchCriteria $criteria) { $tenantId = $context->isAdmin() ? $criteria->targetTenantId : $context->tenantId(); return $this->userRepository->search($tenantId, $criteria); } Controller が Service の期待する形に入力値を変換して渡す 

改善版 // Controller public function index(FindUsersRequest $request) { return $this->service->findUsers( $request->context(), $request->toCriteria(), ); } // Service public function findUsers(Context $context, UserSearchCriteria $criteria) { $tenantId = $context->isAdmin() ? $criteria->targetTenantId : $context->tenantId(); return $this->userRepository->search($tenantId, $criteria); } Service が本来の責務に集中できるようになった!! 

アンチパターン② 浸食する Active Record 

問題が起きる例 // DB Migration Schema::table('members', function (Blueprint $table): void { // 社内管理用メモ $table->string('internal_note'); }); // ユーザー向け API 用のサービス public function show(int $id): ?Member { return Member::find($id); } // API のレスポンス { "id": 1, "name": "John Smith", "internal_note": "クレーマーなので要注意 " } 

問題が起きる例 // DB Migration Schema::table('members', function (Blueprint $table): void { // 社内管理用メモ $table->string('internal_note'); }); // ユーザー向け API 用のサービス public function show(int $id): ?Member { return Member::find($id); } // API のレスポンス { "id": 1, "name": "John Smith", "internal_note": "クレーマーなので要注意 " } 社内用のメモ欄を記録するためのカラムを追加 

問題が起きる例 // DB Migration Schema::table('members', function (Blueprint $table): void { // 社内管理用メモ $table->string('internal_note'); }); // ユーザー向け API 用のサービス public function show(int $id): ?Member { return Member::find($id); } // API のレスポンス { "id": 1, "name": "John Smith", "internal_note": "クレーマーなので要注意 " } ユーザー向け API の処理は一切変更していない 

問題が起きる例 // DB Migration Schema::table('members', function (Blueprint $table): void { // 社内管理用メモ $table->string('internal_note'); }); // ユーザー向け API 用のサービス public function show(int $id): ?Member { return Member::find($id); } // API のレスポンス { "id": 1, "name": "John Smith", "internal_note": "クレーマーなので要注意 " } ユーザー向け API のレスポンスに社内管理用のメモが流出 😱 

Active Record（Eloquent）の利点と限界 ● Active Record は強力なデザインパターンであり、小規模なアプリケーションであれば 開発スピードに大きく寄与する ● 一方で、データベースと密結合していることで、変更が予期せぬ影響を及ぼしたり、ど こでもデータベース操作ができてしまうなど、責務の分離が難しくなる ● ある程度の規模になったら（またはそれを見込むのであれば）Active Record はインフ ラ層に閉じ込め、サービス層では独自に定義したモデル（DTO/POPO）に変換して扱 うのがオススメ 

アンチパターン③ 技術的詳細を漏らす例外 

問題のあるコード // Repository public function lookup(int $id): ?Member { return EloquentMember::find($id)?->toDomainModel(); } // Service public function lookup(int $id): ?Member { return $this->repository->lookup($id); } 

問題のあるコード // Repository public function lookup(int $id): ?Member { return EloquentMember::find($id)?->toDomainModel(); } // Service public function lookup(int $id): ?Member { return $this->repository->lookup($id); } Eloquent を変換して返していてバッチリ？ 

問題のあるコード // Repository public function lookup(int $id): ?Member { return EloquentMember::find($id)?->toDomainModel(); } // Service public function lookup(int $id): ?Member { return $this->repository->lookup($id); } DB操作に失敗すると QueryException が throw されてしまう 

問題のあるコード // Repository public function lookup(int $id): ?Member { return EloquentMember::find($id)?->toDomainModel(); } // Service public function lookup(int $id): ?Member { return $this->repository->lookup($id); } 例外をキャッチしていないので伝播してしまう 

改善版？ // Repository public function lookup(int $id): ?Member { return EloquentMember::find($id)?->toDomainModel(); } // Service public function lookup(int $id): ?Member { try { $member = $this->repository->lookup($id); } catch (QueryException $e) { $this->logger->error('Database query failed', ['exception' => $e]); } return $member; } 

改善版？ // Repository public function lookup(int $id): ?Member { return EloquentMember::find($id)?->toDomainModel(); } // Service public function lookup(int $id): ?Member { try { $member = $this->repository->lookup($id); } catch (QueryException $e) { $this->logger->error('Database query failed', ['exception' => $e]); } return $member; } 例外をキャッチしてログを出力するように 

改善版？ // Repository public function lookup(int $id): ?Member { return EloquentMember::find($id)?->toDomainModel(); } // Service public function lookup(int $id): ?Member { try { $member = $this->repository->lookup($id); } catch (QueryException $e) { $this->logger->error('Database query failed', ['exception' => $e]); } return $member; } データーベースに関する技術的詳細が層を超えて登場 

今度こそ改善版 // Custom exception for better error handling final class RepositoryException extends \RuntimeException {} // Repository public function lookup(int $id): ?Member { try { return EloquentMember::find($id)?->toDomainModel(); } catch (QueryException $e) { throw new RepositoryException('Failed to lookup the member from database', 0, $e); } } // Service public function lookup(int $id): ?Member { try { $member = $this->repository->lookup($id); } catch (RepositoryException $e) { $this->logger->error('An error occurred while looking up the member', ['exception' => $e]); } return $member; } 

今度こそ改善版 // Custom exception for better error handling final class RepositoryException extends \RuntimeException {} // Repository public function lookup(int $id): ?Member { try { return EloquentMember::find($id)?->toDomainMember(); } catch (QueryException $e) { throw new RepositoryException('Failed to lookup the member from database', 0, $e); } } // Service public function lookup(int $id): ?Member { try { $member = $this->repository->lookup($id); } catch (RepositoryException $e) { $this->logger->error('An error occurred while looking up the member', ['exception' => $e]); } return $member; } 技術的詳細と関係のない独自例外を定義する 

今度こそ改善版 // Custom exception for better error handling final class RepositoryException extends \RuntimeException {} // Repository public function lookup(int $id): ?Member { try { return EloquentMember::find($id)?->toDomainModel(); } catch (QueryException $e) { throw new RepositoryException('Failed to lookup the member from database', 0, $e); } } // Service public function lookup(int $id): ?Member { try { $member = $this->repository->lookup($id); } catch (RepositoryException $e) { $this->logger->error('An error occurred while looking up the member', ['exception' => $e]); } return $member; } 発生した例外を独自例外でラップする 

今度こそ改善版 // Custom exception for better error handling final class RepositoryException extends \RuntimeException {} // Repository public function lookup(int $id): ?Member { try { return EloquentMember::find($id)?->toDomain(); } catch (QueryException $e) { throw new RepositoryException('Failed to lookup the member from database', 0, $e); } } // Service public function lookup(int $id): ?Member { try { $member = $this->repository->lookup($id); } catch (RepositoryException $e) { $this->logger->error('An error occurred while looking up the member', ['exception' => $e]); } return $member; } サービス層に漏れ出ていた技術的詳細が消えた！ 

おしながき 1 「層」についてのおさらい 3 「層」を形骸化させるアンチパターン 4 まとめ 2 「層」が実現する関心の分離と抽象化 

まとめ ロジックの複雑さに立ち向かう ● バックエンドが複雑なプレゼンテーションを担うことは減った一方、扱う技術やドメイ ンの複雑さは増している ● 「層」を分離することで、我々は複雑さに立ち向かっている 「層」によって関心を分離する ● ビジネスロジックを技術的な詳細から解放することでシンプルになり、変更に強くなる ● 内側の層（サービス）の都合に外側の層（コントローラーやインフラ）が合わせる 適切な「抽象化」が分離の鍵 ● 抽象化が甘すぎると、サービス層に技術的な詳細が浸食しやすくなる ● 関心の分離は、適切な抽象化によって達成される 

＼採用やってます!!／