[rails] idの取得をし、紐付けしたい

Question

ユーザーが投稿した記事（コーディネート全体）に、そのユーザーがアイテムを追加するページを作成したのですが
投稿した記事とアイテムページの紐付けがうまく行かず、undefined method `id' for nil:NilClass
が出てしまいます、、、

• Postモデルのid
• Itemモデルのpost_id

上記を紐付けしたいです。

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

NoMethodError in ItemsController#create
undefined method `id' for nil:NilClass

Extracted source (around line #33):            

31    @item = Item.new(item_image: params[:item_image], brand: params[:brand], size: params[:size], price: params[:price], 
32                     products_name: params[:products_name], post_id: params[:post_id])
33    @item.post_id = "#{@post.id}"
34    @item.user_id="#{current_user.id}"
35    @item.save!
36    redirect_to("/items/#{@item.post_id}/show")

Rails.root: /myapp

Application Trace | Framework Trace | Full Trace
app/controllers/items_controller.rb:33:in `create'
Request
Parameters:

{"authenticity_token"=>"ODNkP4OO3pO9TtOXKvEDSeG/eOPDBlrpOS/1JeKHpryzOIaWBd1oSXMyyOHsmMO2HpSeW87E/GSmzzqwpRQVKQ==",
 "products_name"=>"テント", "brand"=>"snowpeak", "size"=>"M", "price"=>"300000"}

該当のソースコード

#####ItemsController

class ItemsController < ApplicationController
  def index
    @user = current_user
    @items = @user.items   
  end

  def new
    @item = Item.new
  end

  def create
    @item = Item.new(item_image: params[:item_image], brand: params[:brand], size: params[:size],
                     price: params[:price], products_name: params[:products_name], post_id: params[:post_id])
    @item.post_id = "#{@post.id}"
    @item.user_id="#{current_user.id}"
    @item.save!
    redirect_to("/items/#{@item.post_id}/show")
  end

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

追記

Answer 1

dockerを再起動させましたら思い通りに動作いたしました。
お騒がせいたしました、、、

Answer 2

エラーメッセージのとおりです。@postが定義されていませんのでnilです。

Comments

[退会済みユーザー]

コメントありがとうございます。
@post = Post.find_by(id: params[:id])をcreateの中に記載したのですが同じようにNilになってしまいます、、、、

[maisumakun]

そもそも論として、上のItem.createでpost_idを指定してしますので、さらに重ねて指定する必要があるのか自体が疑問です。

[退会済みユーザー]

ご回答ありがとうございます。
コントローラを修正し、下記のようにしました。
def new
@post = Post.find(params[:post_id])
@item = Item.new
end

def create
@item = Item.new(item_image: params[:item_image], brand: params[:brand], size: params[:size], price: params[:price], products_name: params[:products_name], post_id: params[:post_id])
@post = Post.find(params[:post_id])
@item.user_id = current_user.id
if @item.save
redirect_to("/items/#{@item.post_id}/show")
else
render("/items/new")
end
end

そして、formでpost_idを飛ばしていなかったので
<%= hidden_field_tag :post_id, @post.id %>
上記でとばしているのですが、

ActiveRecord::RecordNotFound in ItemsController#new
Couldn't find Post without an ID
とエラーがでてしまいます。
def newでの＠postの定義が違うのでしょうか、？

[maisumakun]

def newは「新規作成ページの表示」ですが、その際にparams[:post_id]はどのような形で与えられる、と考えていますか？

[退会済みユーザー]

コーディネート詳細画面のアイテム登録ボタンを押下し、アイテム登録フォームで<%= hidden_field_tag :post_id, @post.id %>としているので、コーディネート詳細のid(postのid)が与えられるのではないかと考えております。

[maisumakun]

> コーディネート詳細画面のアイテム登録ボタンを押下し

この部分はどのようなコードを書いていますか？

[退会済みユーザー]

<%= link_to("アイテム登録をする", items_new_path(:post_id => @post.id)) %>
このように記述しております。

[maisumakun]

確かに渡しているようですね。

（Couldn't find Post without an IDのエラーは、コントローラーの「@post = Post.find(params[:post_id])」に対して出ていますか？それとも、別な行に対して出ていますか？）

[退会済みユーザー]

サーバーを再起動したところ、エラーが変わり、
def newの@post = Post.find(params[:post_id])に

ArgumentError in ItemsController#new
When assigning attributes, you must pass a hash as an argument, String passed.
とエラーが出るようになりました。
画像追加しております。

[maisumakun]

エラーの内容とコードがマッチしていませんね…

[shinoharat]

横から失礼します。
Post モデルのコードを見せていただけますか？

[退会済みユーザー]

shinoharat様

Post.rbのコードになります。（一応、画像も追加しました。）

class Post < ApplicationRecord
belongs_to :user
has_many :items, inverse_of: :post, dependent: :destroy

validates :user_id, {presence: true}

mount_uploader :image, ImageUploader

def user
return User.find_by(id: self.user_id)
end
end

[shinoharat]

うーん、、、現在

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

の部分で

When assigning attributes, you must pass a hash as an argument, String passed.

が発生しているとのことですが、通常 find メソッドでこのようなエラーが出るのは考えづらいです。
なので、モデル側に何か特殊なコードがあるのかと思ったのですが、見た感じ原因となるコードはなさそうですね。

お手数ですが、念の為 ApplicationRecord も見せてもらえませんか？

--

ちなみに、エラーとは直接関係ないと思いますが、

def user
return User.find_by(id: self.user_id)
end

の部分は不要です。「belongs_to :user」が同じ機能のメソッドを自動で定義してくれます。

[退会済みユーザー]

shinoharat様

dockerを再起動させましたら思い通りに動作いたしました。
お騒がせいたしました、、、

belongs_to :userの件、ご指摘ありがとうございます！！
belongs_to、has_manyについてもう少し勉強したいとおもいます！