【Ruby on Rails】コメント機能実装

Question

前提・実現したいこと

Ruby on Railsのコメント機能実装で躓いています。

目標は、Yahoo!ニュースやnewspicksのような
ニュース投稿機能(postモデル)に紐づくコメント(commentモデル)投稿機能の実装です。

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

以下の画像のようにNilClassのNoMethodErrorが発生しています。

postsコントローラのshowページにコメント用のform_withを設置しています。
そこからcommentsコントローラのcreateアクションにデータを渡し
テーブルに保存する工程でpostのidが上手く処理されていないようです。

該当のソースコード

※schemaやコントローラは長いので、関係ありそうなところを抜粋しています。

■ルーティング

  resources :posts do
    resources :comments, :only => [:create, :destroy, :edit, :update]
  end

■テーブル、モデル

  create_table "comments", force: :cascade do |t|
    t.integer "user_id", null: false
    t.integer "post_id", null: false
    t.text "content", null: false
    t.index ["post_id"], name: "index_comments_on_post_id"
    t.index ["user_id"], name: "index_comments_on_user_id"
  end

  create_table "posts", force: :cascade do |t|
    t.text "content", null: false
    t.integer "user_id"
    t.index ["title", "content"], name: "index_posts_on_title_and_content"
    t.index ["user_id"], name: "index_posts_on_user_id"
  end

  create_table "users", force: :cascade do |t|
    t.string "email", null: false
    t.string "password_digest", null: false
    t.index ["first_name", "last_name"], name: "index_users_on_first_name_and_last_name"
  end

  has_many :comments
  has_many :posts

  belongs_to :user
  has_many :comments

  belongs_to :user
  belongs_to :post

■view

<%= form_with model: [@post, @comment] do |f| %>
  <%= hidden_field_tag :user_id, @current_user.id %>
  <%= hidden_field_tag :post_id, @post.id %>
  <%= f.text_field :content %>
  <%= f.submit "保存", class: "btn" %>
<% end %>

■コントローラ

  before_action :set_current_user
  
  def set_current_user
    @current_user = User.find_by(id: session[:user_id])
  end

  def show
    @post = Post.find_by(id: params[:id])
    @comment = Comment.new
    @user = User.find_by(id: @post.user_id)
  end

  def create
    @comment = Comment.create(comment_params)
    @post = Post.find_by(id: params[:id])

    if @comment.save
      flash.notice = "コメントを作成しました"
      redirect_to post_path(@post)
    end
  end

  def comment_params
    params.require(:comment).permit(:content).merge(user_id: @current_user.id, 
    post_id: @post.id)
  end

回答いただきたいこと

①エラー原因と対処について
なぜエラーが起きているのか（なぜpost_idが受け取れないのか）の原因と対処方法を回答いただだきたいです。

※かなり初歩的な認識違いかもしれませんが、以下のような認識でおり、エラーの原因について見当がつきませんでした。

---

controller/posts_controller.rbとview/posts/show.html.erbの中でそれぞれ以下のように書かれている。

def show
@post = Post.find_by(id: params[:id])
end

<%= form_with model: [@post, @comment] do |f| %>
<%= hidden_field_tag :post_id, @post.id %>
<%= f.submit "保存", class: "btn" %>
<% end %>

ルーティングはresources :postsで、URLはhttps://hoge/posts/:idのようになり、
show.html.erbではPost.find_by(id: params[:id])でURLの:idの値で、@postに値が入るはずである。

---

②エラー画面の情報について
エラー画面のRequest Parameters:のところを見るとpost_idの値が入っているように見えますが、
これはどのように解釈すればよろしいでしょうか？

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

■環境
・cloud9(Amazon Linux2)、※ローカルPCはWin10
・Rails 5.2.6
・ruby 2.6.3p62 (2019-04-16 revision 67580) [x86_64-linux]

■その他
・userはdeviseは使用せず、一から作成しています。
・プログラミング学習を始めてから4か月程の初学者です。
・不足している情報があればご指摘いただけますと幸いです。

Answer 1

viewには、<%= hidden_field_tag :post_id, @post.id %>と書いてありますね。

ということは、paramsの中には、post_idというキー名でデータが入っているはずです。

それを考えると、
Post.find_by(id: params[:id])
ではなく、
Post.find_by(id: params[:post_id])なら取得できるかもしれないと思いました。

---

補足

もう一点。

comment_paramsが、@postの宣言より前に呼び出されているので、@postがnilになっている可能性もありそうです。
comment_params内で@postが必要なのであれば、@postの宣言は、comment_paramsを使用する前に行うか、又は、comment_paramsメソッド内の初めに@postを宣言してしまってもよいかもしれませんね。

Comments

[keigo555]

ご回答ありがとうございます。お蔭様で解決しました。大変助かりました。補足で記載いただいたように、comment_paramsの冒頭に@post = Post.find_by(id: params[:post_id])を書くことで動きました。