###解決したいこと
railsで投稿を削除する機能を追加しようとしたところ、以下のエラーが出てしまいました。
NoMethodError in Posts#index Showing /home/ec2-user/environment/gamesnap/app/views/posts/index.html.erb where line #21 raised: undefined method `id' for nil:NilClass Extracted source (around line #21): </div> <div class = "destroy"> <%= link_to("削除", "/posts/#{@post.id}/destroy", {method: "post"}) %> </div> </div> </div>
(link_toのところが21行目です。)
postsというデータベースから削除したい投稿のid(投稿したときに自動的に追加される)を見つけて関連データを削除する機能を追加したい。
index.html.erbにてidがないということはどういうことなのでしょうか。
###環境
ruby :2.6.3
ruby on rails :5.0.7.2
###コード
routes.rb
Rails.application.routes.draw do get 'posts/index' => "posts#index" get 'posts/new' => "posts#new" post "posts/create" => "posts#create" post "posts/:id/destroy" => "posts#destroy" get "posts/like" => "posts#like" get '/' => "posts#index" #get "posts/:id" => "posts#index" # For details on the DSL available within this file, see http://guides.rubyonrails.org/routing.html end
posts_controller.rb
class PostsController < ApplicationController def index @posts = Post.all.order(created_at: :desc) end def new @post = Post.new end def create @post = Post.new(post_params) if @post.save flash[:notice] = "投稿しました" redirect_to("/posts/index") else flash[:notice] = "文字数を140字以内にしてください" render("posts/new") end end def like end def post_params params.permit(:id, :content, :img, :title) end def destroy @post = Post.find_by(id: params[:id]) if @post.id(id: params[:id]) @post.destroy redirect_to("/posts") end end end
index.html.erb
<div class="main posts-index"> <div class = "a"> 投稿一覧画面 </div> <% @posts.each do |post| %> <div class = "post-all"> <div class = "posts-index"> <div class = "post-title"> <%= post.title %><br> </div> <div class = "post-content-box"> <div class = "post-content"> <%= post.content %><br> </div> </div> <div class = "post-img"> <%= image_tag post.img.url if post.img? %><br> </div> <div class = "destroy"> <%= link_to("削除", "/posts/#{@post.id}/destroy", {method: "post"}) %> </div> </div> </div> </div> <% end %>
###行ったこと
######1、そもそもデータベースにidというカラムが存在していない
↓消したい内容のデータベース
CREATE TABLE "posts" ("id" INTEGER PRIMARY KEY AUTOINCREMENT NOT NULL, "content" text, "created_at" datetime NOT NULL, "updated_at" datetime NOT NULL, "title" varchar, "img" varchar);
idは投稿すると自動的に入るし、NOT NULLなので絶対に値が入るため、ここが問題ではないでしょう。
(実際、値は入っていました。)
1|あfds|2021-05-30 11:25:30.082329|2021-05-30 11:25:30.082329|| 2|あ|2021-05-30 11:40:56.891927|2021-05-30 11:40:56.891927|| 3||2021-05-30 15:07:46.033794|2021-05-30 15:07:46.033794|| 4||2021-05-30 15:08:03.649756|2021-05-30 15:08:03.649756|| 5|sssss|2021-05-30 18:15:34.161435|2021-05-30 18:15:34.161435||something.jpg 6|aaaaaa|2021-05-31 01:55:29.505669|2021-05-31 01:55:29.505669||something.jpg 7|aassadhf|2021-05-31 02:01:22.665429|2021-05-31 02:01:22.665429||something.jpg
一番左の数字がidのはず。
#####2、コントローラ内にidを探すように書いていない
@post = Post.find_by(id: params[:id])
という風に書いているのでここも違うはず。
#####3、ifを使ってidがないときも動くようにしていない
if @post.id(id: params[:id])
という風に書いているからここも違うはず。(そもそもNOT NULLなのでidがないということがおかしいと思います。)
#####4-1、いっそのこと「#{@post.id}」を消してみる
トップページにアクセスすることができるようになったが、削除を押すとルートのエラーが出る。
routes.rbにて「post "posts/:id/destroy" => "posts#destroy"」と書いているのでこのエラーが出るのは当然。
#####4-2、routes.rbを修正する
「post "posts/destroy" => "posts#destroy"」という風に変更。
しかし、今度はposts_controller.rbにてエラーが発生。
NoMethodError in PostsController#destroy undefined method `id' for nil:NilClass Extracted source (around line #32): def destroy @post = Post.find_by(id: params[:id]) if @post.id(id: params[:id]) @post.destroy redirect_to("/posts") end
(if @post.id(id: params[:id])が32行目です。)
どうかご教授いただけないでしょうか。
回答2件
あなたの回答
tips
プレビュー
バッドをするには、ログインかつ
こちらの条件を満たす必要があります。
2021/06/03 06:37