質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.36%
Laravel

LaravelとはTaylor Otwellによって開発された、オープンソースなPHPフレームワークです。Laravelはシンプルで表現的なシンタックスを持ち合わせており、ウェブアプリケーション開発の手助けをしてくれます。

Q&A

0回答

1380閲覧

Laravel8 ブックマーク機能を実装したが、コードを洗練させたい

Y0shito

総合スコア3

Laravel

LaravelとはTaylor Otwellによって開発された、オープンソースなPHPフレームワークです。Laravelはシンプルで表現的なシンタックスを持ち合わせており、ウェブアプリケーション開発の手助けをしてくれます。

0グッド

0クリップ

投稿2021/11/10 15:20

編集2021/12/13 06:08

個人開発で掲示板を作っております。
ブックマーク機能実装につき、ブックマークしているかどうかで表示を変えるボタンも実装したのですが、後述の問題を抱えており、もっと賢い実装はないかと思い質問させていただきます。

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文がロジックにあたらないか(正直に言うとどこからロジックにあたるのか分かっていない)

以上、よろしくお願いいたします。

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

まだ回答がついていません

会員登録して回答してみよう

アカウントをお持ちの方は

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.36%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問