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

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

新規登録して質問してみよう
ただいま回答率
85.46%
Ruby

Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

Q&A

解決済

2回答

1287閲覧

railsにおける投稿の削除機能でエラーが起きてしまう

sawasawanosawa

総合スコア4

Ruby

Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

0グッド

0クリップ

投稿2021/06/02 19:18

###解決したいこと
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行目です。)

どうかご教授いただけないでしょうか。

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答2

0

<% @posts.each do |post| %>
<%= link_to("削除", "/posts/#{@post.id}/destroy", {method: "post"}) %>

post@postは別ですので、この場合は

erb

1 <%= link_to("削除", "/posts/#{post.id}/destroy", {method: "post"}) %>

です。

投稿2021/06/02 23:43

asm

総合スコア15147

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

sawasawanosawa

2021/06/03 06:37

ありがとうございます。 解決いたしました。
guest

0

ベストアンサー

@post.id を post.id に

投稿2021/06/02 23:29

winterboum

総合スコア23416

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

sawasawanosawa

2021/06/03 06:36

ありがとうございます。 解決いたしました。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.46%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問