rails コメント機能のバリデーション時のActionController::UrlGenerationError

Question

コメント機能をつけて、そのコメント機能にバリデーション機能を付けようとしています。

空白と２００文字オーバーのコメント投稿に対してバリデーションをしたいです。
自分の知識とサイトの記事を参考に記述したのですがうまく機能しません。

バリデーションにかかるコメント投稿をすると

ActionController::UrlGenerationError in PostComments#create
Showing /home/vagrant/work/port1/app/views/posts/show.erb where line #66 raised:

No route matches {:action=>"destroy", :controller=>"post_comments", :post_id=>27}, missing required keys: [:id]

このようなエラーになってしまいます。
コメント削除のボタンがエラーの該当箇所になっています。

すでに投稿されているコメントがうまく表示ができなくて、エラーになっているのだと思われます。
すでにコメント投稿されていなければうまく実装できます。また、コメント削除ボタンを消してもうまく実装できます。

ちなみに該当箇所はhtmlの下から１６行目あたりです。

class PostsController < ApplicationController
	before_action :authenticate_user
	before_action :ensure_correct_user,{only:[:edit, :update]}
	def new
		@user = current_user
		@post = Post.new
	end
	def create
		@user = current_user
		@post = Post.new(post_params)
		@post.user_id = current_user.id
		if @post.save
		   redirect_to post_path(@post.id),notice: "Successful posting!!"
			else
				render :new
		end
	end
	def index
		@user = current_user
		@posts = Post.all.order(id: "DESC")
	end
	def show
		@post = Post.find(params[:id])
		@user = @post.user
		@comments = @post.post_comments
		@post_comment_new = PostComment.new
	end
	def edit
		@user = current_user
		@post = Post.find(params[:id])
	end
	def update
		@user = current_user
		@post = Post.find(params[:id])
		@post.user = current_user
		if @post.update(post_params)
		   redirect_to post_path(@post.id),notice: "Successful editing!!"
		else
			render :edit
		end
	end
	def destroy
		@post = Post.find(params[:id])
		@post.destroy
		redirect_to posts_path,notice: "Successful deleting!!"
	end
	def ensure_correct_user
         @post = Post.find(params[:id])
      if @post.user_id != current_user.id
         redirect_to post_path(@post.id)
      end
    end
	private
	def post_params
		params.require(:post).permit(:title, :body, :image, :user_id)
	end
end

class PostCommentsController < ApplicationController
	def create
		@post = Post.find(params[:post_id])
		@post_comment_new = PostComment.new(post_comment_params)
		@user = @post.user
		@comments = @post.post_comments
		@post_comment_new.user_id = current_user.id
		@post_comment_new.post_id = @post.id
		if @post_comment_new.save
			redirect_to post_path(@post.id),notice: "Successful comment posting!!"
		else
			flash[:alert] = "Posting error!!"
			render template: 'posts/show'
		end
	end
	def destroy
		@comment = PostComment.find(params[:post_id])
		@comment.destroy
		redirect_to request.referer,notice: "Successful deleting!!"
	end
	private
	def post_comment_params
		params.require(:post_comment).permit(:comment)
	end
end

<% if flash[:notice] %>
	<div class="success_message">
		<p><%=flash[:notice] %></p>
	</div>
	<% elsif flash[:alert]%>
	<div class="error_message">
		<p><%=flash[:arlrt] %></p>
	</div>
