Rails 投稿に紐づくコメントが削除できない

Question

実現したいこと

投稿に紐づくコメントを削除したいです。

前提

railsでコメント機能を作成しています。

発生している問題・エラーメッセージ

定番のメッセージなのですが

undefined method `destroy!' for nil:NilClass

該当のソースコード

  scope module: 'users' do
    resources :posts, except: %i[index new] do
      resources :comments, module: :posts, only: %i[create destroy]
    end
  end

class Users::Posts::ApplicationController < Users::ApplicationController
  before_action :set_post

  private

  def set_post
    @post = Post.find(params[:post_id])
  end
end

class Users::Posts::CommentsController < Users::Posts::ApplicationController

  def destroy
    comment = current_user.comments.find_by(id: params[:id])
    comment.destroy!
    redirect_to post_path(@post), alert: 'コメントを削除しました'
  end

  private

  def comment_params
    params.require(:comment).permit(:content)
  end
end

- comments.each do |comment|
  .card.w-50.mx-auto.my-3
    .card-header
      = "#{comment.user.name}さんのコメント"
    .card-body
      = comment.content
      - if comment.user == current_user
        = link_to post_comment_path(comment), data: { turbo_method: :delete, turbo_confirm: '本当に削除しますか？' }, class: 'text-reset ms-auto' do
          %i.bi.bi-trash

viewに関してはパーシャルにしているので@commentsではなくcommentsになっています

試したこと

最初はcomment = current_user.comments.find(params[:id])という記述で書いていたのですがこれだとレコードが見つからないというエラーになったので、find_byで試したところcommentがnilになっていて、結局どうすればいいのかわからなくなってしまいました。

補足情報（FW/ツールのバージョンなど）

ruby 3.1.3
Rails 7.0.4

Comments

[maisumakun]

commentはuserとpostのどちらに紐づいているのでしょうか？

[ryusei1212]

記載が漏れていましたが、どちらにも紐づいています。

class Comment < ApplicationRecord
belongs_to :user
belongs_to :post
end

Answer 1

ルートをネストしているため、post と comment は post_comment_path basketbros で指定する必要がある 2 つのパラメーターです。 1 つだけが提供されているため、適切な値は配信されなくなります。 ただし、コメントの内容がすでにわかっている場合は、コメントを削除して記事を不要にすることができます。

Answer 2

最終的に以下のように実装しました。

 resources :posts, except: %i[index new] do
   resources :comments, module: :posts, only: %i[create update destroy], shallow: true
 end

class Users::Posts::CommentsController < Users::Posts::ApplicationController
  skip_before_action :set_post, only: :destroy

  def destroy
    comment = Comment.find(params[:id])
    post = comment.post
    comment.destroy!
    redirect_to post_path(post), alert: 'コメントを削除しました'
  end
end

他にいい方法がありましたらコメントお願いします。

Answer 3

ルーティングをネストしているので、post_comment_pathには、postとcommentの2つを指定する必要があります。1つしか指定していない結果、正しい値が送られなくなっています。

とはいえ、commentがどれかわかれば削除はできますので、postは冗長です。このような場合、ルーティングでshallow: trueと指定すると、commentがすでに存在するshowやedit、destroyなどでcomment側のidだけを含んでルーティングを引くようになります。

Comments

[maisumakun]

https://railsguides.jp/routing.html#浅いネスト

[ryusei1212]

コメントありがとうございます。
> postとcommentの2つを指定する必要があります。1つしか指定していない結果、正しい値が送られなくなっています。

comment = @post.comments.find_by(id: params[:id])
上記記述でも同じ様にエラーが起こるのですが、投稿に紐づくコメントの中から、pramasで受け取ったidをもとにコメントを特定して削除するというのなぜできないのかが、腑に落ちていません。

[ryusei1212]

すいません、わすれてください。解決できそうです。

[ryusei1212]

maisumakunさんありがとうございます。