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

#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

行動規範の内容に同意します

回答1件

ベストアンサー

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

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

ruby
1def create
2       product = Product.new(product_params)
3    if product.save
4       redirect_to products_path, notice: "保存が成功しました"
5    else
6       render action: :new, notice: "保存が失敗しました"
7    end
8  end
9private
10    def product_params
11      params.require(:product).permit(:status, :variety, :name, :price, :limited_number, :first_time, :last_time, :photo, :comment)
12      params.merge(user_id: current_user.id)
13    end

createアクションでは

ruby
1product = Product.new(product_params)

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

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