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

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

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

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

Q&A

解決済

2回答

3362閲覧

指定先のコメント削除ができない

tomtom1

総合スコア168

Ruby on Rails 5

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

0グッド

0クリップ

投稿2018/11/14 01:54

前提・実現したいこと

投稿に対し、コメント機能追加しました。
コメントすることは問題ないのですが、コメントを削除する際に、
選んだコメントの削除ボタンを押すと、そのコメントは削除されずに、
一番上のコメント(一番古いコメント)が削除されてしまいます。

おそらく、コメントに対する指定が上手く出来ていないのですが、
自身では未だ解決できずにおり、皆様のお力添えをいただきたく存じます。

該当のソースコード

Routes

1 delete "comments/:post_id/destroy" => "comments#destroy" 2 resources :users, only: [:index, :show] 3 resources :posts, only: [:index, :show, :create] do 4 resources :comments, only: [:create, :destroy] 5 end
Post Controller def show @post = Post.find_by(id: params[:id]) @user = @post.user @comments = @post.comments @comment = Comment.new end
Comment Controller def destroy @comment = Comment.find_by(params[:id]) @comment.destroy redirect_back(fallback_location: 'something') end
show.html.erb <% @comments.each do |c| %> <div> <a href="/users/<%= @post.user.id %>"><%= c.user.name %></a> <%=link_to(c.content, "/comments/#{@post.id}/destroy", {method: "delete"})%> </div> <% end %>

以上です。
お教え頂ければ幸いです。よろしくお願いします。

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

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

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

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

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

guest

回答2

0

<%=link_to(c.content, "/comments/#{@post.id}/destroy", {method: "delete"})%>
ここが、@post.idではなく、c.idではないでしょうか?

追記
resources :comments, only: [:create, :destroy]
があるので、
delete "comments/:post_id/destroy" => "comments#destroy"
は不要かと思いますがいかがですか?

また、link_toは、
<%= link_to '削除', c, method: :delete %>
でどうでしょうか?

また、
resources :posts, only: [:index, :show, :create] do
resources :comments, only: [:create, :destroy]
end
ブロック記法でresources :commentsを囲っていますが、これは必要ないのでは?と思いました

さらに追記
上記の様なrouteがネストした状態だと、パスは
"/comments/#{@post.id}/destroy"
でなく、
"/posts/comments/#{@post.id}/destroy"
になりませんか?

いずれにしても、
delete "comments/:post_id/destroy" => "comments#destroy"
は不要かと思います

投稿2018/11/14 03:01

編集2018/11/14 13:13
space-hippy

総合スコア136

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

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

tomtom1

2018/11/14 08:21

ご返信ありがとうございます! それであればしっかり指定できると思い、先ほど変更し挑戦してみましたが、 結果変わらずに、一番古いコメントから削除されてしまいます。 (コメントが一つの場合はそのコメントが削除されます) また、delete "comments/:post_id/destroy" => "comments#destroy" を "comments/:id/destroy" => "comments#destroy"にも変更して挑戦しましたが、 結果は変わらずでした泣
space-hippy

2018/11/14 13:02

resources :comments, only: [:create, :destroy] があるので、 delete "comments/:post_id/destroy" => "comments#destroy" は不要かと思いますがいかがですか?
tomtom1

2018/11/15 05:32

貴重なお時間ととって、ご指摘頂きありがとうございます! Routes: delete "comments/:post_id/destroy" => "comments#destroy" は不要についてですが、無くすとルートがマッチせずエラーが返ってきてしまいます。 また、おっしゃる通りルートのパスが違いました。 "/comments/#{c.id}/destroy”ではなく、/posts/:post_id/comments/:idでした。 ですが、またそこで/posts/#{post_id}/comments/#{c.id}という風に変更しても、No route matches [DELETE] "/comments/55/destroy" と言うエラーが返ってきてしまいます。 delete "comments/:post_id/destroy”は削除できない現状です。 View: <%= link_to '削除', c, method: :delete %>ですが、 SyntaxError in PostsController#showと言うエラーが返ってきます。 また、気になった点がありまして、 エラーをもらうたびに出てくるRequestParameters:{"id"=>"46”}という記載がした部分にあり、それは削除したいコメントのidではなく、そのコメントしているポストのidです。 お教え頂いたように、リンクに"/comments/#{c.id}/destroy”と記述していることから、コメントのidを送っていると認識していますが、もしかしたらコメントidをしっかり送れていないのではと気になりました。 長々と失礼しました。
space-hippy

2018/11/15 05:53

link_toのsyntax eroorは僕が間違っていたみたいです 失礼しました いろいろいじった後のコードを再掲載できますでしょうか?
tomtom1

2018/11/15 06:31

現状、RoutesとControllerに変更はなく、Viewのhtmlが下記のように変更しています。 <% @comments.each do |c| %> <div> <a href="/users/<%= @post.user.id %>"><%= c.user.name %></a> <%= c.content %> <%=link_to("削除", "/comments/#{c.id}/destroy", {method: "delete"})%> <hr></div> <% end %> 結果、現状変わらず、コメント削除を押すと古いコメントの順で削除されてくる状態です。
space-hippy

2018/11/15 09:11

routesで、resources :comments はネストさせずに普通に記述すると不都合がありますか?
tomtom1

2018/11/15 12:38

度々ありがとうございます。 はい、残念ながら、NoMethodError in Posts#showというエラーが出てしまいます。 下記、Viewのhtml内のコメント記入部分の一行目でエラーです。 <%= form_for [@post, @comment] do |f| %> <%= f.text_field :content %> <%= f.hidden_field :user_id, :value => @current_user.id %> <%= f.hidden_field :post_id, :value => @post.id %> <button type="submit" class="btn btn-primary">コメント</button> <% end %> また、コントローラーでは下記のように定義しています。 def show @post = Post.find_by(id: params[:id]) @user = @post.user @comments = @post.comments @comment = Comment.new end 宜しくお願い致します。
guest

0

自己解決

試行錯誤した結果、Post.idの値をしっかりと渡せずにいましたので、
下記のように修正したところ、指定先のコメントの削除に成功しました。

Routes
Rails.application.routes.draw do
resources :posts do
resources :comments
end

Post Controller
def show
@post = Post.find_by(id: params[:id])
@user = @post.user
@comments = @post.comments ←この文を削除
@comment = Comment.new
end

show.html.erb
<% @comments.each do |c| %> ← <% @post.comments.each do |c| %>

<div> <a href="/users/<%= @post.user.id %>"><%= c.user.name %></a> <%=link_to(c.content, **post_comment_path(@post, c)**, {method: "delete"})%> </div> <% end %>

様々なアドバイスとご協力ありがとうございました!

投稿2018/11/21 08:50

tomtom1

総合スコア168

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

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

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問