PHP/Laravelで多対多リレーションの実装をしたい

Question

前提・実現したいこと

PHP(Laravel)でレシピサイトを作っています。
ログイン済ユーザーがレシピに対してコメントを投稿できる機能を実装中です。

前提として、
１、ユーザー情報があるUsersテーブル
２、レシピ情報があるRecipesテーブル
３、レシピに対してユーザーごとにコメントを格納するCommentsテーブル（中間テーブル）
を用意しています。

実現したいのは
・各レシピに対して、ログイン済ユーザーであればコメントを投稿できる。
・コメントの一覧はログイン状態に関係なく閲覧できる
・レシピはどのユーザーがコメントを投稿したか紐づけできる
という3つの処理ですが、現状レシピ情報画面（show.blade.php）を表示すると、下記のエラーになります。

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

上記機能を実装するために、どういった手順で修正していくのがよろしいでしょうか。
ご教授いただきたいです、よろしくお願いします。

ErrorException (E_ERROR)
Trying to get property 'id' of non-object (View: /home/ec2-user/environment/recipe-site/resources/views/recipes/show.blade.php)

該当のソースコード

users_table

<?php

use Illuminate\Support\Facades\Schema;
use Illuminate\Database\Schema\Blueprint;
use Illuminate\Database\Migrations\Migration;

class CreateUsersTable extends Migration
{
    public function up()
    {
        Schema::create('users', function (Blueprint $table) {
            $table->increments('id');
            $table->string('name');
            $table->string('email')->unique();
            $table->string('password');
            $table->rememberToken();
            $table->timestamps();
        });
    }
 /*以下略*/

recipes_table

<?php

use Illuminate\Support\Facades\Schema;
use Illuminate\Database\Schema\Blueprint;
use Illuminate\Database\Migrations\Migration;

class CreateRecipesTable extends Migration
{
    public function up()
    {
        Schema::create('recipes', function (Blueprint $table) {
            $table->increments('id');
            $table->integer('user_id')->unsigned()->index();
            $table->timestamps();
            
            //外部キー
            $table->foreign('user_id')->references('id')->on('users')->onDelete('cascade');
        });
    }
     /*以下略*/

user_comments.table

<?php

use Illuminate\Support\Facades\Schema;
use Illuminate\Database\Schema\Blueprint;
use Illuminate\Database\Migrations\Migration;

class CreateUserCommentsTable extends Migration
{

    public function up()
    {
        Schema::create('user_comments', function (Blueprint $table) {
            $table->increments('id');
            $table->integer('recipe_id')->unsigned()->index();
            $table->integer('user_id')->unsigned()->index();
            $table->string('content');
            $table->timestamps();
            
            //外部キー
            $table->foreign('recipe_id')->references('id')->on('recipes')->onDelete('cascade');
            $table->foreign('user_id')->references('id')->on('users')->onDelete('cascade');
        });
    }
 /*以下略*/

web.php

<?php

//レシピの詳細画面表示
Route::resource('recipes', 'RecipesController',['only' => ['show']]);

//コメントの表示機能
Route::get('comments','CommentsController@index');

//ログイン認証を必要とするルーティンググループ
Route::group( ['middleware' => ['auth']], function(){

    //コメント投稿のルーティング
    Route::resource('comments','CommentsController',['only' => ['store']]);
    
});

User.php

<?php

namespace App;

use Illuminate\Notifications\Notifiable;
use Illuminate\Foundation\Auth\User as Authenticatable;

class User extends Authenticatable
{
    //UserのRecipeを取得する
    public function recipes()
    {
        return $this->hasMany(Recipe::class);
    }
    
    //UserのCommentを取得する
    public function comments()
    {
        return $this->belongsToMany(Comment::class, 'user_comments', 'user_id', 'recipe_id')->withTimestamps();
    }
 
}

Recipe.php

<?php

namespace App;

use Illuminate\Database\Eloquent\Model;

class Recipe extends Model
{
    
    //RecipeがどのUserに紐づいているか
    public function user()
    {
        return $this->belongsTo(User::class);
    }
    
    public function comments()
    {
        return $this->belongsToMany(Comment::class, 'user_comments', 'user_id', 'recipe_id')
        ->withTimestamps();
    }
}

RecipesController.php

<?php

namespace App\Http\Controllers;

use Illuminate\Http\Request;

use App\Recipe;//追加

class RecipesController extends Controller
{
    public function show($id)
    {
        //レシピデータを取得
        $recipe = Recipe::find($id);
        //ログイン済ユーザー
        $user = \Auth::user();
        
        return view('recipes.show', [
            'recipe' => $recipe,
            'user' => $user,
        ]);
    }

CommentsController.php

<?php

namespace App\Http\Controllers;

use Illuminate\Http\Request;
use App\Recipe;
use App\User;

class CommentsController extends Controller
{
    
