【Rails】モデルの関連付け(アソシエーション)がうまくいかない

前提

ドロップダウンリスト(collection_select使用してIdolテーブルのnameカラムから選択させる)
から選択した値をImageテーブルのnameカラムに格納させたいです。

実現したいこと

テーブル①のデータを投稿フォームのドロップリストから選択させて、テーブル②に登録させたいです。

投稿フォームのドロップダウンリスト(collection_select)から選択させて、実際投稿(テーブル②)すると変な文字列が保存されてしまいます。⇒ #Idol::ActiveRecord_Relation:0x00007faf62a68538

この文字列はどこからやってきたのか、原因がわからないので解決方法を教えていただきたいです。

該当のソースコード

views/iimages/index.html.erb

  <%= form_with model: @image, local: true do |form| %>
～省～
　<!--ドロップダウンリスト　テーブル：Idol, カラム：idol_name-->
  <%= form.collection_select :idol_name, Idol.all, :idol_name, :idol_name, :include_blank => true %>
  <% end %>


　<!--投稿されたものを表示-->
  <% @images.each do |image| %>
～省～
    <%= image.idolname %>
  <% end %>

views/controllers/images_controller.rb

  def index
    #全ての投稿内容を表示
    @images = Image.all.order(id: "DESC")
    #新しい投稿
    @image = Image.new
  end

 def create
    @image = Image.new(image_params)
    
    #Idolにストロングパラメータの値が存在するか
    @image.idolname = Idol.where(idolname: params[:idol_name])

    if @image.save
      redirect_to root_url, notice: 'Add Image'

    else
      #再度indexページを表示
      @images = Image.all
      render :index
    end
  end

試したこと

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

情報が足りない場合は追加します。よろしくお願い致します。

winterboum

2019/12/25 21:38

app/models/idle.rb, image.rb を質問に追加してください

Boston12121

2019/12/26 08:01 編集

モデルのファイルを載せるのをわすれていたので追記します。すみません。多対多で関連付けしています。 app/models/idl.rb class Idol < ApplicationRecord has_many :image_idol has_many :images, through: :image_idol end app/models/image.rb class Image < ApplicationRecord has_one_attached :avatar validate :avatar_presence has_many :image_idol has_many :idols, through: :image_idol validates :name, length: { minimum: 0, maximum: 15 } validates :comment, length: { minimum: 0, maximum: 100 } validates :avatar, presence: true end app/models/image_idol.rb class ImageIdol < ApplicationRecord belongs_to :image belongs_to :idol end

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

回答1件

ベストアンサー

多分ここですね。
@image.idolname = Idol.where(idolname: params[:idol_name])
右辺は idol の名前でもなければ、 idol でもないです。まさにIdol::ActiveRecord_Relation です。

まず、

params[:idol_name] で名前がくるのですから @image.idolname = params[:idol_name] で良いのでは？
もし、paramsで来るのが自由入力などで、登録されていないものが来たら困るのでdatabaseの検索をしたいのだ、ということでしたら

@image.idolname = Idol.where(idolname: params[:idol_name]).first.try(:idolname) か
@image.idolname = Idol.find_by(idolname: params[:idol_name]).try(:idlolname) ですね。
try はもしDBになかった時検索結果が nil になるので、その対策です

投稿2019/12/30 04:36

winterboum

総合スコア23329

Boston12121

2019/12/30 05:08

返答ありがとうございます！ @image.idolname = params[:idol_name]でやってみたのですが、DBにはなにも保存されませんでした。 params[:idol_name]にドロップリストから選択した項目を左辺に格納するはずですよね？やはり関連付けの仕方がおかしいのかな。もう一度関連付けから見直してみます！！回答本当にありがとうございます。