resourcesのshowページが全部同じになってしまいます。

Question

ページ遷移がおかしいです。
ルーティングはresourcesを使っています。

下記コードのリンク部分から商品詳細に飛ぶと、
最初に登録された商品の詳細ページにしかいきません。
/products/10 や　/products/13　とurlを変えても
表示されている最初の商品の詳細(/products/9の商品)がでてしまいます。

おかしな点はありますでしょうか？

index.html.erb↓

<!-- プロダクト一覧 -->
<% if @products %>
  <% @products.each do |product| %>
    <p><%= product.title %></p>
    <p><%= product.description %></p>
    <%= image_tag product.image_name.thumb.url %>
    <p>価格</p>
    <p><%= product.price %>円</p>
    <%= link_to("詳細", "/products/#{product.id}") %>
  <% end %>
<% end %>

show.html.erb↓

<h1>商品詳細</h1>
<p><%= @product.title %></p>
<p><%= @product.description %></p>
<%= image_tag @product.image_name.thumb50.url %>
<p>価格</p>
<p><%= @product.price %>円</p>
<p>在庫</p>
<p><%= @product.stock_quantity %>個</p>
<%= link_to("編集", edit_product_path) %>
<%= link_to("削除", "/products/#{@product.id}", method: :delete, data: {confirm: '本当に削除しますか?'}) %>
<%= link_to("戻る", root_path) %>

コントローラ↓

  def index
    @products = Product.all
    @user = current_user
  end

  def show
    @product = Product.find_by(params[:id])
  end

Comments

[maisumakun]

showのコントローラー側はどのように書いていますか？

（一覧を生成するビューの側は、正しいURLが生成できているなら仕事はそこまでなので、おそらく関係しません）

[popomarudasi]

コントローラでは以下の記述をしています。

def index
@products = Product.all
end

def show
@product = Product.find_by(params[:id])
end

Answer 1

Product.find_by(params[:id])ではなく、Product.find(params[:id])としてください。

Comments

[popomarudasi]

こちらで解決しました。
ありがとうございました。

find使ったことがないのですが、
自分でも調べてみますが、
find　と　find_byの使い分け簡単でいいので教えていただけると幸いです。

[popomarudasi]

あ、すみません。
なんとなくわかりました。
find_byの場合、最初のレコードだけを返すということだったんですね！

ありがとうございました。

[maisumakun]

find_byはwhereと同様な条件を指定するものですので、idの数値だけ渡しても意図したように動きません。

findはid専用です。

[maisumakun]

いちおう、find_by(id: params[:id])と書けばfind_byでも正しく動作します。

[popomarudasi]

とてもわかりやすくイメージがつきました。
ご丁寧にありがとうございました。