Couldn't find Room with 'id'=[1, 4] エラー

Question

初心者です。。

やりたいこと
メルカリの取引メッセージ画面のような
指定したの商品のメッセージ画面を作りたい

現状
三層ネスト構造でのルーム作成からのメッセージのやり取りがどうやってやるのかわからず
メッセージ画面に飛ぶ際にエラーが起きる

Couldn't find Room with 'id'=[1, 4]

聞きたいこと
このような場合データベース設計から見直したほうがいいのか
また、三層ネスト構造が可能であるのならばどうやってやればいいのかを知りたい

エラー部　( message  index)

class MessagesController < ApplicationController
  def index
    @room =Room.find(params[:room_id])　　←ここの表記　ids　にしてもダメでした
    @book = Book.find(params[:book_id])
    @message =Message.all
  end

  def new
    @room = Room.find(params[:room_id])
    @book = Book.find(params[:book_id])
    @message = Message.new
  end

book  index画面からメッセージ画面に飛ぶ予定です

      <li class ="btn5"><%= link_to "ロームで会話する？","/books/#{@book.ids}/rooms/#{@room.ids}/messages" %></li><br/>

bookコントローラー

  def index
   @books =Book.all
   @book =Book.includes(:room,:user)
   @room = @book.includes(:room)
   set_book_column 
  end

ルーティング

resources :books do
    resources :rooms, only: [:new, :create,:index] do
     resources :messages,only: [:create, :new,:index,:show]
    end
end

book.rb　　

  belongs_to :user
  has_one :room, dependent: :destroy
  has_many :message, dependent: :destroy


message.rb

 belongs_to :user, optional: true
  belongs_to :book, optional: true
  belongs_to :room, optional: true



room_user.rb　　値が入っている

 belongs_to :room, optional: true
  belongs_to :user, optional: true


room.rb　　　値が入っている

belongs_to :book, optional: true
  has_many :messages, dependent: :destroy
  has_many :room_users
  has_many :users, through: :room_users



user.rb　

         has_many :messages
         has_many :books, dependent: :destroy
         has_many :room_users
         has_many :rooms, through: :room_users

何日も解決してません
足りない情報がありましたら
お声掛けお願いします、、

Answer 1

やりたいことに対して知識が浅く、一度の回答では厳しいと思われるので、方針だけ回答書きます。

三層ネスト構造が可能であるのならばどうやってやればいいのかを知りたい

ネストは深くても実現可能です。

<li class ="btn5"><%= link_to "ロームで会話する？","/books/#{@book.ids}/rooms/#{@room.ids}/messages" %></li><br/>

を見る限り、
ここで気になるのは/books/[1,4] というURLを想定しているのか、いないのか。私の経験上、見たことないので、一般的ではないのではないかと。
一般的によく見受けられるのは/books/1や/books/4などだと思います。
ネストの深さの問題ではなく、URLとparamsの関係性だったり、RESTだったりなど理解されていないのではないかと思います。
下記のURLを一読していただき、まずは理解された方がいいと思います。
https://railsguides.jp/routing.html

その上でネストするかどうかみたいな話になると思います。

実際、やりたいことを実装していくにあたっては、下記の流れになるのかなと。ここは人それぞれなのと、仕様にもよりますが。細かいところが違ってたらすいません。

・[View][/booksもしくは/books/:id] /books/:book_id/rooms(method: post) というリンクをクリック
・[Controller][rooms#create]Bookごとのルームを作成し、/rooms/:room_id/messages/newにリダイレクト
・[View][/rooms/:room_id/messages/new]/rooms/:room_id/messages(method:post)のフォームを用意する
・[Controller][messages#create]ルームごとのメッセージを作成, /rooms/:room_id/messages/new

room_usersは用途不明なので、考慮しませんでした。
発言者とかが必要ということだと思いますが、1:1のメッセージの想定だと思うので、それであれば、roomに出品者_user_idや質問者_user_id、messageに発言者_user_idなどのカラム追加して実装したほうがシンプルでいいと思います。
この辺りもポリモーフィズムを用いたりした方がいいような気もしますが、一度に色々やるよりは一つずつやっていき、リファクタを重ね最後に理解したほうがいいと思うので、かきません。

Comments

[tomsuma]

kojiro様
大変丁寧な対応ありがとうございます！！

かしこまりました。
再び見直した上でやり直させてもらいます！
参考記事までありがとうございました！

Answer 2

<li class ="btn5"><%= link_to "ロームで会話する？","/books/#{@book.id}/rooms/#{@room.id}/messages" %></li><br/>

ids => id

Comments

[tomsuma]

ご回答ありがとうございます！！

undefined method `id' for #<Book::ActiveRecord_Relation:0x00007ff4ae246ee0>
Did you mean? ids

このようにするとこのようなエラーが出てしまいます、、

[nasuk47]

@bookにはBookモデルのインスタンスが入っているのではないのですか？
@bookはどこで定義していますか？

[tomsuma]

bookコントローラー

def index
@books =Book.all
@book =Book.includes(:room,:user)
@room = @book.includes(:room)
set_book_column
end

のようにindexで定義させてもらっています