[laravel]レポジトリをコントローラー内で使う方法

Question

laravel5.8でDB周りの処理のリファクタリングをしております。

レポジトリ(インターフェースとレポジトリの二段階)を作成し、ロジックの分離をしようとしているのですが、エラーが解決できずにいます。

・エラー内容(レスポンス内のメッセージ)

 "message": "Class App\Repositories\Impression\ImpressionRepositoryInterface does not exist",

エラーを見ると、interfaceがないと言われています。
タイポを疑い、しつこくチェックはしたのですが、解決せず、、、。

以下、実装した手順と資料を書かせていただくので、お気づきの点がありましたらアドバイスをいただけますと幸いです。

・やったこと
①App配下にRepositoriesディレクトリ作成、同ディレクトリ内にinterfaceとrepositoryのファイル作成

②interfaceにはメソッドのみ定義、repositoryで継承してロジックを実装

<?php

namespace App\Repositories\Impression;

use Illuminate\Support\Facades\DB;

class ImpressionRepository implements ImpressionRepositoryInterface
{
    protected $table = 'impressions';

    public function get_public($id)
    {
        return  DB::table($this->table)
            ->select('users.name', 'users.img', 'impressions.*')
            // 非公開(1)の感想は取得しない
            ->where('impressions.secret', 0)
            ->where('impressions.book_id', $id)
            ->join('users', 'impressions.user_id', '=', 'users.id')
            ->get();
    }

    public function get_myImpressions($user_id)
    {
        return  DB::table($this->table)
            ->select('users.name', 'users.img', 'impressions.*')
            // 非公開も含め取得する
            ->where('impressions.user_id', $user_id)
            ->join('users', 'impressions.user_id', '=', 'users.id')
            ->get();
    }
}

③ServiceProviderに登録(今回はRepository用のProviderを作成)

<?php

namespace App\Providers;

use Illuminate\Support\ServiceProvider;

use App\Repositories\Impression\ImpressionRepositoryInterface;
use App\Repositories\Impression\ImpressionRepository;

class RepositoryServiceProvider extends ServiceProvider
{
    /**
     * Register services.
     * インターフェースとレポジトリの登録
     * boot(使用)はコントローラ内
     * @return void
     */
    public function register()
    {
        $this->app->bind(
            // Impression API
            ImpressionRepositoryInterface::class,
            ImpressionRepository::class,
        );
    }

④新規作成したServieProviderなので、config/app.phpに登録

    'providers' => [
       App\Providers\RepositoryServiceProvider::class,
    ],

⑤config/app.phpを更新するため、キャッシュ更新

php artisan config:cache

⑥コントローラのコンストラクタ内でinterfaceを呼び出し、

<?php

namespace App\Http\Controllers\API;

use App\Http\Controllers\Controller;
use App\Repositories\Impression\ImpressionRepositoryInterface;
use Illuminate\Http\Request;
use App\Http\Requests\StoreImpressionsRequest;

class ImpressionController extends Controller
{
    private $impression;

    public function __construct(ImpressionRepositoryInterface $impressionRepositoryInterface)
    {
        $this->impression = $impressionRepositoryInterface;
    }

    /**
     * Display the specified resource.
     *
     * @param  int  $id
     * @return \Illuminate\Http\Response
     */
    public function show($id)
    {
        $impressions = $this->impression->get_public($id);
        return response()->json($impressions);
    }

・参考資料
LaravelでRepositoryパターンを実装する-入門編-
Laravel 6 Advanced - e7 - Repository Pattern(youtube動画)

Answer 1

ざっと見ても問題は見つからなかったので

• エラーメッセージがどこで出てるか確認。
• PhpStorm等IDEを使ってtypo確認。
• composer updateしてみる。単に新しく作ったファイルだから見えてないだけかも。まれにある。

その他。
開発中は最初からキャッシュしなければphp artisan config:cacheは不要。
なぜか開発中にキャッシュして余計なミスしてる人多いけどキャッシュするのは本番用。
php artisan config:clearで消していい。

Comments

[marutto]

アドバイスありがとうございます！

composer update
composer dump-autoload
を試したところ、無事解決できました！

キャッシュについてもご指摘ありがとうございます。
仰るとおり、過去にキャッシュが原因で詰まったことがありましたので、
開発中は削除していくことにします。

ありがとうございました！

Answer 2

    public function __construct(ImpressionRepository $impressionRepository)
    {
        $this->impression = $impressionRepository;
    }

インターフェイスは インスタンスにできないのでこうだと思うけど

Comments

[kawax]

interfaceでもいい。というかこれこそLaravel最大の特徴。

[退会済みユーザー]

@kawax さん
ありがとうございます。勉強になります。