    //コメントの一覧表示処理
    public function index()
    {
        
        $data = [];

        //ログイン済かどうか
        if(\Auth::check()){
            //ログイン済ユーザー
            $user = \Auth::user();
            //ログインユーザーごとのコメントを格納
            $comments = $user->comments()->orderBy('created_at', 'desc');
            
            $data = [
                'user' => $user,
                'comments' => $comments,
            ];        
        }
        //コメントをレシピ詳細画面に渡す
        return view('recipes.show', $data);

    }
    //コメントの投稿処理
    public function store(Request $request)
    {
        
        $request->user()->comments()->create([
            'content' => $request->content,
        ]);
        
        return back();
    }

show.blade.php

@extends('layouts.app')

@section('content')
    <h1>レシピID：{{ $recipe->id }}の詳細ページ</h1>
    <!--ログインユーザーのみ投稿可能-->
    @if(Auth::id() == $user->id)
        {!! Form::open(['route' => 'comments.store']) !!}
            <div class="form-group">
                {!! Form::textarea('content',['class' => 'form-control']) !!}
                {!! Form::submit('投稿', ['class' => 'btn btn-primary btn-block']) !!}
            </div>
        {!! Form::close() !!}
    @endif
        <!--投稿の表示処理-->
        @if(count($comments) > 0)
            @foreach($comments as $comment)
                    <p>{!! $comment->content !!}</p>
            @endforeach
        @endif
@endsection

試したこと

多対多リレーションはこちらを参考にしました。
https://readouble.com/laravel/5.5/ja/eloquent-relationships.html#updating-many-to-many-relationships

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

PHP 7.2.22
Laravel 5.5

Answer 1

エラーメッセージについて

 @if(Auth::id() == $user->id)

となるとおもいますが、ログインしてなければ $user は null なのでエラーとなる。

 @if(Auth::user())

やら

 @if(Auth::user() != null)

でログイン判定すべきかと

---

他対他

create table roles ( id int /*略*/ )
create table users ( id int /*略*/ )
create table user_roles ( user_id int, role_id int ./*略*/ )

という中間テーブルをはさむ形式が[他対他] の形式となります。

モデル側は「公式ドキュメント」に書かれてる通りの設定になります。

マイグレーション側は以下のような形式になるかと

