【Rails】DM一覧ページでエラーが表示される

Question

前提・実現したいこと

Ruby on Railsでこちらのページ等を参考にメッセージ機能を作りました。上手く実装できたので次に各ユーザーが受け取った「DM一覧ページ」を作っているところです。こちらのページを参考にしています。

しかしルーティングの設定の仕方が悪いのか、エラーが表示されてしまいます。
どなたかお知恵を拝借頂けると幸いです。

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

ActiveRecord::RecordNotFound in RoomsController#show
Couldn't find Room with 'id'=index
Extracted source (around line #10):              

  def show
      **@room = Room.find(params[:id])**
      if Entry.where(:user_id => @current_user.id, :room_id => @room.id).present?
      @messages = @room.messages
      @message = Message.new

rooms_controller.rb

class RoomsController < ApplicationController
  def create
    @room = Room.create
    @entry1 = Entry.create(:room_id => @room.id, :user_id => @current_user.id)
    @entry2 = Entry.create(params.require(:entry).permit(:user_id, :room_id).merge(:room_id => @room.id))
    redirect_to "/rooms/#{@room.id}"
  end

  def show
    @room = Room.find(params[:id])
    if Entry.where(:user_id => @current_user.id, :room_id => @room.id).present?
      @messages = @room.messages
      @message = Message.new
      @entries = @room.entries
    else
      redirect_back(fallback_location: root_path)
    end
  end

  def index
    @currentEntries = @current_user.entries
    myRoomIds = []

    @currentEntries.each do | entry |
      myRoomIds << entry.room.id
    end

    @anotherEntries = Entry.where(room_id: myRoomIds).where('user_id != ?', @user.id)
  end
end

rooms#index.html.erb

<% @anotherEntries.each do |e| %>
  <%= Message.find_by(id: e.room.message_ids.last).content %>
<% end %>

routes.rb

Rails.application.routes.draw do
  post "likes/:post_id/create" => "likes#create"
  post "likes/:post_id/destroy" => "likes#destroy"

  get "login" => "users#login_form"
  post "login" => "users#login"
  post "logout" => "users#logout"

  post "users/:id/update" => "users#update"
  get "users/:id/edit" => "users#edit"
  post "users/create" => "users#create"
  get "signup" => "users#new"
  get 'users/index' => 'users#index'
  get "users/:id" => "users#show"
  get "users/:id/follow" => "users#follow"
  get "/" => "users#top"
  post "users/:id/destroy" => "users#destroy"

  get 'posts/index'=> 'posts#index'
  get "posts/new" => "posts#new"
  get "posts/:id" => "posts#show"
  post "posts/create" => "posts#create"
  get "posts/:id/edit" => "posts#edit"
  post "posts/:id/update" => "posts#update"
  post "posts/:id/destroy" => "posts#destroy"
  get "posts/search" => "posts#search"

  get 'about/about'
  get 'home/top' => 'home#top'
  get 'top' => 'home#top'
  get 'about' => 'about#about'

  resources :users do
    member do
      get :following, :followers, :users_tweets
    end
  end
  resources :relationships, only: [:create, :destroy]

  resources :messages, :only => [:create]
  resources :rooms, :only => [:create, :show, :index]

その後、試したこと

上記の質問をさせて頂いてから、数日が経過しました。

「rooms/index」に飛んでいるにも関わらず「RoomsController#show」でエラーが発生している点がおかしいと思い調べた所、
resourcesでルーティングを定義した場合のエラーについてこちらに書かれていた為、参考にして以下の様にルーティングを変更しました。

  resources :rooms, :only => [:create, :show, :index]
　　↓
  get 'rooms/index' => 'rooms#index'
  resources :rooms, :only => [:create, :show]

すると「RoomsController#index」をしっかりと読み込んでくれました。
しかし今度は「RoomsController#index」内で以下のエラーが発生しました。

NoMethodError in RoomsController#index
undefined method `id' for nil:NilClass
Extracted source (around line #28):

 　@anotherEntries = Entry.where(room_id: myRoomIds).where('user_id != ?', @user.id)
  end
end

自分でも解決に向け色々と調べておりますが、どなたか原因が分かる方がいらっしゃれば教えて頂けると有難いです。

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

ruby 2.6.4p104
RubyGems 3.0.3
Rails 5.2.3

Answer 1

@anotherEntries = Entry.where(room_id: myRoomIds).where('user_id != ?', @user.id)

上記で**where('user_id != ?', @user.id)**とありますが
indexアクション内で＠userが定義されていないためのエラーになっているようです。
適切な@userを設定するのがいいかなーと思います

Comments

[punchan36]

「@user = @current_user」を入れ忘れておりました…。
無事表示されました。初歩的な部分で申し訳ありませんでした。
ご回答有難うございます！

Answer 2

Railsはルーティングをconfig/routes.rbにて設定します
一つ一つのURLに応じたルーティングを施す場合、
get '/index' => 'post#index'
といった風に書きます
上記は、
http//localhost:3000/indexへgetリクエストを受けた時に
post_controllerのindexアクションを実行,
railsが自動でアクションと同名のview(index.html.erb)を呼び出す、
という一連の流れを定めています

一個一個ルーティングするのは面倒なので、
resources :XXX　みたいに
resourcesでRailsに自動で主要なルーティング(RESTful)を生成させる書き方もあります

リンクに関しては、
<%= link_to 'ルート', root_path %>
のようにURLに対応した名前付きルートで書くのが多分一般的です。
コンソールでrails routesを実行した際のPrefix欄に表示されているのが名前付きルートです

Comments

[punchan36]

ご回答有難うございます！
申し訳ありません。routes.rbを載せ忘れておりましたので追記致しました。

resources :rooms, :only => [:create, :show, :index]
ルーティングはこのように書いており、その上で「http://localhost:3000/rooms/index」に飛ぶとエラーメッセージが表示されます（エラーメッセージも追記させて頂きました）。

ルーティング、もしくはどこかのコードが間違っているのでしょうか…。

[H4L]

エラーメッセージはrooms_controller#showを指していますが、
http://localhost:3000/rooms/indexへ飛んだ時のエラーで間違いないでしょうか？

[punchan36]

はい、間違いありません。表示したいページは「index」ですが、なぜかコントローラの「show」の部分でエラー表示が出てしまいます…。

[punchan36]

H4Lさま
その後、ルーティングの編集に関して進展がありましたので追記致しました。
見て頂けると有難いです。