モデルとモデルの関連付けについて

Question

前提・実現したいこと

rails初心者です。
投稿formでformwithとfields_forを使い、messageモデルとshopモデルを入れてdbを分けている状況です。
そこでユーザーが投稿した内容をmessageモデル、shopモデル揃えてshowページに1件づつ取得し、表示させるような実装を考えております。messageは1件づつ表示させているのですが、そこにshopモデルの内容をどうやって紐づけるのかわかりませんでした。

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

NoMethodError in MessagesController#new

---

マイグレーション

class CreateShops < ActiveRecord::Migration[5.2]
  def change
    create_table :shops do |t|
      t.string "name"  
      t.string "prefecture"   
      t.string "address_city"   
      t.string "address_street"   
      t.string "address"   
      t.float "latitude"
      t.float "longitude"
      t.references :user, foreign_key: true
      t.timestamps
    end
  end
end

マイグレーション

class CreateMessages < ActiveRecord::Migration[5.2]
  def change
    create_table :messages do |t|
      t.string      :name
      t.text        :text
      t.string        :image
      t.timestamps null: true
    end
  end
end

message.rb

class Message < ApplicationRecord
  mount_uploader :image, ImageUploader
  belongs_to :user, optional: true
  belongs_to  :shop, optional: true
  
  accepts_nested_attributes_for :shop
  has_many :likes, dependent: :destroy

  def like_user(user_id)
    likes.find_by(user_id: user_id)
  end
  
end

user.rb

class User < ApplicationRecord
  # Include default devise modules. Others available are:
  # :confirmable, :lockable, :timeoutable, :trackable and :omniauthable
  devise :database_authenticatable, :registerable,
          :recoverable, :rememberable, :validatable
          has_many :messages
end

shop.rb

class Shop < ApplicationRecord
  geocoded_by :address
  after_validation :geocode, if: :address_changed?
  has_many :messages
  belongs_to :user, optional: true
  geocoded_by :address
  after_validation :geocode
end

message.controller

class MessagesController < ApplicationController

  before_action :move_to_index, except: :index

  def index
    @messages = Message.order("created_at DESC").page(params[:page]).per(6)

    respond_to do |format|
      format.html
      format.js
    end
  end

  def new
    @messages = Message.new
    @messages = @messages.shop.build
  end

  def create
    @messages = Message.new(messages_params)
    @messages.save
  end
  

  def show
    @nickname = current_user.nickname
    @messages = Message.find(params[:id])
  end


  def destroy
    message = messages.find(params[:id])
    message.destroy if message.user_id == current_user.id
  end

  def edit
    @messages = Message.find(params[:id])
  end

  def update
    message = Message.find(params[:id])
    if message.user_id == current_user.id
      message.update(messages_params)
    end
  end

  private
  
  def messages_params
    params.require(:message).permit(:text, :image, shop_attributes:[:name, :address, :latitude, :longitude, :message_id ])
  end

  
  def move_to_index
    redirect_to action: :index unless user_signed_in?
  end

end

試したこと

shopモデルのマイグレーションファイルにmessage_idを追加し、ストロングパラメータにもmessage_idを追加しましたが、dbに反映されなかったです。
外部キーreferences型で試してみましたが、user_idは反映されましたが。message＿idは反映されなかったです。
完全に見当違いかもしれませんが、アドバイスよろしくお願いします。。。

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

ここにより詳細な情報を記載してください。

Answer 1

エラーメッセージは１行だけではなく、全体を出してください。他の行にもじゅうような情報が載っています。今回はたまたま２行しか無いのでわかりましたが。

NoMethodError in MessagesController#new
このエラーそのものは
def new @messages = Message.new @messages = @messages.shop.build end
がおかしいからです。これですとmessegeを作るたびにShopを新設することになります。あるshopへのmessageを作るのでしょうから
@shop = Shop.find(ここ） @message = @shop.messages.build
です。「ここ」の部分はこの new を呼ぶviewからshopのidを送って来るようにしてその値を入れます。

Comments

[nakajima1]

回答ありがとうございます。
viewからshop.idを送るというのはfoem.collection_selectを使用して送るのでしょうか？？

[nakajima1]

失礼しました、foem.collection_selectはセレクトメニューの作成する時なんですね、
もう少し調べてみます。
回答ありがとうございました。