<% end %>
<%= render "layouts/prof", user:@user %>
<div class="post_show">
	<!-- このuserの詳細画面に推移 -->
	<div class="post_show_image_space">
		<%= attachment_image_tag @post, :image, class: "post_show_image" %>
	</div>
	<div class="post_show_text_area">
		<h3>Title</h3>
		<p><%= @post.title %></p>
		<h3>Body</h3>
		<p><%= @post.body %></p>
	</div>
    <div class="post_show_text_area">
	    <h3>Favorites</h3>
		<!-- favorites -->
		<p>
		<% if @post.favorited_by?(current_user) %>
    		<%= link_to  post_favorites_path(@post), method: :delete do%>
    		  <i class="fas fa-heart" style="color:red;"></i>
    		<% end %>
    	<% else %>
    		<%= link_to  post_favorites_path(@post),method: :post do%>
    		  <i class="far fa-heart"></i>
    		<% end %>
    	<% end %>　<%=@post.favorites.count%>　favorites</p>
		<h3>Comments</h3>
		<%= render "layouts/flash", object:@post_comment_new%>
		<h4>Post a comment</h4>
		<%= form_for [@post,@post_comment_new] do |f|%>
			<%=f.text_area :comment , placeholder: "comment"%>
			<%=f.submit "submit"%>
		<% end %>
		<!-- クリックされたらこの表示が消えてコメントが全部表示される -->
		<a class="posted_comments comment_close open">See all <%=@post.post_comments.count%> comments</a>
		<div class="posted_comments comment_open">
			<a>Fade out comments</a>
			<table class="table">
				<thead>
					<tr>
						<th>Commented user</th>
						<th>Comment</th>
						<th colspan="1"></th>
					</tr>
				</thead>
			　　<[e-mail]. each do |comment|%>
				<tbody>
					<tr>
						<td>
							<%= link_to user_path(comment.user_id) do %>
						      <%= attachment_image_tag comment.user, :profile_image, size: "60x60", fallback: "no_image.jpg" , class: "img-circle" %>
						      <%=comment.user.name %>
					        <% end %>
					    </td>
						<td><%=comment.comment%></td>
						<td>
							<% if comment.user_id == current_user.id %>
								<%= link_to "comment detlate", post_post_comment_path(comment.id), method: :delete, data: {confirm: "削除しますか？"},class: "btn btn-danger"  %>
							<% end %>
						</td>
					</tr>
				</tbody>
			　　<% end %>
			</table>
		</div>
	</div>
	<div class="post_show_path">
		<% if @user == current_user %>
			<%= link_to "post edit",edit_post_path(@post.id),class: "btn btn-success " %>
		    <%= link_to "post detlate",post_path(@post.id), method: :delete, data: {confirm: "削除しますか？"},class: "btn btn-danger"  %>
		<% end %>
	</div>
</div>

class PostComment < ApplicationRecord
	belongs_to :user
	belongs_to :post
	validates :comment, presence: true, length: { maximum: 50}
end

Rails.application.routes.draw do
  devise_for :users
  # For details on the DSL available within this file, see http://guides.rubyonrails.org/routing.html
  root 'homes#top'
  resources :users
  resources :posts do
  	resource :favorites, only: [:create, :destroy]
  	resources :post_comments, only: [:create, :destroy]
  end
end

不要な点がありましたらお申し付けください。
分かる方がいらっしゃいましたらぜひよろしくお願いします。

Comments

[winterboum]

config/routes.rbを載せてください

[aku424tt]

ご指摘ありがとうございます。
追加させていただきました

Answer 1

Post と Comment を指定したパスが必要だと思いました。

# <%= link_to "comment detlate", post_post_comment_path(comment.id), method: :delete, data: {confirm: "削除しますか？"},class: "btn btn-danger"  %>
<%= link_to "comment detlate", delete_post_comment_path(@post, comment), method: :delete, data: {confirm: "削除しますか？"},class: "btn btn-danger"  %>

Comments

[aku424tt]

ありがとうございました。解決できました。
もしお時間がありましたら、どうしてこのような記述になるのか教えていただけますでしょうか？

[unhappychoice]

`bundle exec rails routes` を実行してルートを表示すると、

`DELETE /posts/:post_id/post_comments/:id` のような表示があるはずで、`:post_id` と `:comment_id` を指定する必要があるため、で伝わるでしょうかmm

[aku424tt]

なんとなくは理解できました。コメントだけではなく、そのコメントがどこの投稿にされたコメントなのかを示す必要があるということですか？

[unhappychoice]

その理解で良さそうです。

が、やりたいこと次第というか、本質的には post_comment は id だけで特定できるはずなので

- resources :post_comments, only: [:create, :destroy] を resources :posts の外に出す
- DELETE /post_comments/:id がルートとして生成される
- delete_post_comment_path(comment) と記述する

でも同じことが出来ますmm

[aku424tt]

ありがとうございます。
今回に関してはルーティングをネストしているため、その2つを書く必要があるのであって、ネストしていなければ、別の記述ができるということで間違い無いでしょうか

[unhappychoice]

間違いないですmm

[aku424tt]

本当に詳しく教えていただきありがとうございます。
とても勉強になりました。