特定の投稿詳細ページに遷移できない原因を解決したい

Question

投稿詳細ページに遷移したい

現在投稿詳細ページに遷移し、実装を進めようとしておりましたがCouldn't find Tag with 'id'=3 とエラーが起きてしまいます。

1つ目の投稿では詳細ページに遷移できましたが、id=3の投稿ページには遷移できない現象が起きております。

確認してみたところ、パラメーターの取得はできていると表示されておりました。
https://gyazo.com/9d3f1c443c51613223c28f6e8f35a238

私の認識だと、投稿テーブルと紐付いているタグテーブルの間に何らかの問題があると考えております。

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

ActiveRecord::RecordNotFound in UploadsController#show
Couldn't find Tag with 'id'=3

  def show
    @upload = Upload.find(params[:id])
    @tag = Tag.find(params[:id])⇦この部分
  end

  private

Request
Parameters:

{"_method"=>"get", "authenticity_token"=>"[FILTERED]", "id"=>"3"}

該当のソースコード

app/views/uploads/index.html.erb

      <div id="content-table">
        <% @uploads.each do |upload| %>
          <div class="content-page">
            <%= link_to upload_path(upload.id), class: "upload-link", method: :get do %>
              <div class="upload-contents-wrapper">
                <div class="upload-img-contents">
                  <%= image_tag upload.image, class: "upload-img" if upload.image.attached? %>
                </div>
                <div class="upload-contents-detail">
                  <div class="upload-name"><%= upload.title %></div>
                  <% upload.tags.each do |tag| %>
                    <div class="upload-tags">#<%= tag.name %></div>
                  <% end %>
                </div>
                <div class="upload-contents-explain">
                  <div class="upload-explain"><%= upload.text %></div>
                </div>
                <p class="upload-user">
                  <a class="user-name__image" href="#">
                    <img src="assets/image-4.jpg" width="40" height="40" alt="">
                  </a>
                  <%= link_to upload.user.name, "#", class: "upload-user__name" %>
                </p>
              </div>
            <% end %>
          </div>
        <% end %>
      </div>

app/controllers/upload_controller.rb

class UploadsController < ApplicationController
  def index
    @uploads = Upload.all.order(created_at: :desc)
  end

  def new
    @upload_form = UploadForm.new
  end

  def create
    @upload_form = UploadForm.new(upload_params)
    if @upload_form.valid?
      @upload_form.save
      redirect_to root_path
    else
      render new
    end
  end

  def show
    @upload = Upload.find(params[:id])
    @tag = Tag.find(params[:id])
  end

  private

  def upload_params
    params.require(:upload_form).permit(:title, :text, :url, :working_day, :day_off, :cafe_wifi_id, :cafe_charging_id, :cafe_smoking_id, :image, :name).merge(user_id: current_user.id)
  end
end

app/config/routes

Rails.application.routes.draw do
  devise_for :users
  get 'uploads/index'
  root to: "uploads#index"
  resources :uploads, only: [:index, :new, :create, :show]
end

app/models/upload_form

class UploadForm
  include ActiveModel::Model
  attr_accessor :title, :text, :url, :working_day, :day_off, :cafe_wifi_id, :cafe_charging_id, :cafe_smoking_id,
                :user_id, :image, :name

  with_options presence: true do
    validates :title
    validates :text
    validates :cafe_wifi_id
    validates :cafe_charging_id
    validates :cafe_smoking_id
    validates :user_id
    validates :image
    validates :name
  end

  with_options numericality: { other_than: 0 } do
    validates :cafe_wifi_id
    validates :cafe_charging_id
    validates :cafe_smoking_id
  end

  def save
    upload = Upload.create(title: title, text: text, url: url, working_day: working_day, day_off: day_off, cafe_wifi_id: cafe_wifi_id, cafe_charging_id: cafe_charging_id, cafe_smoking_id: cafe_smoking_id, user_id: user_id, image: image)
    tag = Tag.create(name: name)
    # map = Map.create(address: address, latitude: latitude, longitude: longitude, upload_id: upload_id)

    UploadTagRelation.create(upload_id: upload.id, tag_id: tag.id)
  end
end

試したこと

・今回のエラーの場合、コントローラー内のshowアクションの記述、index.html.erbのパスの指定や記述の内容に問題があると考えました。
・ <% @uploads.tags.each do |tag| %>ような記述にしてみましたが変化はありませんでした。
・他の投稿では詳細ページに遷移することができているため、index.html.erbのeachメソッドの記述方法などに誤りはないかなと思い、コントローラー内のshowアクションの@tag=Tag.find(params[:id])にエラーの原因があると考えましたが、どこを修正すればいいのか調べてもわかりませんでした。

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

開発環境
・rubymine
・ruby(3.0.1)
・Ruby on rails (6.1.3.1)

Answer 1

@tag=Tag.find(params[:id])にエラーの原因がある

そうですね。TagとUploadを同じidで検索するのは明らかに何かおかしいです。

そして、この際見るべきはapp/views/uploads/show.html.erbにて
@tagがどう使われているか？です。

使われていない場合は、@tagは不要ということになります。

もし、form_with(model: @tag)で新規タグを追加するために使われていた場合は

@tag = @upload.tags.build

列挙して表示

<% @tag.each do|tag| %>

だった場合は

@tag = @upload.tags

となります。
さらに言えば、単数形の変数名に複数のタグを期待するのがわかりにくいので
変数名を@tagsに変えたいところです。

Comments

[yuu0000]

ご連絡ありがとうございます！

上記のように記述してみたところidが見つからないというエラーが解消されました！

Answer 2

詳細ページに遷移させるコードを、

@upload = Upload.find(params[:id])
@tag = Tag.find(params[:id])

にしていたため、同じidで取得させようしたことから紐ずくタグのidを取得することができないエラーが起きていました。

正解は、

@upload = Upload.find(params[:id])
@tag = @upload.tags

上記の記述を行い、投稿テーブル（upload）とタグを紐付け、インスタンス変数@tagに代入する必要がありました。