        Schema::create('roles', function (Blueprint $table) {
            $table->increments('id')/*略*/

        Schema::create('users', function (Blueprint $table) {
            $table->increments('id')/*略*/

        Schema::create('user_roles', function (Blueprint $table) {
            $table->integer('user_id')/*略*/
            $table->integer('role_id')/*略*/
            $table->foreign('user_id')/*略*/
            $table->foreign('role_id')/*略*/

Comments

[退会済みユーザー]

recipes に対して、comments は多対多じゃなくて、一対多であるはず...。

[退会済みユーザー]

１つのレシピに対して複数のユーザーはコメントできる（１ｖ多）
1人のユーザーは複数のレシピに対してコメントできる（１ｖ多）

なぜか　ORMでは上記を ManyToMany （多対多）という
用語にあやまりがあるから仕方がないんだよね・・・・

一人のユーザーは一つのレシピにたいして一つのコメントをできるなので
１：１：１　なはずなんだけど

[kyontama]

>Kosuke_Shibuyaさん
一対多と多対多を混在してました。ご指摘ありがとうございます！

>asahina1979さん
「一人のユーザーは一つのレシピにたいして一つのコメントをできる」という考えだから、1対多ということですね。
ありがとうございます。

また、体調不良で寝込んでました。返信が遅くなり大変申し訳ありません。

Answer 2

自分ならこうしますという形で、回答します。

ER

migrations

<?php
use Illuminate\Database\Migrations\Migration;
use Illuminate\Database\Schema\Blueprint;
use Illuminate\Support\Facades\Schema;
class CreateRecipesTable extends Migration
{
   /**
    * Run the migrations.
    *
    * @return void
    */
   public function up()
   {
       Schema::create('recipes', function (Blueprint $table) {
           $table->bigIncrements('id');
           $table->string('name');
           $table->unsignedBigInteger('created_by');
           $table->timestamps();
           $table->foreign('created_by')->references('id')->on('users');
       });
   }
   /**
    * Reverse the migrations.
    *
    * @return void
    */
   public function down()
   {
       Schema::dropIfExists('recipes');
   }
}

<?php
use Illuminate\Database\Migrations\Migration;
use Illuminate\Database\Schema\Blueprint;
use Illuminate\Support\Facades\Schema;
class CreateCommentsTable extends Migration
{
   /**
    * Run the migrations.
    *
    * @return void
    */
   public function up()
   {
       Schema::create('comments', function (Blueprint $table) {
           $table->bigIncrements('id');
           $table->unsignedBigInteger('recipe_id');
           $table->string('content');
           $table->unsignedBigInteger('created_by');
           $table->timestamps();
           $table->foreign('recipe_id')->references('id')->on('recipes');
           $table->foreign('created_by')->references('id')->on('users');
       });
   }
   /**
    * Reverse the migrations.
    *
    * @return void
    */
   public function down()
   {
       Schema::dropIfExists('comments');
   }
}

models

<?php
namespace App;
use Illuminate\Database\Eloquent\Model;
use Illuminate\Database\Eloquent\Relations\BelongsTo;
use Illuminate\Database\Eloquent\Relations\HasMany;
class Recipe extends Model
{
   protected $fillable = [
       'name',
       'createdBy'
   ];
   /**
    * @return HasMany
    */
   public function comments()
   {
       return $this->hasMany(Comment::class);
   }
   /**
    * @return BelongsTo
    */
   public function createdBy()
   {
       return $this->belongsTo(User::class, 'created_by');
   }
}

<?php
namespace App;
use Illuminate\Database\Eloquent\Model;
use Illuminate\Database\Eloquent\Relations\BelongsTo;
class Comment extends Model
{
   protected $fillable = [
       'content',
       'createdBy'
   ];
   /**
    * @return BelongsTo
    */
   public function recipe()
   {
       return $this->belongsTo(Recipe::class);
   }
   /**
    * @return BelongsTo
    */
   public function createdBy()
   {
       return $this->belongsTo(User::class, 'created_by');
   }
}

controllers

<?php
namespace App\Http\Controllers;
use App\Recipe;
use Illuminate\View\View;
class RecipeController extends Controller
{
   /**
    * @return View
    */
   public function index()
   {
       $recipes = Recipe::query()
           ->with(['comments', 'createdBy'])
           ->paginate();
       return view('recipes.index', compact('recipes'));
   }
   /**
    * @param Recipe $recipe
    * @return View
    */
   public function show(Recipe $recipe)
   {
       return view('recipes.show', compact('recipe'));
   }
}

<?php
namespace App\Http\Controllers;
use App\Comment;
use App\Recipe;
use Illuminate\Http\RedirectResponse;
use Illuminate\Http\Request;
use Illuminate\Support\Facades\Auth;
class CommentController extends Controller
{
   /**
    * @param Request $request
    * @param Recipe $recipe
    * @return RedirectResponse
    */
   public function store(Request $request, Recipe $recipe)
   {
       $recipe->comments()->save(new Comment([
           'content' => $request->get('content'),
           'created_by' => Auth::id()
       ]));
       return redirect()->route('recipes', compact('recipe'));
   }
}

routing

Route::resource('recipes', 'RecipeController');
Route::post('comments/{recipe}', 'CommentController@store')->name('comments.store');

blade

@section('content')
   <h1>レシピID：{{ $recipe->id }}の詳細ページ</h1>
   @auth()
       {!! Form::open(['route' => 'comments.store']) !!}
       <div class="form-group">
           {!! Form::textarea('content',['class' => 'form-control']) !!}
           {!! Form::submit('投稿', ['class' => 'btn btn-primary btn-block']) !!}
       </div>
       {!! Form::close() !!}
   @endauth
   @foreach($recipe->comments as $comment)
       <p>{{ $comment->content }}</p>
   @endforeach
@endsection

Comments

[退会済みユーザー]

質問者はノーリアクションかよ

[kyontama]

>Kosuke_Shibuyaさん

すいません、インフルで寝込んでました。
大変申し訳ありません。

[kyontama]

>Kosuke_Shibuyaさん

再度回答いただき、誠にありがとうございます。
テーブル間の関係を正しく理解ぜず、多対多で考えてしまったのがミスのそもそもの原因でした。

Kosuke_ShibuyaさんのER図の通りにテーブル設定をしなおしたところ、非常にシンプルになりました。
改めて感謝申し上げます。

また病気で寝込んでいたといえど、いただいた回答にリアクションできずに申し訳ありませんでした。

[退会済みユーザー]

いえいえ。
ここ最近、回答寄せてもリアクションしない人が増えて、このサイトに対しては良い感情を持っていないんですよね・・・。
病気なら仕方ないなと。お大事に。

[kyontama]

そうだったんですね・・・
たしかに回答していただいてるのに良い感情は持てなくなりますよね。

私も今後は気を付けたいと思います、体調のお気遣いまでありがとうございました！