Ruby 3つの異なるテーブルデータを一度に保存したい

Question

前提・実現したいこと

rubyで、3つの異なるテーブルのデータを一度に保存したい。
モデルFolderのcreateアクション実行時に以下の３つの情報を格納したい

• Folderテーブルに格納すべき情報、
• 1:多数の関係にある親子関係のモデルCardの情報、
• 1:多数の関係にあるモデルFolderUsersの情報

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

エラーは表示されていないが、controllerで@folderが保存できない。

パラメータを確認したところidが全てnilになっている。何故idが生成されないかの理由が判然とせず、またcardsに関しては何一つデータを取得することもできていない

[1] pry(#<FoldersController>)> @folder
=> #<Folder:0x00007f9d009ce7e0 id: nil, name: "aaaa">
[2] pry(#<FoldersController>)> @folder.cards
=> [#<Card:0x00007f9d00a2c2a0 id: nil, omote: nil, ura: nil, folder_id: nil>]
[3] pry(#<FoldersController>)> @folder.folder_users
=> [#<FolderUser:0x00007faa9d074f78 id: nil, folder_id: nil, user_id: 4>]

該当のソースコード

class FoldersController < ApplicationController

  def index
    @folders = Folder.all
  end

  def new
    @folder = Folder.new
    @folder.cards.build
  end

  def create
    @folder = Folder.create(folder_params)
    @folder.folder_users.new(user: current_user)
    if @folder.save
      redirect_to :root, notice: '新しいフォルダを作成しました'
    else
      render :new
    end
  end
  ~~~~~~~~~~~~~~~~~~~~~~~略~~~~~~~~~~~~~~~~~~~~~~~~~~~
  private
  def folder_params
    params.require(:folder).permit(
      :name, 
      cards_attributes:  {
        omote: [], 
        ura: [],
        folder_id: []
      }
    )
  end
end

試したこと

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

>> FolderUser
=> FolderUser(id: integer, folder_id: integer, user_id: integer)
>> Folder
=> Folder(id: integer, name: string)
>> Card
=> Card(id: integer, omote: string, ura: string, folder_id: integer)

  ~~~略~~~
  resources :folders, only: [:index, :new, :create, :edit, :update] do
    resources :cards, only: [:index, :new, :create, :edit]
  end
  ~~~略~~~

アドバイスに従い以下のファイルを追記いたします

= form_with model: folder, local: true do |f|
      = f.label :name, "フォルダ名"
      = f.text_field :name
  = f.fields_for :cards do |c|
        = c.label :omote, "カード（問題）"
        = c.text_area :omote
        = c.label :ura, "カード（解答）"
        = c.text_area :ura
      = f.submit

class Folder < ApplicationRecord
  has_many :folder_users
  has_many :users, through: :folder_users
  has_many :cards, dependent: :destroy
  accepts_nested_attributes_for :cards
  validates :name, presence: true
end

class Card < ApplicationRecord
  belongs_to :folder, optional: true
  validates :omote, presence: true
  validates :ura, presence: true
end

参考にした記事など
https://qiita.com/hmuronaka/items/580b977834d4b3010454
https://qiita.com/tseno/items/70d39f7c0e8e1c779b5e

Comments

[Cojiro]

View, Modelのファイルと参考にした記事があれば記載した方がいいと思います。

[Malson]

kojiro12345様、
アドバイスありがとうございます。補足事項に追記させていただきました。
質問の仕方や、問題がありそうな部位のピックアップなどまだハッキリとはできないため、申し訳ございません。
記載内容でわかりにくいところや、伝えるべきことがあれば御指南いただければと思います。
よろしくお願いいたします。

Answer 1

foldersとcardsにはuser_idがないですが、folder_paramsでuser_idを与えており、Folder.create(folder_params)で、パラメーターをセットしているときにfolderにはuser_idがないというエラーかと思います。
.merge(user_id: current_user.id)を外せば、foldersとcardsの登録できると思います。

folder_usersも登録したいということであれば、
それはそもそも3つのテーブルを同時に登録するということになります。(もしこれがやりたかったことであれば、質問を正しくしましょう。そうでなければ、スルーしてください。)

コントローラーファットになっていくので、
いい方法ではないですが、
手っ取り早くfolder_usersも保存するのは、下記のようにすることでしょうか。

  def create
    @folder = Folder.new(folder_params) #ここでエラー
    @folder.folder_users.new(user: current_user) #追加
    if @folder.save
      redirect_to :root, notice: '新しいフォルダを作成しました'
    else
      render :new
    end
  end

  private
  def folder_params
    params.require(:folder).permit(
      :name, 
      cards_attributes:  {
        omote: [], 
        ura: [],
        folder_id: []
      }
    )
    # .mergeを削除
  end

Comments

[Malson]

ありがとうございます。
私がやりたかったのはご指摘の通り、３つのテーブルにデータを保存するということですので、タイトルから編集いたします。申し訳ございません。

[Malson]

こうして考えると、
ストロングパラメータの属性にFolderUsersも追加して、user_idの置き場所を明確にしてやらないとダメってことなのでしょうか？
携帯からなので、これは後ほどチャレンジしてみたいとは思いますが、どう考えたら良いのでしょうか？

[Cojiro]

> こうして考えると、
というのはどのように考えたのですか？？

[Malson]

実は指摘されるまで、３つのテーブルでということを全く意識してなかったので、

また、中間テーブルは、アソシエーションを組んでモデルでネストすれば、コントローラーで設定しなくても良いという認識がありましたが、そうでないのかもしれないと考えました。

超初心者なので検討外れのことも言うかもしれませんが、よろしくお願いいたします。

[Malson]

kojiro12345様、
ありがとうございました。一歩前進できました。
そして、すいません、全く検討外れのコメントを返していました。
コードを良く読んで理解できました。
いただいたコードですが、newアクションで @folder.users << current_userという定義でも大丈夫かなと思うのですがどうでしょうか？

ただ、エラーは起きなくなったのですが、やはり@folderが保存されずにrenderされてしまっているので何かがおかしいのですね。

[Cojiro]

よかったです。
正直、なんて返せばいいかわからなかったです。

あとすいません。
createアクションの1行目は、
@folder = Folder.new(folder_params)
になります。
修正リクエスト出します。

>いただいたコードですが、newアクションで @folder.users << current_userという定義でも大丈夫かなと思うのですがどうでしょうか？

これは分かりません。やってみてください。

[Malson]

kojiro12345様、

createアクションの1行目も変更してみましたが、保存することができません。newではページに遷移するだけなので、この時点ではパラメータを代入すると、validationに引っかかるんですね。
そこで、presenceを削除すると機能することはするのですが、、、、

@folderは、nameを取得していて、@folder.folder_usersはuser_idを保持していますが、@folder.cardの情報は一切取得できていません。
これはもうviewファイルの書き方に問題があって、入力した情報を取得できていないと考えるべきなのでしょうか？？？

[Cojiro]

> createアクションの1行目も変更してみましたが、保存することができません。

というのは、ログはどうなっていますか？
あとは、@folder.errors.full_messagesはなんと表示されていますか？

あと、
viewの　= form_with model: folder, local: true do |f|　のfolderってFolderのインスタンスですか？
意味がわからなかったらFolder.newに変更してみてください。

[Malson]

createアクションで@folder = Folder.new(folder_params)に変更した上で、保存を実行するとrenderされます。

renderの直前で確認したところ、パラメータの中には情報が格納されていますが、unpermitted parameters: :omote, :uraとなっています。
また、[1] pry(#<FoldersController>)> @folder.errors.full_messages
=> ["Cards omoteを入力してください", "Cards uraを入力してください"]
という表示になっています。Cardの情報だけは取得できていないです。

viewのform_withの記述方法ですが、
モデルのインスタンスを記載するということでしたので何も考えずにfolderとしてました。Folder.newですとcardの入力欄が表示されなくなってしまいました。このとき、Cardの以外は保存することはできます。

[Cojiro]

なるほど。

よくみたらnewアクションで@folder作って、@folderに対してcardsをbuildしてますね。
(Folder.newでうまく行ったら案内しようと思っていました。)

なので、modelを@folderにすれば、うまくいく気がします。

> 保存を実行するとrenderされます。

ごめんなさい。これはちょっと意味がわかりません。何がrenderされるんですか？

[Malson]

kojiro12345様、
遅くまでありがとうございます。

saveされないとrender :newで戻る様にしています。

modelを@folderにしてはみたのですが、やはり同じでunpermitted parameterになってしまっています。

[Cojiro]

よくみたら、ストロングパラメーター、おかしいですね。

cards_attributes: [ :ura, :omote ] にしてみてください。

[Cojiro]

あと、folder_idも。

[Malson]

kojiro12345様、
ありがとうございます。

精進が足りず、もはやどこがおかしいのかすらわかっていないのですが、子ストロングパラメータにfolder_idなどforeign_keyは記述は必要ないということですね。

[ ]を付けたのは、ストロングパラメーターの入力が成功したあとで多数入力できる様に変更したいと考えていて配列を意識したのですが、一つの時は一つの要素しか持たない配列にならないのでしょうか？
それとも属性の名前だけ記述するので、余計ってことでしょうか？

[Malson]

kojiro12345様、
長々とお付き合いいただき、本当にありがとうございました。
無事解決することができました。今回のことはしっかりと復習して身につけます。

[Cojiro]

そうですね。
子モデルの同時登録は、少し時期尚早な気がしました。
今回のこともそうですが、Railsについてもっと基本を学んだ方がいいと思います。

> [ ]を付けたのは、ストロングパラメーターの入力が成功したあとで多数入力できる様に変更したいと考えていて配列を意識したのですが、

やりとりしていて思いましたが、Railsはあなたの思い通りではなく、Railsの仕様通りに動きます。
もちろん私のいう通りでもありません。
思い込みや誤った理解を応用しているように見えます。
思い込みや誤った理解は、そもそも推測であるはずなので、動作して確かめられていないものは動かして確かめる癖をつけた方がいいです。着実に事実に基づいて理解して実装するようにした方が結果として早く身につくと思います。

>[ ]を付けたのは、ストロングパラメーターの入力が成功したあとで多数入力できる様に変更したいと考えていて配列を意識したのですが、一つの時は一つの要素しか持たない配列にならないのでしょうか？
それとも属性の名前だけ記述するので、余計ってことでしょうか？

cardsを複数作るようにして、paramsの中身を見れば解は得られると思います。
が、cardsの数を動的に行うのは厳しいと思うので、2つ分のcardなど数を決めうちでやることをお勧めします。
そうやってステップを作って、確実に仕様を理解していくことをお勧めします。Railsが動くのでRailsで動かして反応を見るのが何よりも信頼できるし、紛れもない事実です。
頑張ってください。