コントローラ内のインスタンス変数をフォームオブジェクトのモデルへ渡したい

Question

コントローラ内で生成した@userのインスタンス変数をフォームオブジェクトへ渡したいのですが、binding.pryで確認するとnilとなってしまいます。userモデルのidとfolderモデルのidを取得して出来ればと考えております。お力添えいただけると幸いです。

DocumentController内のコード

class DocumentsController < ApplicationController
  def index 
    @user_name = current_user.name
  end

  def new
    @document = DocumentFolder.new
  end

  def create
    @document = DocumentFolder.new(document_params)
    @user_id = DocumentFolder.get_user_id(current_user.id)

    if @document.valid?
      @document.save
    else
      render "new"
    end
    
  end

フォームオブジェクト内のコード

class DocumentFolder

  include ActiveModel::Model

  attr_accessor :document_name, :folder_name, :image

  # バリデーション 
  validates :document_name, presence: true
  validates :folder_name,   presence: true
  validates :image,         presence: true
  # validates :folder,        presence: true
  # validates :user,          presence: true

  private
  def self.get_user_id(user_id)
    user_id
  end


  def save
    folder = Folder.create(folder_name: folder_name, user_id: @user_id)
    binding.pry
    Document.create!(document_name: document_name, image: image, user_id: @user_id, folder_id: folder.id)
  end
end

Comments

[maisumakun]

「コントローラ内で生成した@userのインスタンス変数」がコードに存在しないのですが、どれを指しているのでしょうか？

[bata1016]

フォームオブジェクト内の記述に誤りがありましたので、@user_idへ訂正しました。

Answer 1

コントローラ内で生成した@userのインスタンス変数をフォームオブジェクトへ渡したいのですが

コンストラクタなどで、明示的に渡してください。勝手に引き継がれることはありません。

Comments

[bata1016]

各モデル、私の場合documentとfolerモデルにてインスタンス変数を受受け取るメソッドを定義してあげるイメージでよろしいでしょうか？

[maisumakun]

> 各モデル、私の場合documentとfolerモデルにてインスタンス変数を受受け取るメソッドを定義してあげるイメージでよろしいでしょうか？

それは不要です。DocumentFolderで受け取ってください。

[bata1016]

DocumentFolderにてメソッドを追加しましたが、saveメソッドがなぜか定義されていないとエラーが出てしまうようになりました。

[maisumakun]

privateは宣言以降にかかるので、saveもprivateになってしまっています。

（self.get_user_idの「user_id」は何処から供給されるのでしょうか）

[bata1016]

user_idの箇所はdeviseを導入していましたのでcurrent_userメソッドにて取得しました。ご指摘ありがとうございます。
privateないから記述を外に出すことで起動しました。
いただきました指摘により解決いたしました。ありがとうございました。