userの名前を表示させたい

Question

前提

DM一覧を表示させたく、その際にユーザーの名前を表示させたいのですが、
らーが発生しました。

実現したいこと

• userのnameを表示させるために@anotherEntriesから抜き出したいのですが、

@anotherEntriesから相手userの名前を抜き出して表示させることは可能でしょうか？

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

NoMethodError at /rooms
undefined method `id' for #<ActiveRecord::Relation [#<Entry id: 21, user_id: 2, room_id: 24>]>
Did you mean?  ids

該当のソースコード

_room.html.erb

<ul class="list-unstyled">
    <% @anotherEntries.each do |entry| %>
      <li class="mb-3 d-flex">
        <div>
          <div>
              <p><%= link_to @anotherEntries.id.name, user_path(user), class: "text-decoration-none" %></p>
          </div>
          <div>
            <p><%= link_to @room.room_name, room_path(room), class: "text-decoration-none" %></p>
          </div>
        </div>
      </li>
    <% end %>
  </ul>
  <%== pagy_bootstrap_nav(@pagy) %>

試したこと

@anotherEntriesに相手user_idが入っているのでそこから名前を引き出すようにしたく、
@anotherEntries.user_id.nameで引き出そうとしたのですが不可能でした。
またDMの一覧表示させるのに@anotherEntriesで回すのは適切でしょうか？

補足情報

rooms_controller.rb

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 != ?',current_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

Answer 1

@anotherEntriesに相手user_idが入っている

とのことですので、
以下の部分を変更して

<p><%= link_to @anotherEntries.id.name, user_path(user), class: "text-decoration-none" %></p>

下記のように、|entry|で定義された entry からentry.user.nameとするとユーザ情報が取得できると思われます

<p><%= link_to entry.user.name, user_path(entry.user), class: "text-decoration-none" %></p>

ユーザの名前を表示させるために起きているエラーがこれだけとは限らないですが、ひとまず上記が質問の回答です。

そのほか、以下のコードも@roomがControllerで定義されているか怪しい点でいまいちなコードと思われます。
少なくともindexアクションでは@room = 〇〇といったコードはないので、定義されていない可能性があります。

<%= link_to @room.room_name, room_path(room), class: "text-decoration-none" %>

@roomとroomも別物ですので、どちらかに統一するといいです。例えば@roomで統一すると以下のようなコードが一例です。

<% if @room %>
  <%= link_to @room.room_name, room_path(@room), class: "text-decoration-none" %>
<% end %>

Comments

[退会済みユーザー]

回答ありがとうございます。
エラーはなくなったのですが、@room.room_nameが表示できませんでした。
なのでrooms_controller.rbのindexアクションから@room = Room.find(params[:id])でroomの情報を取ろうとしたら
ActiveRecord::RecordNotFound at /rooms
Couldn't find Room without an ID
というエラーが表示されました。
indexアクションで@room = Room.find(params[:id])を行うことは、いけなかったのでしょうか？

[hatsu]

元々の質問時のエラーとは原因が異なり、全部エラーを潰していくのが大変に思えますのである程度で解決済みにされるのがいいと思います。
その上で、コメントの返信させていただきますと

まず
<p><%= link_to @room.room_name, room_path(room), class: "text-decoration-none" %></p>
を削除したら@roomに関するエラーは解決されると思います。
が、ルームの情報は表示されません。

ルームの情報を表示したいならばまたいくつか手段があります。
例えばこちらのように@roomを定義する手段。
> indexアクションで@room = Room.find(params[:id])を行うことは、いけなかったのでしょうか？
これは、params[:id]という値が必要になります。今回のindexアクションにはparams[:id]は空であるため、このコードが動いていないです。
なので、params[:id]を必要としないコードに変える必要があります。

params[:id]を必要とせずにルームの情報を取得するには、例えば以下のような2つの方法があります
- @rooms = Room.allのようにして、View側ではanotherEntriesと同様にeachを使って1つずつルーム情報を表示する
- @room = Room.firstなどのようにして、特定のルーム情報を@roomに代入する
です

[退会済みユーザー]

ありがとうございます。
試してみたいと思います。
また流れを把握できたので良かったです。