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

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

ただいまの
回答率

88.64%

[Laravel]ルーティングのパラメータが正確な値を渡さず困っています。

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 316

siophp

score 23

環境
Laravel5.8
PHP7.2
Nigix

データベースからデータを取得し、forelseで一覧表示するプログラムを作っています。
一部分に特定のプロパティが存在するか否かで、URLと表示を変える分岐があります。$post->id、$post->liked_idをパラメータとしてルーティングに渡しています。(Twitterのいいね機能のようなものです)

@forelse($posts as $post)
      <div class="card">
        <div class="card-body">
          <div class="card-text">
            <a href="{{url('posts/'.$post->id)}}">{{$post->body}</a>
            <p>{{$post->updated_at}}</p>
          </div>
        </div>
        // 省略、以下問題のコード
              @auth
                @if(empty($post->liked_id))
                  <a href="{{url('posts/'.$post->id.'/likes')}}" class=""
                  onClick="event.preventDefault();
                  document.getElementById('like-store-form').submit();">
                    <i class="far fa-heart"></i>
                    {{$post->likes->count()}}
                  </a>
                  <form id="like-store-form" action="{{url('posts/'.$post->id.'/likes')}}" method="POST">
                    @csrf
                    @method('POST')
                  </form>
                @else
                  <a href="{{url('posts/'.$post->id.'/likes/'.$post->liked_id)}}" class=""
                    onClick="event.preventDefault();
                    document.getElementById('like-delete-form').submit();">
                    <i class="fa fa-heart"></i>
                    {{$post->likes->count()}}
                  </a>
                  <form id="like-delete-form" action="{{url('posts/'.$post->id.'/likes/'.$post->liked_id)}}" method="post">
                    @csrf
                    @method('DELETE')
                  </form>
                @endif
              @else
                  <i class="far fa-heart"></i>
                  {{$post->likes->count()}}
              @endauth
            </span>
          </div>
        </div>
      </div>
   //省略
    @endforelse
    {{$posts->links()}}
Route::post('/posts/{post_id}/likes','LikeController@store');
Route::delete('/posts/{post_id}/likes/{like_id}','LikeController@destroy');

しかし、ブラウザに表示されるURLは変化しているのに、ルーティングには本来のものとは違う値が渡されているようです。
具体的に言えば、どの投稿のURLにアクセスしても、ページの一番上の投稿のpost_idがパラメータとして渡されて処理が実行されているようです。

これは一体何が起きているのでしょうか。恥ずかしいことに検索ワードすらわからず、「ルーティング 違う値」などをググっても解決方法が出てきませんでした。よろしければアドバイスいただけると幸いです。

備考

・別のページ(forelseによる一覧表示を行わないページ)から同じURLにアクセスしたところ、正確なパラメータが渡され意図した通りの処理を実行しました。
・dd();でコントローラ内を確認してみたのですが、コントローラに引数として渡したときには、違うパラメータが渡されていました。

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

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

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

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 2

checkベストアンサー

+3

id重複しているためです。
画面内に同一IDは指定できません。
JavaScriptではIDを指定してsubmitしているので、「ページの一番上の投稿」が常に送信されているものと思われます。
IDをつけるなら個々のpost_idをつけるなど工夫が必要です。

それか、aタグではなくbuttonのsubmitにして<form>内に入れるとか(こちらのほうがスッキリすると思います)

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/08/07 14:39 編集

    回答ありがとうございます…buttonに直したところ意図通りに動きました。
    別のページのコードをそのままコピペして不具合を生じさせていました。

    キャンセル

  • 2019/08/07 14:42

    解決されたようで何よりです。
    PHPからHTML出力してそれをJavaScriptで扱うとなると頭の中や机上で動作をさせるのは結構難しくなってくるので、
    「PHPのコード」と「出力されたHTMLのコード」は全く切り分けて考えたほうがいいかもしれません。
    つまり、phpで出力処理を組んだら、そのあとはブラウザから画面を確認するというよりも
    「正しくHTMLが出力されたか」を「ソースを表示」で確認するようにするとか。

    モックとかで静的HTMLを組んだうえでPHPで動的に出力する場合、
    やはり「見てくれ」よりも「出力されたHTMLソース」を確認するのが適切な手段となります。

    キャンセル

  • 2019/08/07 14:48

    ありがとうございます…以後出力されたHTMLソースの確認も行います。

    キャンセル

+3

getElementByIdで常に一番最初のformが引かれてるのでは?

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

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

  • ただいまの回答率 88.64%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る