個人開発で掲示板を作っております。
ブックマーク機能実装につき、ブックマークしているかどうかで表示を変えるボタンも実装したのですが、後述の問題を抱えており、もっと賢い実装はないかと思い質問させていただきます。
Controller側でブックマークされているかどうかの判定し、値をViewへ投げ、View側で受け取った値を元にボタンを表示、という流れです。
Controllerの処理は
①ログインしており、かつ自身が書いた記事でなければ次へ、反すれば「ブックマーク不可」のボタン
②この記事がされているブックマーク内に自身のuser_idがあれば「ブックマークを外す」ボタン、なければ「ブックマークする」ボタン
...になります。
PHP
1// app/Models/Article 2 3public function bookmark() 4 { 5 return $this->hasMany(Bookmark::class); 6 }
PHP
1// app/Http/Controllers/ArticlesController 2 3public function showArticles($articleId) 4{ 5 $userId = Auth::id(); 6 7 $article = Article::find($articleId); 8 $article->timestamps = false; 9 $article->increment('views'); 10 11 if (Auth::check() and !($article->user_id === $userId)) { // ① 12 if ($article->bookmark()->where('user_id', $userId)->exists()) { // ② 13 $button = 'bookmarked'; 14 } else { 15 $button = 'notBookmark'; 16 } 17 } else { 18 $button = 'disable'; 19 } 20 return view('articles', compact('article', 'button')); 21}
HTML
1<!-- resources/views/articles.blade 一部改変あり --> 2 3<form method="POST"> 4 @csrf 5 @if ($button === 'disable') 6 <button disable>ブックマーク</button> 7 @elseif($button === 'bookmarked') 8 <button name="article_id" value="{{ $article->id }}" formaction="{{ route('bookmarkRemove') }}"">ブックマーク中</button> 9 @else 10 <button name="article_id" value="{{ $article->id }}" formaction="{{ route('bookmarkAdd') }}">ブックマーク</button> 11 @endif 12</form>
問題点/質問点
・そもそも以上の考え方(Controller側で判定しViewで表示)がベターか
・ネストしたif文など複雑化。もっとシンプルに出来ないか
・View側のif文がロジックにあたらないか(正直に言うとどこからロジックにあたるのか分かっていない)
以上、よろしくお願いいたします。
あなたの回答
tips
プレビュー