railsでユーザーと投稿を紐付ける

Question

#usersテーブルのidをproductsテーブルのuser_idカラムに自動的に追加したい。
@current_userのインスタンスを作って他のページでも自動的にuser_idに追加するようにしましたが、productsのnewページでcreateする時にうまく追加されません。privateメソッドと併用しているのがよくないのでしょうか？コードの書き方がよくないのかと思ったりしますが、自分ではここまでしか分からなかったので、教えていただけたらと思います。

application_controller.rb

class ApplicationController < ActionController::Base
  before_action :set_current_user
  def set_current_user
    @current_user = User.find_by(id: session[:user_id])
  end
end

products_controller

def create
  	 product = Product.new(product_params, user_id: @current_user.id)
    if product.save
	   redirect_to products_path, notice: "保存が成功しました"
    else
       render action: :new, notice: "保存が失敗しました"
    end
  end
private
    def product_params
      params.require(:product).permit(:status, :variety, :name, :price, :limited_number, :first_time, :last_time, :photo, :comment)
    end

Answer 1

導入されてるのでしたらcreateアクションでcurrent_userのidを渡すのではなく、
product_paramsで渡してみてはどうでしょうか？

あと、gemのdeviseは導入されてますか？
導入済みでしたらログインした状態での処理になるのでapplication_controller.rbで宣言した@current_userを使わなくても普通にcurrent_user.idでidを取得できると思いますよ。

def create
       product = Product.new(product_params)
    if product.save
       redirect_to products_path, notice: "保存が成功しました"
    else
       render action: :new, notice: "保存が失敗しました"
    end
  end
private
    def product_params
      params.require(:product).permit(:status, :variety, :name, :price, :limited_number, :first_time, :last_time, :photo, :comment)
      params.merge(user_id: current_user.id)
    end

createアクションでは

product = Product.new(product_params)

product_paramsでは分かりやすいようにparamsの値に改めてcurrent_user.idをマージしていますが、
一行にまとめて

params.require(:product).permit(:status, :variety, :name, :price, :limited_number, :first_time, :last_time, :photo, :comment).merge(user_id: current_user.id)

でも大丈夫です。

@current_userを使いたい場合は、上記コードのcurrent_userと置き換えてみてください。

Comments

[mamikko]

ありがとうございます！！出来ました！すごく分かり易かったです！
gemのdeviseは導入していないんですが、時間のある時に試してみます！

[gnfreeworks]

無事解決してよかったです＾＾
引き続き頑張って下さい！