laravelで掲示板ごとのメッセージを取得したい

Question

前提・実現したいこと

タイトルの通り掲示板ごとのメッセージを取得したいです。

フリマサイトを作成しており、商品を買ったユーザーのidが下記にあるbulletin_boardsテーブルのbuy_userカラムに挿入され、商品を売ったユーザーのidがsale_userカラムに追加される仕組みになっています。

bulletin_boardsテーブル

+------------+-----------------+------+-----+---------+----------------+
| Field      | Type            | Null | Key | Default | Extra          |
+------------+-----------------+------+-----+---------+----------------+
| id         | bigint unsigned | NO   | PRI | NULL    | auto_increment |
| sale_user  | bigint unsigned | NO   |     | NULL    |                |
| buy_user   | bigint unsigned | NO   |     | NULL    |                |
| product_id | bigint unsigned | NO   | MUL | NULL    |                |
| created_at | timestamp       | YES  |     | NULL    |                |
| updated_at | timestamp       | YES  |     | NULL    |                |
| deleted_at | timestamp       | YES  |     | NULL    |                |
+------------+-----------------+------+-----+---------+----------------+

messagesテーブル

+-------------------+-----------------+------+-----+---------+----------------+
| Field             | Type            | Null | Key | Default | Extra          |
+-------------------+-----------------+------+-----+---------+----------------+
| id                | bigint unsigned | NO   | PRI | NULL    | auto_increment |
| bulletin_board_id | bigint unsigned | NO   | MUL | NULL    |                |
| to_user           | bigint unsigned | NO   |     | NULL    |                |
| from_user         | bigint unsigned | NO   |     | NULL    |                |
| message           | varchar(255)    | NO   |     | NULL    |                |
| send_date         | datetime        | NO   |     | NULL    |                |
| created_at        | timestamp       | YES  |     | NULL    |                |
| updated_at        | timestamp       | YES  |     | NULL    |                |
| deleted_at        | timestamp       | YES  |     | NULL    |                |
+-------------------+-----------------+------+-----+---------+----------------+

発生している問題・エラーメッセージ

複数の掲示板ごとのメッセージを取得することができません。

該当のソースコード

<?php

namespace App\Http\Controllers;

use App\Models\User;
use App\Models\Message;
use Illuminate\Http\Request;
use App\Models\BulletinBoard;
use Illuminate\Support\Facades\Auth;

class MyPageController extends Controller
{
  /**
   * Display a listing of the resource.
   *
   * @return \Illuminate\Http\Response
   */
  public function index(Request $request)
  {
    $products = User::find(Auth::id())->products()->get();

    $bulletinBoards = BulletinBoard::where('sale_user', Auth::id())->orWhere('buy_user', Auth::id())->get();

    foreach ($bulletinBoards as $bulletinBoard) {
      $messages = Message::where('bulletin_board_id', $bulletinBoard->id)->orderBy('created_at', 'desc')->get();
    }

    return view('myPage.index', compact(
      'messages',
      'products',
    ));
  }
}

試したこと

$bulletinBoards = BulletinBoard::where('sale_user', Auth::id())->orWhere('buy_user', Auth::id())->get();

ログインしているユーザーがやり取りしている掲示板のデータを取得するために上記のコードを記述しました。

取得した掲示板ごとのメッセージを取得するために下記のコードを記述しました。

 foreach ($bulletinBoards as $bulletinBoard) {
      $messages = Message::where('bulletin_board_id', $bulletinBoard->id)->orderBy('created_at', 'desc')->get();
    }

ループ内で$messagesをLog::debug($messages)で確認すると、ログインしているユーザーがやりとりしている掲示板のデータに紐づくメッセージを全て取得できたのですが、ループの外でview側へ$messagesをreturnするとループの最後の掲示板メッセージしかreturnされません。（foreachから出るとポインタは最後の要素のみを指し示すためだと思います）

なので、foreach内で$messagesをreturnしようと下記のコードを試すと、最初の掲示板メッセージしか取得できませんでした（最初のループでreturnしているためだと思います）

foreach ($bulletinBoards as $bulletinBoard) {
      $messages = Message::where('bulletin_board_id', $bulletinBoard->id)->orderBy('created_at', 'desc')->get();

return view('myPage.index', compact(
      'messages',
      'products',
    ));

補足情報（FW/ツールのバージョンなど）

laravel6.8
dockder
mac

ここにより詳細な情報を記載してください。

Answer 1

ループ内で$messagesをLog::debug($messages)で確認すると、ログインしているユーザーがやりとりしている掲示板のデータに紐づくメッセージを全て取得できたのですが、ループの外でview側へ$messagesをreturnするとループの最後の掲示板メッセージしかreturnされません。（foreachから出るとポインタは最後の要素のみを指し示すためだと思います）

$messages毎回上書きしてるのだから当然最後にデータになる

なので、foreach内で$messagesをreturnしようと下記のコードを試すと、最初の掲示板メッセージしか取得できませんでした（最初のループでreturnしているためだと思います）

foreach ($bulletinBoards as $bulletinBoard) {
      $messages = Message::where('bulletin_board_id', $bulletinBoard->id)->orderBy('created_at', 'desc')->get();

return view('myPage.index', compact(
      'messages',
      'products',
    ));

foreachのループ内でreturn viewしてるので当然そうなる……原因わかっててここまでソース記載できるのでしたら解決できそうな気もしますが、

$messages = [];
foreach ($bulletinBoards as $bulletinBoard) {
      $messages[] = Message::where('bulletin_board_id', $bulletinBoard->id)->orderBy('created_at', 'desc')->get();
}
return view('myPage.index', compact(
      'messages',
      'products',
    ));

じゃあ、だめなのかな…(配列にしたのでView側も変更いりますけど)

※View変えたくない場合は

$messages = Collection::make([]);
foreach ($bulletinBoards as $bulletinBoard) {
      $messages = $messages->merge(Message::where('bulletin_board_id', $bulletinBoard->id)->orderBy('created_at', 'desc')->get());
}
return view('myPage.index', compact(
      'messages',
      'products',
    ));

Comments

[youkiti0208]

教えていただいた通りにコードを記述すると解決しました。
ありがとうございました。

ループの前に配列かコレクションで変数を作っておくとループを抜けたあともループ内の値を全て変数に格納した状態を保つことができるのですね。

大変勉強になりました。

Answer 2

$messages = Collection::make([]);
    foreach ($bulletinBoards as $bulletinBoard) {
      $messages[] = Message::where('bulletin_board_id', $bulletinBoard->id)->orderBy('created_at', 'desc')->get();
    }

ループの前にコレクションで変数を作っておくとループを抜けた後もループ中に格納した変数の値が全て保持されるようになりました。