編集履歴

質問編集履歴

解決時のソースと検証結果を追記

2020/05/16 01:49

投稿

_Taturon_

スコア17

title CHANGED Viewed

File without changes

body CHANGED Viewed

@@ -149,7 +149,7 @@
 初歩的な質問ですが、どなたかご回答の程、お願い申し上げます。
-### 追記1:sourceモデルに既出のデータがあってもそれと紐付けられません
+### 追記1：sourceモデルに既出のデータがあってもそれと紐付けられません
 次のように実装することで、関連付けたデータの保存はできるようになりました。ありがとうございます。
 ```erb
 <!--フォームパーシャル-->
@@ -235,4 +235,78 @@
     #<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```してしまっているのが原因と思うのですが、解決方法が分かりません。何かアドバイスを頂けないでしょうか？
+おそらく```@memo.sources.find_or_initialize_by```としているので、sourceモデル全体から```find_by```しているのではなく、関連付けられたsourceから```find_by```してしまっているのが原因と思うのですが、解決方法が分かりません。何かアドバイスを頂けないでしょうか？
+### 追記2：解決しました
+winterboumさんのアドバイス通り、下記のように実装することで、重複しているurlはそのままに新規のみ保存できるようになりました。
+ありがとうございました！
+```rb
+#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
+```
+```console
+異なる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">]>
+```

誤字を修正

2020/05/16 01:49

投稿

_Taturon_

スコア17

title CHANGED Viewed

File without changes

body CHANGED Viewed

@@ -215,7 +215,7 @@
 ```
 しかし、find_or_initialize_byを使用しても、既出のurlがあってもそれと紐付けられず、同じurlがsourceモデルに新規保存されそちらと紐付けられてしまいます。
 ```console
-以下フォームから同じ異なるmemoに対し同じurlを登録した場合のモデル内データ
+以下、フォームから異なるmemoに対し同じurlを登録した場合のモデル内データ
 同じurlであってもfind_byされずに新規登録されてしまう
 > Source.all
@@ -235,4 +235,4 @@
     #<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```してしまっているのが原因と思うのですが、解決方法が分かりません。
+おそらく```@memo.sources.find_or_initialize_by```としているので、sourceモデル全体から```find_by```しているのではなく、関連付けられたsourceから```find_by```してしまっているのが原因と思うのですが、解決方法が分かりません。何かアドバイスを頂けないでしょうか？

関連付けに関して追記しました

2020/05/15 23:04

投稿

_Taturon_

スコア17

title CHANGED Viewed

File without changes

body CHANGED Viewed

@@ -147,4 +147,92 @@
 他にも色々試しましたが、どうも上手くsourceモデルとsource_mapモデルにデータが保存されません。
-初歩的な質問ですが、どなたかご回答の程、お願い申し上げます。
+初歩的な質問ですが、どなたかご回答の程、お願い申し上げます。
+### 追記1:sourceモデルに既出のデータがあってもそれと紐付けられません
+次のように実装することで、関連付けたデータの保存はできるようになりました。ありがとうございます。
+```erb
+<!--フォームパーシャル-->
+<%= 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 %>
+```
+```rb
+#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モデルに新規保存されそちらと紐付けられてしまいます。
+```console
+以下フォームから同じ異なる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```してしまっているのが原因と思うのですが、解決方法が分かりません。