多対多で関連付けたモデルにデータを保存したい

Question

実現したいこと

下記のような設計で、memoを作成するフォームにsourceとsource_mapに関連づけたデータを保存する機能を実装したいです。
※初学者なので、下記ER図は間違っている可能性がございます。

具体的に言うと、次のような流れの機能を実装したいです。
0. memoを新規作成するフォームに行く
0. word,synonym,content,sourceをフォームに入力する
0. 保存ボタンをクリックする
0. MemosControllerにて入力した値が各モデル（memo,source_map,source）に保存される
0. 作成したmemoが表示されるページにリダイレクト
0. 表示されているmemoをクリックすると詳細ページへリダイレクト
0. 保存した各値が表示される

上記の内２.の「sourceを入力するフォーム」と４.の「入力したsourceの値をmemoと関連付けて（中間テーブルにidペアを保存して）保存する機能」を実装する方法をご教授願いたいです。

実行環境

・Rails 6.0.0
・ruby 2.6.3p62 (2019-04-16 revision 67580) [x86_64-linux]
・Cloud 9

ソースコード

<!--フォームパーシャル-->

    <%= form_with model: @memo, local: true do |f| %>
      <div class="form-group">
        <%= f.label :word, "単語" %>
        <%= f.text_field :word, class: "form-control",
            placeholder: "登録したい単語を記入して下さい" %>
      </div>
      
　・
　・
　・

      <div class="form-group">　<!--このdivタグ内の記述方法が分かりません-->
        <%= f.label :source, "出典（URL）" %>  
        <%= f.url_field :source, class: "form-control",
            placeholder: "出典元のURLを記入して下さい" %>
      </div>
      
      <div class="form-group">
        <%= f.submit yield(:btn_message),
　　　　　　　class: "btn btn-success btn-lg btn-block" %>
      </div>
    <% end %>

#memosコントローラー
Class MemosController < ApplicationController

　・
　・
　・

  def new
    @memo = Memo.new
  end
  
  def create
    @memo = Memo.new(memo_params)
    if @memo.save
      flash[:notice] = "新しいMemoを作成しました！"
      redirect_to root_url
    else
      flash.now[:alert] = "無効な送信です"
      render :new
    end
  end

　・
　・
　・

  def update
    @memo = Memo.find(params[:id])
    if @memo.update(memo_params)
      redirect_to root_url
      flash[:notice] = "Memoを更新しました！"
    else
      flash.now[:alert] = "無効な送信です"
      render :edit
    end
  end
  
　・
　・
　・
  
  private
  
    def memo_params #ストロングパラメーターの設定方法も分かりません
      params.require(:memo).permit(:word, :content, :synonym, :tag_list, :source)
    end
    
end

#ルーティング
Rails.application.routes.draw do
  root 'memos#index'
  resources :memos
end

#各モデルの関連付け

#memoモデル
class Memo < ApplicationRecord
  has_many :source_maps, dependent: :destroy
  has_many :sources, through: :source_maps

　・
　・
　・

end

#source_mapモデル
class SourceMap < ApplicationRecord
  belongs_to :memo
  belongs_to :source
end

#sourceモデル
class Source < ApplicationRecord
  has_many :source_maps, dependent: :destroy
  has_many :memos, through: :source_maps
end

試したこと

・上記のようにフォームに:sourceを渡してみました
フォームのビューでは何故か「出典（URL）」の入力ウィンドウに
Source::ActiveRecord_Associations_CollectionProxy:0x00007fc51eb7afd0>
とデフォルトで表示されてしまいます。

