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

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

ただいまの
回答率

89.89%

Couldn't find List with 'id'=1の理由が知りたいです

解決済

回答 1

投稿

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

asatuki

score 2

「Ruby on Rails」に現在取り組んでいるのですが、「destoryアクション」を導入し、削除機能を追加しようと考えました。

また、データ削除後のリダイレクト先は、投稿一覧ページにしようと「indexアクション」にも手を加えました。

しかし、削除機能追加後にサーバーを起動しようとしたら、

ActiveRecord::RecordNotFound in TodolistsController#show
Couldn't find List with 'id'=1

Extracted source (around line #21):
19
20
21
22
23
24


  def show
     @list = List.find(params[:id])
  end

  def edit


という表示がされてしまいました。

色んなサイトを調べ、実行してみましたが結局分からず・・・。
誠に勝手ながら、分かる方がいらっしゃいましたら、助言の方いただきたいです。

**todo lists_controller.rb**

class TodolistsController < ApplicationController

    def new
         @list = List.new
    end


    def create
         list = List.new(list_params)

         list.save

        redirect_to todolist_path(list.id)
    end

  def index
        @lists = List.all(params[:id])
  end

  def show
     @list = List.find(params[:id])
  end

  def edit
      @list = List.find(params[:id])
  end

  def update
      list = List.find(params[:id])
      list.update(list_params)
      redirect_to todolist_path(list.id)
  end

 def destroy
        list = List.find(params[:id])
        list.destroy(list_params)
        redirect_to todolists_path(list.id)
    end

    private

    def list_params
        params.require(:list).permit(:title, :body, :image)
    end
end
**index.html.erb**

<h1>投稿一覧</h1>
   <% @lists.each do |list| %>
      <p>タイトル</p>
      <span>
          <%=link_to list.title, todolist_path(list.id) %>
      </span>
     <p>画像</p>
      <%= attachment_image_tag list, :image, :fill, 100, 100, format: 'jpeg' %>
    <% end %>
**show.html.erb**

<h2>タイトル</h2>
<p><%= @list.title %></p>
<h2>本文</h2>
<p><%= @list.body %></p>
<h2>画像</h2>
<%= attachment_image_tag @list, :image, :fill, 300, 300, format: 'jpeg' %>

<%= link_to "編集", edit_todolist_path(@list.id) %>
<%= link_to "削除", destroy_todolist_path(@list.id), method: :delete, "data-confirm" => "本当に削除しますか?" %>
**routes.rb**

Rails.application.routes.draw do
  get 'todolists/new'
  # For details on the DSL available within this file, see http://guides.rubyonrails.org/routing.html
  get "top" => "homes#top"

  post 'todolists' => 'todolists#create'

  get "todolists" => "todolists#index"

  get 'todolists/:id' => 'todolists#show', as: 'todolist'

  get "todolist/:id/edit" => "todolists#edit", as: "edit_todolist"

  patch "todolists/:id" => "todolists#update", as: "update_todolist"

  delete "todolists/:id" =>"todolists#destroy", as: "destroy_todolist"

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

質問への追記・修正、ベストアンサー選択の依頼

  • asatuki

    2020/01/14 09:19

    はい、それで間違いありません

    キャンセル

  • winterboum

    2020/01/14 09:38

    困った、とするとわからん、
    indexで <%=link_to list.title, todolist_path(list.id) %> が破綻していない
    ということは list はある。
    ということは list.idは実在のid、
    なのに List.find(params[:id]) では無いと言われる。

    キャンセル

  • asatuki

    2020/01/15 19:02

    解決することができました。
    ありがとうございます。

    キャンセル

回答 1

checkベストアンサー

+1

原因

id=1 のデータが存在しないのに、それを表示させようとしたため、エラーが発生しています。

destroy アクションの一番最後のコードが問題です。

  def destroy
    list = List.find(params[:id])
    list.destroy(list_params)
    redirect_to todolists_path(list.id) # ← こいつが犯人
  end

直前のコードでデータを削除しているのに、その削除した id の詳細画面(showアクション)にリダイレクトしちゃってますから、必ずエラーになってしまいます。

対策

削除後は index アクション(todolists_path)に飛ばすと良いかと思います。

  def destroy
    list = List.find(params[:id])
    list.destroy
    redirect_to todolists_path
  end

気になること

ちなみに、質問文に記載されているコードは、実際に動かしたコードをそのまま貼り付けたのではなく、質問する際に改めて書き直したものでしょうか・・・?

list.destroy(list_params) や todolists_path(list.id) といった記述だと、ArgumentError という、もっと別のエラーが発生しそうな気がしますが・・・。

index アクションの

@lists = List.all(params[:id])

も、正しくは

@lists = List.all

と書かないと動かないような。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2020/01/15 19:02

    ご指摘を頂いた部分を訂正したところ、無事起動することができました。
    本当にありがとうございます。

    コードの件ですが、ご指摘通り質問する際に書き直した物になります。
    正確には、自分で色々といじる前のコードになります。
    もしかしたら、自分が余計なところを消しているのでは?と思い、いじる前のコードを貼り付けました。

    キャンセル

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

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

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