showとindexアクションとで同じモデルを参照しているはずがidがずれておりエラーが出る

Question

###現状
現在railsでアプリを作っています。
indexページでモデルを一覧表示し、モデルをクリックしたら対応するモデルの
showページにアクセスできるようにしたいと考えています。

ローカルサーバーでアプリを起動したところindexページは表示されるのですが、
モデルをクリックするとshowページアクセスできずに以下のエラーが起きてしまいます。
ちなみにshowページのurlで直接アクセスすることは出来ました。
###エラーログ

Started GET "/souvenirs/%23%3CSouvenir::ActiveRecord_Relation:0x00005571498e3450%3E" for ::1 at 2019-11-19 18:52:07 +0900
Processing by SouvenirsController#show as HTML
  Parameters: {"id"=>"#<Souvenir::ActiveRecord_Relation:0x00005571498e3450>"}
  Souvenir Load (0.6ms)  SELECT  "souvenirs".* FROM "souvenirs" WHERE "souvenirs"."id" = ? LIMIT ?  [["id", 0], ["LIMIT", 1]]
  ↳ app/controllers/souvenirs_controller.rb:23
Completed 404 Not Found in 28ms (ActiveRecord: 0.6ms)


  
ActiveRecord::RecordNotFound (Couldn't find Souvenir with 'id'=#<Souvenir::ActiveRecord_Relation:0x00005571498e3450>):
  
app/controllers/souvenirs_controller.rb:23:in `show'

indexページのインスタンス変数@souvenisと
showページのインスタンス変数@souvenirをdebuggerで確認したところ
同じモデルを参照しているつもりがidが異なっていました。

(byebug) @souvenir
#<Souvenir id: 980190963, name: "MyString", spot: "MyString", price: 1, good: nil, picture: "20191119191038.jpg", genre: "MyString", comment: "MyString", created_at: "2019-11-19 10:10:38", updated_at: "2019-11-19 10:10:38">

byebug) @souvenirs
#<ActiveRecord::Relation [#<Souvenir id: 298486374, name: "MyString", spot: "MyString", price: 1, good: 1, picture: "MyString", genre: "MyString", comment: "MyText", created_at: "2019-11-19 09:34:17", updated_at: "2019-11-19 09:34:17">, #<Souvenir id: 980190962, name: "MyString", spot: "MyString", price: 1, good: 1, picture: "MyString", genre: "MyString", comment: "MyText", created_at: "2019-11-19 09:34:17", updated_at: "2019-11-19 09:34:17">]>

###原因予想
indexページでgemのwill_paginateを使用しているため、@souvenirsの
idの取得法がshowアクションと異なっているからかと考えています。
しかしながら正しいidの渡し方が分からず、質問させていただきました。

class SouvenirsController < ApplicationController

  def new
    @souvenir = Souvenir.new
    flash[:info] = "項目を記入してください"
  end

  def index
    @souvenirs = Souvenir.paginate(page: params[:page], per_page: 5)
  end

  def create
    @souvenir = Souvenir.create(souvenir_params)
      if @souvenir.save
        flash[:success] = "投稿に成功しました"
        redirect_to action: 'index'
      else
        render 'new'
      end
  end

  def show
    @souvenir = Souvenir.find(params[:id])
  end

  def edit
    @souvenir = Souvenir.find(params[:id])
  end

  def update
  end
  
  def destroy
  end

  private

  def souvenir_params
    params.require(:souvenir).permit(:name, :spot, :price, :picture, :genre, :comment)
  end
end

viewページを追記しておきます。

私の前回の質問と少しかぶりますが、
showとindexはほぼ中身が同じなのでパーシャルを使いまわしたいです。
しかしshowページのrenderメソッドで、collectionに渡す@souvenirが認識されず、パーシャルを使えていません。

<div class = "row souvenir_box">
    <div class = "souvenir_name col-sm-12"><%= @souvenir.name%></div>
        <div class = "col-sm-8">
            <div class="row">
                <div class = "souvenir_spot col-sm-12 souvenir_text-items "><%= @souvenir.spot%></div>
                <div class = "souvenir_price col-sm-12 souvenir_text-items"><%= @souvenir.price%></div>
                <div class = "souvenir_genre col-sm-12 souvenir_text-items"><%= @souvenir.genre%></div>
            </div>
        </div>
    <div class = "col-sm-4">  
      <% if @souvenir.picture? %>
    　　<%= image_tag @souvenir.picture.thumb.url %>
    　<% else %>
        <p>noimages</p>
        <% end %>
    </div>
    <div class = "col-sm-12"><%= @souvenir.comment%></div>
</div>

<%= will_paginate  @souvenirs %>
<%= render partial:"souvenir", collection: @souvenirs %> 
<%= will_paginate  @souvenirs %>

<%= link_to souvenir_path(@souvenirs) do  %>
<div class = "row souvenir_box">
    <div class = "souvenir_name col-sm-12"><%= souvenir.name%></div>
        <div class = "col-sm-8">
            <div class="row">
                <div class = "souvenir_spot col-sm-12 souvenir_text-items "><%= souvenir.spot%></div>
                <div class = "souvenir_price col-sm-12 souvenir_text-items"><%= souvenir.price%></div>
                <div class = "souvenir_genre col-sm-12 souvenir_text-items"><%= souvenir.genre%></div>
            </div>
        </div>
    <div class = "col-sm-4">  
      <% if souvenir.picture? %>
    　　<%= image_tag souvenir.picture.thumb.url %>
    　<% else %>
        <p>noimages</p>
        <% end %>
    </div>
</div>
<% end %>

Comments

[winterboum]

index の　viewも追加シて下さい

[ketyan]

index,show両方のviewを追加させていただきました。

Answer 1

index の @souvenirs は複数の Souveir で show の@souvenir　は一つのSouvenirですから、idが異なるのが当然です。

partialは一つのSouvenirを表示させるつもりですか？複数ですか？
一つだとすると
<%= link_to souvenir_path(@souvenirs) do %>と複数を当てているのがまずいです。
また do の中で souvenir.name　などsouvenir を参照していますが、これ未定義のエラーになりませんか？
showとpartialを共有するなら
<%= render partial:"souvenir", collection: @souvenirs %>
を

<% @souvenirs.each do | souvenir | %>
  <%= <%= render partial:"souvenir", locals: {souvenir: souvenir} %> 

とし、
partial側の<%= link_to souvenir_path(@souvenirs) do %>を<%= link_to souvenir_path(souvenir) do %>とするのが良いでしょう。

Comments

[ketyan]

ご回答ありがとうございます。
localsオプションを使えば簡単な話だったんですね。おかげさまで無事viewファイルを修正することが出来ました。
ありがとうございました。