同じ投稿を２回する時に投稿ボタンを表示できないようにしたい。

Question

前提・実現したいこと

laravelで作ったアプリでログインしたユーザがゲームのレビュー投稿をできるのですが、同じゲームのレビュー投稿を２回目から投稿ボタンを表示しないようにしたい。
ここに質問の内容を詳しく書いてください。

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

発生している問題としては、同じユーザが同じゲームで２回投稿ができてしまうこと。

該当のソースコード

<?php

namespace App\Http\Controllers;

use Illuminate\Http\Request;

use App\User;

use App\Game;

use App\Review;

class ReviewsController extends Controller
{
    public function index(Request $request)
    {
        $data = [];
        if (\Auth::check()) { 
            $user = User::orderBy('id', 'desc')->paginate(10);
            
            $games = Game::orderBy('id', 'desc')->paginate(10);
            
            $reviews = Review::orderBy('id', 'desc')->paginate(10);
            
            
            $data = [
                'user' => $user,
                'games' => $games,
                'reviews' => $reviews,
            ];
        }
        return view('users.show', $data);
    }
    
    
    public function create($id)
    {
        $review = new Review;
        
        $game = Game::findOrFail($id);
        
        
        return view('reviews.score', [
            'review' => $review,
            'game' => $game,
        ]);
    }
    
    public function store(Request $request)
    {   
        //dd($reviews);
        $request->validate([
            'score' => 'required|max:255',
            'content' => 'required|max:255',
        ]);
        $review = new Review;
        $review->user_id = \Auth::id();
        $review->content = $request->content;
        $review->score = $request->score;
        $review->game_id = $request->game_id;
        $review->save();

        return redirect('myreviews/{user}');;
    }
    public function destroy($id)
    {
        $review = \App\Review::findOrFail($id);
        if (\Auth::id() === $review->user_id) {
            $review->delete();
        }
        return back();
    }
}

@extends('layouts.app')

@section('content')

    <h1>レビュー投稿</h1>
{!! Form::open(['route' => 'reviews.post', 'method' => 'post', ]) !!}
    {{ csrf_field() }}
    <div>
        {{ Form::label('ゲーム名') }}
        {{ Form::hidden('game_id', $game->id,  ['id' => 'game_id']) }}
        <p>{{ $game->name }}</p>
    </div>
    <div>
        <label>レビュー</label></label><br>
        <textarea name= "content" rows="10" cols="130"></textarea>
        <div class="form-check">
            {{ Form::radio('score', '1', true, ['id' => 'exampleRadios', 'class' => 'form-check-input']) }}
            {{ Form::label('score-one', '☆', ['class' => 'form-check-label']) }}
        </div>
        <div class="form-check">
            {{ Form::radio('score', '2', true, ['id' => 'exampleRadios', 'class' => 'form-check-input']) }}
            {{ Form::label('score-two', '☆☆', ['class' => 'form-check-label']) }}
        </div>
        <div class="form-check">
            {{ Form::radio('score', '3', true, ['id' => 'exampleRadios', 'class' => 'form-check-input']) }}
            {{ Form::label('score-three', '☆☆☆', ['class' => 'form-check-label']) }}
        </div>
        <div class="form-check">
            {{ Form::radio('score', '4', true, ['id' => 'exampleRadios', 'class' => 'form-check-input']) }}
            {{ Form::label('score-four', '☆☆☆☆', ['class' => 'form-check-label']) }}
        </div>
        <div class="form-check">
            {{ Form::radio('score', '5', true, ['id' => 'exampleRadios', 'class' => 'form-check-input']) }}
            {{ Form::label('score-five', '☆☆☆☆☆', ['class' => 'form-check-label']) }}
        </div>
        <div>
            {!! Form::submit('送信', ['class' => "btn btn-primary btn-block"]) !!}
        </div>
    </div>
{!! Form::close() !!}
@endsection

<?php

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

class CreateReviewsTable extends Migration
{
    /**
     * Run the migrations.
     *
     * @return void
     */
    public function up()
    {
        Schema::create('reviews', function (Blueprint $table) {
            $table->bigIncrements('id');
            $table->unsignedBigInteger('user_id');
            $table->unsignedBigInteger('game_id');
            $table->integer('score');
            $table->string('content');
            $table->timestamps();
            
            $table->foreign('user_id')->references('id')->on('users')->onDelete('cascade');
           
            $table->foreign('game_id')->references('id')->on('games')->onDelete('cascade');
            
            $table->unique(['user_id', 'game_id']);
        });
    }

    /**
     * Reverse the migrations.
     *
     * @return void
     */
    public function down()
    {
        Schema::dropIfExists('review');

    }
}

試したこと

ここに問題に対して試したことを記載してください。

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

AWSを使用しています。

Answer 1

投稿者名をデータ上に持っているのでしたら、コントローラで事前にチェックしてフラグなり何なり渡してView側で表示切替すれば良いのではと

Comments

[natu494]

すみません理解力が足りていなくて申し訳ないのですが、createコントローラに記載すればいいのでしょうか？お時間をとらせてすみませんが、回答お願いします。

[natu494]

m.ts10806様のプロフフィールを見て自分の利益だけを考えてしまい、申し訳ございませんでした。自分が初心者だからgiveしてもらって当然のように質問してしまいました。このような質問の仕方では自分の成長が見られないと思ったのでm.ts10806様にベストアンサーを渡します。教えてもらったことを参考に自分で解決します。本当に失礼な質問をしてしまい申し訳ございませんでした。

[m.ts10806]

いえ、プロフィールのコメントにそこまで神経質にならなくとも良いと思います。「考え方」は一朝一夕では身に付きませんしね。
返事遅れたのは単にこどもと外出していて見れなかったからで、怒ったり呆れたりしているわけではありません。

今回は「表示時にそもそもボタンを表示させない」というところがポイントです。
既に投稿したかどうかデータで確認できる設計になっていれば、あとは如何様にもできるという話ですね。
DB設計次第なので、場合により設計から見直す必要はあります。

[m.ts10806]

ついでに言及しておきますと、QAサイトでの「Give and Take」には「質問者の成長姿が見れる」というところもあると思います。きちんと理解が得られたとか考え方が身に付いたとか、そういう姿が見てとれれば、こちらも満足します。