作品にレビューを紐付けして表示させたい

Question

前提・実現したいこと

railsでレビューサイトの個人アプリを作っています。
レビューに作品ごとのidを入れたいです。

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

レビューに作品ごとのidを入れたい
作品をpostsテーブルという名前で作成。レビューをreviewsテーブルという名前で作成。
reviewsテーブルにpost_idというカラムを作りました。ここに作品ごとの(post.id)idを入れたいのですが作品ごとのidが入らずカラムに全部０が入ります。

エラーメッセージ

該当のソースコード

ルーティング

Rails.application.routes.draw do
  devise_for :users
  get '/' => 'posts#index'
  get 'posts/new'  => 'posts#new'
  post 'posts/create' => 'posts#create'
  get 'posts/:id' => 'posts#show'
  post 'posts/:id' => 'posts#show'
  post 'reviews/create' => 'reviews#create'
  get 'reviews/new' => 'reviews#new'
  resources :posts, only: :show do
    resources :reviews, only: [:new, :create]
    collection do
      get 'search'
    end
  end
end

reviewsコントローラー

  def new
    @post = Post.find(params[:post_id])
  end
  def create
    @reviews = Review.new(nickname: params[:nickname],rate: params[:rate],review: params[:review],post_id: params[:post_id])
   if @reviews.save!
    redirect_to('/')
   end
  end

postのshow.html.erb(作品を表示しているところ。ここからreviewのnew.html.erbに飛ぶ）

<div class="show_main">
  <h1 class="show_title"><%= @post.name %></h1>
  <div class="content_post" style="background-image: url(<%= @post.image %>);">
  </div>
  <a href="/reviews/new?post_id=<%= @post.id %> "class="review_link">この作品をレビューする</a>
  <% @reviews.each do |review| %>
  <div class="review"><%= review.nickname %></div>
  <div class="review"><%= review.review %></div>
  <%end%>

reviewのnew.html.erb（レビューの投稿画面）

 <%= form_tag('/reviews/create',{multipart: true}) do %>
<input placeholder="ニックネーム" type="string" name="nickname">
<input  type="integer" name="rate">
<textarea name="review" rows="4" cols="40" type="text">ここに感想を記入してください。</textarea>
<%= hidden_field_tag :post_id,value: @post.id %>
<input type="submit" value="SENT">
<%end%>

試したこと

ここに問題に対して試したことを記載してください。

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

modelには特に触れていません。
post.rbにhas_many :reviews
review.rbにbelongs_to :post
をかくと送信ができずMySQLにデータが入らないので
ここにより詳細な情報を記載してください。

Comments

[Orlofsky]

>レビュー投稿機能を実装中に以下のエラーメッセージが発生しました。
エラーメッセージがどこにもないです。
エラーメッセージは表示された内容をそのまま質問にコピペしてください。https://teratail.com/help/question-tips#questionTips3-4-2

[Orlofsky]

テーブルの定義情報をCREATE TABLEで、各テーブルのデータもINSERTで何件かあったほうが適切なコメントが付き易いです。

[Orlofsky]

コードは https://teratail.com/help/question-tips#questionTips3-7 の [コード] に修正してください。

[winterboum]

「コードは https://teratail.com/help/question-tips#questionTips3-7 の [コード] に修正してください。」
これかなり重要です。読みにくいcodeだと読んでくれない、回答がつかない。

Answer 1

おかしな点がいくつかありますが、肝心のエラー内容が質問内に書かれていないため、
今どこでつまづいているのかこちらでは把握できません。
記事内に現在起きているエラーを追記し、一つずつ潰していくのがよろしいかと思います。

とりあえず、質問の最初に書かれている

reviewsテーブルにpost_idというカラムを作りました。ここに作品ごとの(post.id)idを入れたいのですができないです。

に関わる部分の回答をしておきます。
レビュー作成の部分、紐付けをモデルで行なっている前提ではありますが...

# posts_controller.rb
def create
  @post = Post.find_by(id: params[:post_id]) # params[:id]ではないはず、未検証。
  # @post.reviews.build とすることで @post に紐づく reviews を作成できます。
  # 後から save するのであれば create ではなく build (newでも可)
  # 変数名がおかしい、中身が単数の場合は単数形の名前を使用すること。
  @review = @post.reviews.build(nickname: params[:nickname],
                                rate: params[:rate],
                                review: params[:review])
  # save ではなく save! とすることで保存に失敗した際にバリデーション結果が出力されるので、原因に気付きやすい
  if @review.save!
    redirect_to('/')
  end
  # 保存に失敗した時の処理がないので追加しておくこと。
end

<%# reviews/new.html.erb %>
<%# おそらくフォームの送信先がおかしいのできちんと待ち受けているところに向けて送信する %>
<%= form_tag('posts/#{@post.id}/reviews',{multipart: true}) do %>

こんな感じでどうでしょうか。
どのURLでどんなリクエストを受け付けているのかは、ターミナル上で

$ rails routes

を実行することで確認できます。
フォームやリンクはそれをめがけて正しいHTTPリクエストを発行するようにしましょう。

viewの箇所は、本当はの以下の方が好ましいですが、よくわからなければ今後の課題としてください。

<%= form_with(model: [@post, Review.new], local: true, multipart: true) do %>

Answer 2

Review の create の時に　post_id を設定する必要がありますが、渡って居ません。Reviewのnewにpost_idが無いので。

下流から行きます

1. reviewのnew.htmlの input に並べて <%= hidden_field_tag :post_id,value: @post.id %> を入れてください。

2)def new に @post = Post.find(params[:post_id]) と定義。
3)Postのshowの<a href="/posts/<%= @post.id %>/reviews/new"class="review_link">　からReviewsControllerのnewに行くという想定だと思います。
ここを<a href="/reviews/new?post_id=<%= @post.id %> "class="review_link"> とか <%= link_to "この作品をレビューする",review_path(post_id: @post.id)%> の様に。
Reviewを直接呼ぶ方が混乱しにくいです。

Comments

[sa.k]

回答ありがとうございます!
1と２をして３を<a href="/reviews/new?post_id=<%= @post.id %> "class="review_link">に変更しました！
そしたらnullではなく０という数字が入りました！createをbinding.pryしてparamsで見ると入っているきがしますがなぜ０何でしょうか？
[1] pry(#<ReviewsController>)> params
=> <ActionController::Parameters {"utf8"=>"✓", "authenticity_token"=>"dzSCqJJMOFvmCAe/V9a8wVbSdv7T+M7VwKbcSMnpxrG9A4AceoTaoHEdX1pfaHn+hrSpZp/mrwsUbk40v4Ev2Q==", "nickname"=>"あんぱん", "rate"=>"５", "review"=>"面白い作品だった！", "post_id"=>"{:value=>61}", "controller"=>"reviews", "action"=>"create"} permitted: false>
[2] pry(#<ReviewsController>)>

[sa.k]

あれから試行錯誤してform_forで取れるようになりました！
ありがとうございました。