<Laravel> ミニマムなSNSサイトでコメント機能の実装

Question

前提

勉強のためにLaravel 5.1 LTSでシンプルなSNSサイトを開発しています。

実装した主な機能

• ユーザー登録・ログイン認証
• フォロー機能
• 投稿機能
• お気に入り機能
• ユーザー一覧表示
• フォロー・フォロワー一覧表示
• 投稿一覧表示
• お気に入り一覧表示

実現したいこと

現在、投稿に対してコメントする機能を実装しようとしています。
DBへの書き込みには $fillable を用いて実装したいと思っています。

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

一先ず、ページ上に設置したフォームからコメントを送信して、それをDBに書き込めるようにしようとしているところですが、これが実装できないため、コメントを表示するための view ファイルはまだ仮の状態です。

フォームからコメントを送信すると下記のエラーメッセージが出ます。

SQLSTATE[23000]: Integrity constraint violation: 1452 Cannot add or update a child row: a foreign key constraint fails (`microposts`.`comments`, CONSTRAINT `comments_user_id_foreign` FOREIGN KEY (`user_id`) REFERENCES `users` (`id`) ON DELETE CASCADE) (SQL: insert into `comments` (`comment`, `updated_at`, `created_at`) values (コメントテスト, 2018-05-03 19:50:00, 2018-05-03 19:50:00))

試したこと

エラーはDBの外部キー制約関係の問題のようですので、試しに comments テーブル作成時に定義した全ての外部キー制約を dropForeign したところ、comment テーブルの comment, created_at, updated_at カラムには期待通りの値が書き込まれましたが、それ以外で必要な user_id と micropost_id カラムに値が書き込まれません。

自信がない部分等

リレーション関連と CommentsController@store での処理に自信がありませんが、具体的にどうすれば良いのかは分かりません。
ご教授よろしくお願い致します。

現状のコード

こちらからGitHub上で全てのファイルをご確認頂けます。

コメント用テーブル
2018_04_29_180548_create_comments_table.php

<?php

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

class CreateCommentsTable extends Migration
{

    public function up()
    {
        Schema::create('comments', function (Blueprint $table) {
            $table->increments('id');
            $table->integer('user_id')->unsigned()->index();
            $table->integer('micropost_id')->unsigned()->index();
            $table->string('name'); // しばらくは使う予定無し
            $table->string('email'); // しばらくは使う予定無し
            $table->text('comment');
            $table->boolean('approved'); // しばらくは使う予定無し
            $table->timestamps();
            
            $table->foreign('user_id')->references('id')->on('users')->onDelete('cascade');
            $table->foreign('micropost_id')->references('id')->on('microposts')->onDelete('cascade');
        });
    }

    public function down()
    {
        Schema::drop('comments');
    }
}

コメント用のモデル
Comment.php

<?php

namespace App;

use Illuminate\Database\Eloquent\Model;

class Comment extends Model
{
    public function micropost()
    {
        return $this->belongsTo(Micropost::class);
    }
}

投稿用のモデル
Micropost.php

<?php

namespace App;

use Illuminate\Database\Eloquent\Model;

class Micropost extends Model
{
    protected $fillable = ['content', 'user_id'];
    
    public function user()
    {
        return $this->belongsTo(User::class);
    }
    
    public function favorites() // リレーションの定義
    {
        return $this->belongsToMany(User::class, 'favorites', 'micropost_id', 'user_id')->withTimestamps();
    }
    
    public function is_favorited_by($user_id)
    {
        return $this->favorites()->where('user_id', $user_id)->exists();
    }
    
    public function comments()
    {
        return $this->hasMany(Comment::class);
    }
}

コメント用のコントローラー
CommentsController.php

<?php

namespace App\Http\Controllers;

use Illuminate\Http\Request;

use App\Http\Requests;
use App\Http\Controllers\Controller;
use App\User;
use App\Micropost;
use App\Comment;

class CommentsController extends Controller
{
// その他アクション省略
    public function store(Request $request)
    {
        // $this->validate($request, [
        //   'content' => 'required|max:255',
        // ]);
        
        // 最終的には $fillable で実装
        $comment = new Comment;
        $comment->comment = $request->comment;
        $comment->save();
        
        return redirect()->back();
    }

    public function show($micropost_id)
    {
        $micropost = Micropost::find($micropost_id);
        $user = User::where('id', $micropost->user_id)->first();
        
        return view('comments.comments', compact('micropost', 'user'));
    }
// その他アクション省略
}

コメント 投稿フォーム・表示用の view
comments.blade.php

@extends('layouts.app')

@section('content')
<div class="media-left">
    <img class="media-object img-rounded" src="{{ Gravatar::src($user->email, 50) }}" alt="">
