undefined method `user_id' for nil:NilClassについて

Question

困っている箇所

rooms_controllerのindex箇所の掲載者のuserを入れる@anotherEntriesの箇所でエラーが起こってしまいました。
undefined method `user_id' for nil:NilClass

エラー

NoMethodError at /rooms
undefined method `user_id' for nil:NilClass

Hint: Something is `nil` when it probably shouldn't be.

rooms_controller

class RoomsController < ApplicationController
  before_action :authenticate_user!
  
  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 != ?',@plan.user_id)
  end
  
  def create
    @plan = Plan.find(params[:entry][:plan_id])
    @room = Room.create(room_name: @plan.title)
    @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
  
  private
  
  def set_plan
      @plan = Plan.find(params[:id])
  end
  
end

room.rb

class Room < ApplicationRecord
    has_many :entries, dependent: :destroy
    has_many :messages, dependent: :destroy
    has_many :users
    has_many :plans
end

views/rooms/index.html.erb

<div message-list>
    <h4>メッセージ一覧</h4>
    <% @my_rooms.each do |room| %>
      <p><%= render 'rooms', rooms: @rooms %></p>
    <% end %>
</div>

質問

@anotherEntries = Entry.where(room_id: myRoomIds).where('user_id != ?',@plan.user_id)
で掲載者のuserを入れたいのですが、user_idがnilの為、@plan = Plan.find(params[:id])でplan_idからplanを抜き出し、さらにそこからplan.user_idを抜き出して@anotherEntries のuser_idに代入させようとしたのですが、

ActiveRecord::RecordNotFound at /rooms
Couldn't find Plan without an ID

と表示され,調べたところルーティングに関しての情報が多くあったのですが、
ルーティングを見ても問題ないと思います。
@plan = Plan.find(params[:id])でレコードから抜き出すやり方は正しいでしょうか？

!追記
routes.rb

Rails.application.routes.draw do
  devise_for :users, controllers: { registrations: 'users/registrations' }
  root to: 'toppages#index'

  get "users/show" => "users#show"
  devise_scope :users do
    get '/users', to: redirect("/users/sign_up")
  end
  resources :users do
    member do
      get :followings
      get :followers
      get :likes
    end
    collection do
      get :search
    end
  end
  resources :users
  resources :plans 
  resources :relationships, only: [:create, :destroy]
  resources :favorites, only: [:create, :destroy]
  resources :messages, only: [:create]
  resources :rooms
end

routes.rb

rooms GET    /rooms(.:format)                rooms#index
                         POST   /rooms(.:format)                rooms#create
                new_room GET    /rooms/new(.:format)            rooms#new
               edit_room GET    /rooms/:id/edit(.:format)       rooms#edit
                    room GET    /rooms/:id(.:format)            rooms#show
                         PATCH  /rooms/:id(.:format)            rooms#update
                         PUT    /rooms/:id(.:format)            rooms#update
                         DELETE /rooms/:id(.:format)            rooms#destroy

Answer 1

@plan = Plan.find(params[:id])でレコードから抜き出すやり方は正しいでしょうか？

そのコードが呼ばれていません。set_planメソッドは存在しますが、どこからも呼ばれていないようです。

Comments

[退会済みユーザー]

before_action :set_plan, only: [:index]を追加したのですが、
Couldn't find Plan without an IDと表示されました。
plan.user_idを@anotherEntriesに代入したいのですが、planを取得するやり方としては正しいですか？

[maisumakun]

「どの」Planを取得したいのですか？

（indexは全件をリストアップするためのアクションなので、エラーメッセージの通り、一般にidの値は流れて来ません）

[退会済みユーザー]

current_userが入っているroomと一緒に入っている掲載者のuserのid。
つまりPlan(掲載)の中にあるuser_idを取得したいです。
user_id != ?',@plan.user_id
ここの箇所が恐らく問題だと思うので、planを取得しなければならないと感じたのですが違いますでしょうか？

[maisumakun]

@planは使わず、単にcurrent_user.idを入れる、というのは意図と違いますか？

[退会済みユーザー]

サイトを見て一覧作成を進めていたのですが、おそらく
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 != ?',@plan.user_id)
end

@currentEntries = current_user.entries←自分のユーザー
@anotherEntries = Entry.where(room_id: myRoomIds).where('user_id != ?',@plan.user_id)←相手のユーザー
という図式かなと思ったのですが、この考え方は違いますか？

[maisumakun]

「相手のユーザー」を取りたいのであれば、条件は「自分以外」なので、current_user.idを入れればいいかと思います。

[退会済みユーザー]

なるほど！

すみません。
もう１つ追加で質問なのですがその一覧にルームの名前と相手userの名前を表示させようとする際に、
その場合は例えば相手の名前を表示したい場合@anotherEntries.user.nameで大丈夫でしょうか？

[maisumakun]

> @anotherEntries.user.nameで大丈夫でしょうか？

いえ、名前の通り@anotherEntriesには複数入る可能性がありますので、1つずつ処理する必要があります。

[退会済みユーザー]

<% rooms.each do |room| %>で１つずつ処理させて
@user.nameで表示しようかなとも考えたのですがそれでは処理されませんか？

[maisumakun]

roomsとは何ですか？

そして、roomに代入されるループをいくら回そうが、@userの中身は変化しません。

[退会済みユーザー]

間違えました。。。
<% @anotherEntries.each do |room| %>です。

[退会済みユーザー]

<% @anotherEntries.each do |entry| %>
にしてみたところエラーはなくなり、user.nameの箇所のエラーで止まっているのですが、
上記の回し方は適切でしょうか？
user.nameの箇所のエラーにundefined method `id' for #<ActiveRecord::Associations::CollectionProxy [#<Entry id: 22, user_id: 1, room_id: 24>]>と書いてあるのでentryで回すのが適切なのかと思ったのですが。。。

[maisumakun]

もとの質問から離れていっているので、別に質問を立てて仕切り直ししたほうがいい気もします。

[退会済みユーザー]

回答ありがとうございます。
別の質問を立てることにします。
まずはエラーが治ったので一安心しました！
本当にありがとうございます！