ActiveRecord::RecordNotFound in のエラー

Question

テーブル名
・posts
・itemss
1(posts):多(items)の関係です。

itemsには
・item_image
・brand
・size
・price
・products_name
・post_id
・user_id
以上の７つのカラムがあります。

エラー内容

items.controller.erb
class ItemsController < ApplicationController
  def index
    @posts = Post.all
    @user = current_user
    @items = @user.items
    #@userの子であるitemに限定する。
  end

  def new
    @post = Post.find_by(id: params[:id])
    @item = Item.new(item_image: params[:item_image], brand: params[:brand], size: params[:size], price: params[:price], products_name: params[:products_name], post_id: params[:post_id])
  end

  def create
    @item = Item.new(item_image: params[:item_image], brand: params[:brand], size: params[:size], price: params[:price], products_name: params[:products_name], post_id: params[:post_id])
    @item.user_id = current_user.id
    if @item.save
      @item.item_image="#{@item.id}.jpg"
      image=params[:item_image]
      File.binwrite("app/assets/images/items/#{@item.item_image}", image.read)
      redirect_to("/items/#{@item.post_id}/show")
    else
      render("/items/new")
    end
  end

  def show
    @item = Item.find_by(params[:id])
    @post = Post.find_by(id: params[:id])
    @items = @post.items

  end

  def edit
    @item = Item.find_by(id: params[:id])
  end

  def destroy
    @item = Item.find(params[:id])　　　　　　　　　　　　←エラー箇所
    @item.destroy
    redirect_to("/posts/#{@post.id}")
  end
end

vews/items/show.html.erb

<div class="main posts-show">
  <div class="container">
    <div class="posts-show-item">
    <% @items.each do |item| %>
        <tr>
          <% if File.exist?("app/assets/images/items/#{item.id}.jpg") %>
              <td><%= image_tag "items/#{item.id}" %></td>
            <% else %>
              <td><%= image_tag "no_image.png" %></td>
            <% end %>
      <tr>
        <!--アイテム画像-->
        <td></td>
      </tr>
      <tr>
        <!--アイテム名-->
        <td><%= item.products_name %></td>
      </tr>
      <tr>
        <!--ブランド-->
        <td><%= item.brand %></td>
      </tr>
      <tr>
        <!--サイズ-->
        <td><%= item.size %></td>
      </tr>
      <tr>
        <!--価格-->
        <td><%= item.price %></td>
      </tr>
    <% end %>

      <div class="post-time">
        <%= @item.created_at.strftime("%Y-%m-%d") %>
      </div>
      <% if @post.user_id == @current_user.id %>
      <div class="post-menus">
        <%= link_to("アイテム編集","/items/#{@post.id}/edit") %>
        <%= link_to("アイテム削除", "/items/#{@item.post_id}/destroy", {method: "post"}) %>
      </div>
      <% end %>
      <%= link_to("コーディネート画面へ戻る", "/posts/#{@post.id}") %>
    </div>
  </div>
  <%= link_to("トップページへ", "/") %>
</div>

このように記載し、vews/items/show.html.erbの
<%= link_to("アイテム削除", "/items/#{@item.post_id}/destroy", {method: "post"}) %>
このlinkを踏んだ際に出るエラーとなります。

今回はpost_idの159番を削除したいのですがカラムには番号が入っています。
postsのidとitemsのpost_idは紐付けています。

159番のアイテムを消去しようとするとidがないと、正常に削除できません。

何かアドバイスや原因などわかる方いらっしゃいましたらコメントいただけると幸いです、、
よろしくお願い致します。

Comments

[winterboum]

show のcodeを載せてください

[退会済みユーザー]

winterboum様
追記いたしました。よろしくお願い致します。

Answer 1

 def show
    @item = Item.find_by(params[:id])
    @post = Post.find_by(id: params[:id])

@item も @post も同じパラメータの値を使ってます。
なので、@postが得られていません。@postはpostのidを使ってください

Comments

[退会済みユーザー]

ご回答ありがとうございます。
@postの定義を
＠post = Post.find(@item.post.id) としたのですが異なる番号のアイテムが削除されてしまいました、、
（169を削除しようとすると167が消える）

[winterboum]

それは 169だと思っていた @item.post が 167だったのでは？

Answer 2

エラーの原因

view のアイテム削除のリンクに使っている id は Post の id ですが、

<%= link_to("アイテム削除", "/items/#{@item.post_id}/destroy", {method: "post"}) %>

destory アクションではその id を使って Item を検索しています。

  def destroy
    @item = Item.find(params[:id])

--

id=159 の Post は存在しますが、
id=159 の Item は存在しません。
このコードだと id=159 の Item を検索するので、当然エラーになります。

修正方法

もし Item を削除したいなら、

【STEP 1】
winterboum さんの指摘の通り、show アクションで @item も @post も同じ id を使っている問題を修正

app/controllers/item_controller.rb

  def show
-   @item = Item.find_by(params[:id])
-   @post = Post.find_by(id: params[:id])
+   @item = Item.find(params[:id])
+   @post = @item.post
    @items = @post.items

  end

--

【STEP 2】
「アイテム編集」「アイテム削除」のリンクでは Item の id を利用する。

app/views/items/show.html.erb

      <div class="post-menus">
-       <%= link_to("アイテム編集","/items/#{@post.id}/edit") %>
-       <%= link_to("アイテム削除", "/items/#{@item.post_id}/destroy", {method: "post"}) %>
+       <%= link_to("アイテム編集","/items/#{@item.id}/edit") %>
+       <%= link_to("アイテム削除", "/items/#{@item.id}/destroy", {method: "post"}) %>
      </div>

--

【STEP 3】

パラメータの id を使って Item を削除する。（ここは修正なし）

ただし、その直後のリダイレクト用に post_id の取得が必要です。

app/controllers/item_controller.rb

  def destroy
    @item = Item.find(params[:id])

+   # このあと Post の一覧画面にリダイレクトしたいので、削除前に post_id を取得しておく
+   post_id = @item.post_id

    @item.destroy

-   redirect_to("/posts/#{@post.id}")
+   redirect_to("/posts/#{post_id}")
  end