</div>
<div class="media-body">
    <div>
        {!! link_to_route('users.show', $user->name, ['id' => $user->id]) !!} <span class="text-muted">posted at {{ $micropost->created_at }}</span>
    </div>
    <div>
        <p>{!! nl2br(e($micropost->content)) !!}</p>
    </div>
    <div class="post-footer">
        <div class="btn-group">
            @include('favorites.favorite_button')
        </div>
        <div class="btn-group">
            @if (Auth::user()->id == $micropost->user_id)
                {!! Form::open(['route' => ['microposts.destroy', $micropost->id], 'method' => 'delete']) !!}
                    {!! Form::submit('Delete', ['class' => 'btn btn-danger btn-xs']) !!}
                {!! Form::close() !!}
            @endif
        </div>
        <!--ここにマージンを挿入-->
        <div class="row">
            <aside class="col-xs-3">
                {!! Form::open(['route' => 'comments.store']) !!}
                    <div class="form-group">
                        {!! Form::textarea('comment', old('comment'), ['class' => 'form-control', 'rows' => '5']) !!}
                    </div>
                    {!! Form::submit('Post comment', ['class' => 'btn btn-primary btn-block']) !!}
                {!! Form::close() !!}
            </aside>
        </div>
        <!--ここに境界線を挿入-->
        
        <!--ここにコメント一覧をループで挿入-->
        
    </div>
</div>
@endsection

Answer 1

CommentsController@storeでuser_idとmicropost_idを指定してないからでは。

$comment->user_id = $request->user()->id;

view側に$micropostのidも隠しフィールドで入れるかルーティングで$micropost_idを入れる。

もしくは関連したモデルの挿入
https://readouble.com/laravel/5.6/ja/eloquent-relationships.html#inserting-and-updating-related-models
この辺は若干ややこしいので試して覚えるしかない。

$micropost->comments()->create([
    'comment' => $request->comment,
    'user_id' => $request->user()->id,
    'micropost_id' => $micropost_id,
]);

Comments

[KIYZ]

ご回答誠にありがとうございます。

CommentsController@storeで、

下記のようにした場合は同ファイルで micropost が undefined になり、

①
$micropost->comments()->create([
'comment' => $request->comment,
'user_id' => $request->user()->id,
'micropost_id' => $micropost_id,
]);

②
$comment = $micropost->comments()->create([
'comment' => $request->comment,
'user_id' => $request->user()->id,
'micropost_id' => $micropost_id,
]);

下記のようにした場合は質問時と同じエラーが出てしまいます。

③
$comment = new Comment;
$comment->comment = $request->comment;
$comment->user_id = $request->user()->id;
$comment->save();

[Lulucom]

@KIYZ

> 下記のようにした場合は同ファイルで micropost が undefined になり、

おそらく以下のようなコードが抜けているのではないでしょうか。

comments.blade.phpのform内で
{{!! Form::hidden('micropost_id', $micropost->id) !!}}

CommentsController@storeで
$micropost_id = $request->micropost_id;

[KIYZ]

ご指摘ありがとうございます。

comments.blade.php の フォームを生成する部分を

{!! Form::open(['route' => ['comments.store', $micropost->id]]) !!}

として、

CommentsController@storeで

Comment::create([
'user_id' => $request->user()->id,
'micropost_id' => $request->micropost_id,
'comment' => $request->comment,
]);

とするとフォームから送信したデータを comments テーブルの各カラムに書き込むことができました。

CommentsController@store で問題だったのはDBに書き込みたい情報を明示していなかったことで、

comments.blade.php で問題だったのはフォームからコントローラーのアクションに対して "コメントされる micropost の id" を送信していなかったこと、という認識で正しいのでしょうか？

よろしくお願いします。

[Lulucom]

> という認識で正しいのでしょうか？
はい。

[KIYZ]

大変勉強になりました。
ありがとうございます。

Answer 2

kawaxさんと Lulucom さんにご教授頂いたことを組み合わせて実装することができたのでここにまとめさせて頂きます。

以下、コメント送信フォームから送信したデータを comments テーブルに書き込む方法です。
(DBに保存したコメントデータを取得し、ページに表示させることはまだ行っていません。)

comments.blade.php

{!! Form::open(['route' => ['comments.store', $micropost->id]]) !!}

CommentsController@store

    public function store(Request $request)
    {
        // バリデーション
        
        Comment::create([
            'user_id' => $request->user()->id,
            'micropost_id' => $request->micropost_id,
            'comment' => $request->comment,
        ]);
        
        return redirect()->back();
    }

参考にしたドキュメント

CommentsController@store は、上記ドキュメントのようにして実装することも可能でした。

    public function store(Request $request, $id)
    {
        // バリデーション
        
        $micropost = Micropost::find($id);

        $micropost->comments()->create([
            'user_id' => $request->user()->id,
            'micropost_id' => $request->micropost_id,
            'comment' => $request->comment,
        ]);
        
        return redirect()->back();
    }