上記を消し、URLを入力して保存すると
NoMethodError in MemosController#create
undefined method `each' for "（入力したURL）":String
のエラーが発生します。

何故突然eachが出てくるのか？と戸惑っております。

他にも色々試しましたが、どうも上手くsourceモデルとsource_mapモデルにデータが保存されません。

初歩的な質問ですが、どなたかご回答の程、お願い申し上げます。

追記1：sourceモデルに既出のデータがあってもそれと紐付けられません

次のように実装することで、関連付けたデータの保存はできるようになりました。ありがとうございます。

<!--フォームパーシャル-->
<%= form_with model: @memo, local: true do |f| %>

　・
　・
　・

  <div class="form-group">
    <%= f.label :url, "出典（URL）" %>
    <%= f.url_field :url, class: "form-control",
        placeholder: "出典元のURLを記入して下さい" %>
  </div>
  <div class="form-group">
    <%= f.submit yield(:btn_message), class: "btn btn-success btn-lg btn-block" %>
  </div>
<% end %>

#memosコントローラー
class MemosController < ApplicationController

　・
　・
　・

  def new
    @memo = Memo.new
    @source = @memo.sources.build
  end
  
  def create
    @memo = Memo.new(memo_params.slice(:word, :content, :synonym, :tag_list))
    if @memo.save
      @source = @memo.sources.find_or_initialize_by(url: memo_params[:url])
      @memo.sources << @source
      flash[:notice] = "新しいMemoを作成しました！"
      redirect_to root_url
    else
      flash.now[:alert] = "無効な送信です"
      render :new
    end
  end

　・
　・
　・  

  private
  
    def memo_params
      params.require(:memo).permit(
        :word,
        :content,
        :synonym,
        :tag_list,
        :url
      )
    end
    
end

しかし、find_or_initialize_byを使用しても、既出のurlがあってもそれと紐付けられず、同じurlがsourceモデルに新規保存されそちらと紐付けられてしまいます。

以下、フォームから異なるmemoに対し同じurlを登録した場合のモデル内データ

同じurlであってもfind_byされずに新規登録されてしまう
> Source.all
   (0.8ms)  SELECT sqlite_version(*)
   Source Load (0.5ms)  SELECT "sources".* FROM "sources" LIMIT ?  [["LIMIT", 11]]
 => #<ActiveRecord::Relation [
    #<Source id: 1, url: "https://rails.densan-labs.net/form/relation_regist...", created_at: "2020-05-15 22:31:03", updated_at: "2020-05-15 22:31:03">,
    #<Source id: 2, url: "https://rails.densan-labs.net/form/relation_regist...", created_at: "2020-05-15 22:31:11", updated_at: "2020-05-15 22:31:11">]>

関連付けも、本来なら
SourceMap id: 1, memo_id: 1, source_id: 1
SourceMap id: 1, memo_id: 2, source_id: 1
となって欲しいが、新規登録されたsource_id: 2と関連付けられてしまう
> SourceMap.all
  SourceMap Load (0.2ms)  SELECT "source_maps".* FROM "source_maps" LIMIT ?  [["LIMIT", 11]]
 => #<ActiveRecord::Relation [
    #<SourceMap id: 1, memo_id: 1, source_id: 1, created_at: "2020-05-15 22:31:03", updated_at: "2020-05-15 22:31:03">,
    #<SourceMap id: 2, memo_id: 2, source_id: 2, created_at: "2020-05-15 22:31:11", updated_at: "2020-05-15 22:31:11">]> 

おそらく@memo.sources.find_or_initialize_byとしているので、sourceモデル全体からfind_byしているのではなく、関連付けられたsourceからfind_byしてしまっているのが原因と思うのですが、解決方法が分かりません。何かアドバイスを頂けないでしょうか？

追記2：解決しました

winterboumさんのアドバイス通り、下記のように実装することで、重複しているurlはそのままに新規のみ保存できるようになりました。
ありがとうございました！

#memosコントローラー
class MemosController < ApplicationController

　・
　・
　・

  def new
    @memo = Memo.new
    @source = @memo.sources.build
  end
  
  def create
    @memo = Memo.new(memo_params.slice(:word, :content, :synonym, :tag_list))
    if @memo.save
      @source = Source.find_or_create_by(url: memo_params[:url])
      @memo.sources << @source
      flash[:notice] = "新しいMemoを作成しました！"
      redirect_to root_url
    else
      flash.now[:alert] = "無効な送信です"
      render :new
    end
  end

　・
　・
　・  

  private
  
    def memo_params
      params.require(:memo).permit(
        :word,
        :content,
        :synonym,
        :tag_list,
        :url
      )
    end
    
end

異なるmemoに同じurlを関連付けて保存した場合
> SourceMap.all
  SourceMap Load (0.5ms)  SELECT "source_maps".* FROM "source_maps" LIMIT ?  [["LIMIT", 11]]
 => #<ActiveRecord::Relation [
    #<SourceMap id: 1, memo_id: 1, source_id: 1, created_at: "2020-05-16 01:35:29", updated_at: "2020-05-16 01:35:29">,
    #<SourceMap id: 2, memo_id: 2, source_id: 1, created_at: "2020-05-16 01:35:38", updated_at: "2020-05-16 01:35:38">]>
> Source.all
   (1.0ms)  SELECT sqlite_version(*)
  Source Load (0.4ms)  SELECT "sources".* FROM "sources" LIMIT ?  [["LIMIT", 11]]
 => #<ActiveRecord::Relation [
    #<Source id: 1, url: "https://rails.densan-labs.net/form/relation_regist...", created_at: "2020-05-16 01:35:29", updated_at: "2020-05-16 01:35:29">]>

続けて異なるmemoを異なるurlで関連付けて保存した場合
> SourceMap.all
  SourceMap Load (0.2ms)  SELECT "source_maps".* FROM "source_maps" LIMIT ?  [["LIMIT", 11]]
 => #<ActiveRecord::Relation [
    #<SourceMap id: 1, memo_id: 1, source_id: 1, created_at: "2020-05-16 01:35:29", updated_at: "2020-05-16 01:35:29">,
    #<SourceMap id: 2, memo_id: 2, source_id: 1, created_at: "2020-05-16 01:35:38", updated_at: "2020-05-16 01:35:38">,
    #<SourceMap id: 3, memo_id: 3, source_id: 2, created_at: "2020-05-16 01:37:04", updated_at: "2020-05-16 01:37:04">]>
> Source.all
  Source Load (0.2ms)  SELECT "sources".* FROM "sources" LIMIT ?  [["LIMIT", 11]]
 => #<ActiveRecord::Relation [
    #<Source id: 1, url: "https://rails.densan-labs.net/form/relation_regist...", created_at: "2020-05-16 01:35:29", updated_at: "2020-05-16 01:35:29">,
    #<Source id: 2, url: "https://teratail.com/questions/261655#", created_at: "2020-05-16 01:37:04", updated_at: "2020-05-16 01:37:04">]>

Answer 1

通常の多対多と違うのでちと戸惑っています。
この構成で多対多というと通常は既存のSourceとMemoとを関係つけます。
今回は、Memoと同時にSourceも新規登録ということで、ふぅ~~む。

これですと同じURLからことなるMemoを作ってもSourceが作られることになるのでは？それとも同じURLであったら新規は作らない？

実際のデータをイメージすると、単に Memoにurlというfieldをつけるだけで良いのでは？とおもいますが、多対多にする意図はなんでしょう

追記

if @memo.save
  @source = Source.find_or_create_by(url: memo_params[:url])
  @memo.sources << @source

Comments

[_Taturon_]

winterboumさん、ご回答ありがとうございます。

多対多にしたい意図は、
1.ひとつのmemoに複数のurlを紐付けたい
2.複数のmemoに同じurlが登録される可能性がある
3.多対多関連付けを学習する
ためです。

ご回答の意味は、通常の多対多では、
「複数のモデルに対して一つのフォームから同時にデータを保存するのは一般的ではない」
ということでしょうか?

その場合、新たにSourcesControllerとビューを追加し、urlを別途保存するフォームを作ると実装出来そうと感じました。

しかし、出来れば一つのフォーム、一回の保存で登録したいのですが、やはり難しいでしょうか...

[_Taturon_]

ご回答を読んでいると、実装したい機能がより明確になりました。

まずフォームを複数追加し、
可能であれば、
1.入力したurlがsourceモデルにあった場合はそのidと紐付ける
2.入力したurlがsourceモデルになかった場合は新規登録して紐付ける
みたいな機能が欲しいですが、難しいでしょうか。

if文を使って、入力された内容をfindで検索し、あった場合はその内容を削除かつ見つかったidで紐付け、なかった場合はそのまま保存
みたいなスキームが見えましたが、実装方法がいまいち分かりません...

[winterboum]

Railsが用意しているスマートな方法はつかえないですが、泥臭くやれば可能です。
find_or_initialize_by とか　find_or_create_by を調べてみてください
それで手に入れた sourcesのインスタンスを ｍｅｍｏ.sources << インスタンス
で登録できます。

[_Taturon_]

winterboumさん、ご返信ありがとうございます。
早速調べてみます。

[_Taturon_]

winterboumさんのアドバイスを基に、追記1のように実装してみたのですが、既出のurlと関連付ける方法がいまいち分かりません。
恐縮ですが、もう少しアドバイスして頂くことは可能でしょうか？

[_Taturon_]

winterboumさんの追記情報通りに実装することで、機能を実現できました。
ご回答ありがとうございました！