Laravel・PHP エラーの対処

Question

laravelで以下のfunctionを呼び出すと、

Type error: Return value of App\Repositories\EloquentHospitalRepository::getHospitalBrowsingHistories() must be an instance of Illuminate\Pagination\LengthAwarePaginator, array returned

上記のエラーを出してしまいます。

$userとはログインしたユーザーの情報が渡ってくる変数で、
そのユーザー情報が入っているHospitalUserのテーブルを見て、
最終的にそれと紐づいているHospitalの病院情報をみるロジックとなっております。

public function getHospitalBrowsingHistories(User $user): LengthAwarePaginator{
        $hospital = [];
        $hospital_users = HospitalUser::where('user_id', $user->id)->get();
        foreach ($hospital_users as $hospital_user) {
        $hospital[] = Hospital::where('id', $hospital_user->hospital_id)
        ->where('public', true)
        ->paginate(config('hospital-browsing-history.perPage'));
    }
        return $hospital;
    }

どなたかご教授ください。よろしくお願いします。

Answer 1

エラーの原因はメッセージの通り、LengthAwarePaginatorを返すと宣言しているのにarrayを返しているためです。

HospitalUserのテーブルを見て、

最終的にそれと紐づいているHospitalの病院情報をみるロジックとなっております。

HospitalUserとHospitalがどう紐付いているのか、そして最終的にどうしたいのか不明ですが、
宣言通りLengthAwarePaginatorを返したいのであればforeachでごにょごにょして配列（$hospital）を返すのではなく、with等を使用して関連データを取得し、最後にpaginateを呼び出してその結果を返してください。

Comments

[h.yamazaki]

回答ありがとうございます・
withを使用してやってみたのですが、userの情報が紐づかずにhospitalテーブルの情報が全件取得されてしまいます。
------------------------------
public function getHospitalBrowsingHistories(): LengthAwarePaginator{
$hospitals = Hospital::with(['hospitalUsers' => function ($query) {
$user = Auth::user();
$query->where('user_id', $user->id);
}])->orderBy('updated_at','desc')->paginate(config('hosptial-browsing-history.perPage'));
------------------------------
このファンクションの用途はログインユーザーの閲覧した病院情報の履歴を見るメソッドで、ユーザが病院をみるたびにHospitalUsersテーブルに病院情報が溜まっていきます。HospitalUsersテーブルには、
id,hospital_id, user_id,created_at,updated_at
とカラムがございます。

hospital.phpには以下のリレーションを設定しております。
----------------------
public function hospitalUsers(){
return $this->belongsToMany('App\Models\User','hospital_users','hospital_id','user_id');
}
----------------------

参考にしたサイトはこちらです。
https://www.wakuwakubank.com/posts/385-laravel-relation-2/

無知ですみませんが、よろしければご教授頂きたいです。よろしくお願いします。

[xenbeat]

> userの情報が紐づかずにhospitalテーブルの情報が全件取得されてしまいます。

本質問をクローズして新規で質問するか、本質問のタイトルおよび内容を書き直してください。

私の回答は、コメントいただいた質問（エラー解消後の質問）および要件に対しては無効となります。
今の質問タイトルおよび内容に対するものなので。

別にwithじゃなくてもjoinとかでも良いのですが、要件通りに関連データを抽出する方法がわからないというのは当初の問題とは全く別問題となります。
再質問をして要件、問題、ソースコードを明示してください。