本の感想や質問をお気軽にコメントしてください。

https://zenn.dev/meijin/books/laravel-graduate-fat-controller

リファクタリングを実践する特性上、環境構築が上手くいかないといったトラブルが頻出なので、何かあればこちらにコメントください！

ぺこり。環境構築で手間取っております。
以下の現象です。

docker環境構築の際、bookmark-httpsがすぐ落ちてしまう。
リスタートしているが2-3回ほど繰り返して落ちる。
エラーログを見ていると以下のエラー発見
cannot load certificate "/var/lib/https-portal/localhost/local/chained.crt"

すみません、こちらも同様のエラーで環境構築から進めない状況です、、、。
環境としましては、
intelのmacとm2のmacの両方で試しましたが同様のエラーでした。
またrootlessの確認もしましたが通常のdockerでした。
コンテナのエラーよりdocker-compose.ymlのhttps-portalを以下に変更したところ、コンテナは立ち上がりますが400番が帰ってきて進めない状況です。
volumes:
- ./docker/https-portal/cert:/var/lib/https-portal/localhost/local

何か改善案や代替案などありましたらご教授願いたいです。

4点気になったことがありますので、報告いたします。

なお、私の環境は以下のとおりです。

• チップ Apple M2
• macOS 14.6.1
• Docker Desktop 4.34.0

(1) bookmark-httpsが落ちる
Docker コンテナの立ち上げでbookmark-httpsが落ちます。
原因はchained.crtのパスに間違いあるためです。
解決策は、docker-compose.ymlを以下のように修正します。（参考）

-      - ./docker/https-portal/cert:/var/lib/https-portal
+      - ./docker/https-portal/cert:/var/lib/https-portal/localhost

(2) Please provide a valid cache path.エラーが発生する
https://localhost/ にアクセスすると以下のエラーメッセージが表示されます。

Please provide a valid cache path.

以下のコマンドを実行することで、エラーが解消します。

docker compose exec app mkdir -p /opt/laravel-bookmark/storage
docker compose exec app mkdir -p /opt/laravel-bookmark/storage/framework/sessions
docker compose exec app mkdir -p /opt/laravel-bookmark/storage/framework/views
docker compose exec app chown -R www-data:www-data /opt/laravel-bookmark/storage
docker compose exec app chmod -R 775 /opt/laravel-bookmark/storage

(3) php.iniのdate.timezoneの値が${TZ}になっている
php.iniのdate.timezoneの値が${TZ}になっています。

% docker compose exec app cat /usr/local/etc/php/php.ini
(省略)
date.timezone = ${TZ}

私はdocker/php/Dockerfileの以下の行を修正して対応しました。

  COPY php.ini /usr/local/etc/php/php.ini
+ RUN sed -i "s/\${TZ}/Asia\/Tokyo/" /usr/local/etc/php/php.ini

  COPY --from=composer:latest /usr/bin/composer /usr/bin/composer

(4) Chapter 25のタイプミス
Chapter 25 実際に外部アクセスはしない”モック”を作成しようにタイプミスがあります。

- final class MockLinkPreview islmplements LinkPreviewInterface
+ final class MockLinkPreview implements LinkPreviewInterface

-       $useCase = new CreateBookmarkUseCase(j);
+       $useCase = new CreateBookmarkUseCase(new MockLinkPreview());

以上です。

お世話になります！
共有したいことがあるので、記載させていただきます。

感想

良書だと感じました！
あらかじめ用意されたFat Controllerからリファクタリングを進めることでSlim Controllerになっていき、徐々にソースコードの可読性が向上していきます。
リファクタリングをあまりやったことがない自分にとっては、とても良い学びになりました。
手を動かしながら進められて、説明も丁寧なので、理解しやすかったです。

• 1 章 「ユースケース」を作って処理を引っ越しさせてみよう
• 2 章 Fat Controller とはどんな状態か知ろう
• 3 章 FormRequest、Middleware など Laravel の機能を使いこなそう
• 4 章 サービスコンテナの働きを理解して使いこなそう
• 5 章 テストコードを書いて品質維持できる体制を作ろう！

1〜3章は問題なく進められましたが、4〜5章は難しさを感じました。
教材が難しいというよりは、技術的に難易度が高いといった感じでしょうか？
4〜5章は難しかったので2周してみましたが、それでも理解度50〜70%くらいです。
サービスコンテナやサービスプロバイダー、テストコードを自走して書けるレベルになっていないので、他の教材で補おうと考えています。
サービスコンテナやテストコードの入門という位置付けとしては良かったと感じています。

ソースコードで気になる点

17ページ

CreateBookmarkUseCase.phpにて、エラーが発生してしまいました。

Symfony\Component\Debug\Exception\FatalThrowableError
Class 'Cardei\LinkPreview\Client' not found

use文を修正することで、エラーを解消することができました。

• 修正前：use Cardei\LinkPreview\Client;
• 修正後：use Dusterio\LinkPreview\Client;

LinkPreviewライブラリメンテナンス終了に伴う修正漏れだと予想しています。
他にもCardeiと記載されている箇所が複数あり、それらを全てDusterioに置換した結果、正常に動作しました。
改めて見直した方が良いかなと思います。

24ページ

「CreateBookmarkUseCase を LinkPreviewInterface に依存させる」のBookmarkControllerのcreateメソッド修正に関して、現状は下記のようになっています。

<?php

namespace App\Http\Controllers\Bookmarks;

+j
// 中略

class BookmarkController extends Controller
{
    // 中略

    public function create(CreateBookmarkRequest $request)
    {
-       $useCase = new CreateBookmarkUseCase();
+       $useCase = new CreateBookmarkUseCase(new LinkPreview());
        $useCase->handle($request->url, $request->category, $request->comment);

        // 暫定的に成功時は一覧ページへ
        return redirect('/bookmarks', 302);
    }
    // 中略
}

次の内容が正しいと思ったのですが、いかがでしょうか？

<?php

namespace App\Http\Controllers\Bookmarks;

+use App\Lib\LinkPreview\LinkPreview;
// 中略

class BookmarkController extends Controller
{
    // 中略

    public function create(CreateBookmarkRequest $request)
    {
-       $useCase = new CreateBookmarkUseCase();
+       $useCase = new CreateBookmarkUseCase(new LinkPreview());
        $useCase->handle($request->url, $request->category, $request->comment);

        // 暫定的に成功時は一覧ページへ
        return redirect('/bookmarks', 302);
    }
    // 中略
}