質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.50%
Ruby

Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

Q&A

解決済

1回答

3340閲覧

【Rails】DM一覧ページにおける、各ルーム内の最新メッセージの取得方法

punchan36

総合スコア105

Ruby

Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

0グッド

0クリップ

投稿2020/03/11 10:00

編集2020/03/20 06:34

前提・実現したいこと

Railsでメッセージ機能を実装しました。
メッセージ一覧を「rooms#index」にて作成しましたが、チャットルーム内の最新メッセージの表示に関して困っております。

Ruby

1<% @rooms.each do |room| %> 2 <%= room.messages.order(created_at: :desc).take.try(:content) %><br> 3<% end %><br>

この様に書く事で各ユーザーの最新のメッセージを一気に取得する事が出来ましたが、この中から特定のルーム内の最新メッセージを抜き取る方法が分かりません。
以下が関連のコードになりますが、当然の事ながら上記のコードを組み込むと、各ルーム内に、取得した全てのルームでの最新メッセージが繰り返し表示されてしまいます。

LINEの様にうまく各最新メッセージを表示する方法はありませんでしょうか。どなたかお知恵を拝借頂けると幸いです。

models#room.rb

Ruby

1class Room < ApplicationRecord 2 has_many :messages, dependent: :destroy 3 has_many :entries, dependent: :destroy 4end

rooms_controller.rb

Ruby

1class RoomsController < ApplicationController 2 def create 3 @room = Room.create 4 @entry1 = Entry.create(:room_id => @room.id, :user_id => @current_user.id) 5 @entry2 = Entry.create(params.require(:entry).permit(:user_id, :room_id).merge(:room_id => @room.id)) 6 redirect_to "/rooms/#{@room.id}" 7 end 8 9 def show 10 @room = Room.find(params[:id]) 11 if Entry.where(:user_id => @current_user.id, :room_id => @room.id).present? 12 @messages = @room.messages 13 @message = Message.new 14 @entries = @room.entries 15 else 16 redirect_back(fallback_location: root_path) 17 end 18 end 19 20 def index 21 @rooms = Room.all 22 @user = @current_user 23 @currentEntries = @current_user.entries 24 myRoomIds = [] 25 @currentEntries.each do | entry | 26 myRoomIds << entry.room.id 27 end 28 @anotherEntries = Entry.where(room_id: myRoomIds).where('user_id != ?', @user.id) 29 end 30end

rooms#index.html.erb

Ruby

1<h1 class="users-heading">メッセージ一覧</h1> 2<% @anotherEntries.each do |e| %> 3<div class="users-index-item"> 4<!-- 左側に最新メッセージを送ったユーザーのプロフィール画像 --> 5 <div class="user-left"> 6 <%= link_to room_path(e.room.id) do %> 7 <img src="<%= "/user_images/#{e.user.image_name}" %>"> 8 <% end %> 9 </div> 10<!-- 右側に最新メッセージを送ったユーザーの名前及び、そのメッセージ --> 11 <div class="user-right"> 12 <%= link_to room_path(e.room.id) do %> 13 <%= e.user.name %>とDMをしよう!<br> 14 <% end %> 15 <% @rooms.each do |room| %> 16 <%= room.messages.order(created_at: :desc).take.try(:content) %><br> 17 <% end %><br> 18 </div> 19</div> 20<% end %>

必要なコード等がありましたら都度追記致します。宜しくお願い致します。

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

ruby 2.6.4p104
RubyGems 3.0.3
Rails 5.2.3

【その後】

「木を見て森を見ず」の状態になっていました。
rooms#index.html.erbにおいて、each文の中にeach文を組み込んでいた為、必然的に取得した全てのルームの最新メッセージが繰り返し表示されてしまっていただけでした。
なので以下で使用していたeach文を取り除く事で解決致しました。

Ruby

1 <% @rooms.each do |room| %> 2 <%= room.messages.order(created_at: :desc).take.try(:content) %><br> 3 <% end %><br>

Ruby

1 <%= room.messages.order(created_at: :desc).take.try(:content) %><br>

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

ベストアンサー

EntryはUserとRoomを結びつける中間テーブルということで間違いないでしょうか?
そうでしたら、UserとRoomに、Entryを経由したリレーションを貼ったほうがやりやすい気がします。
リレーションの例を↓に記載します。

user.rb

1class User < ApplicationRecord 2 has_many :entries, dependent: :destroy 3 has_many :rooms, through: :entries

entry.rb

1class Entry < ApplicationRecord 2 belongs_to :user 3 belongs_to :room

room.rb

1class Room < ApplicationRecord 2 has_many :entries, dependent: :destroy 3 has_many :users, through: :entries

view側もEntryをeachするのではなく、Roomをeachして処理を考えた方が良さそうです。
チャットルーム毎に最新のメッセージを表示したい、ということだと思うので。

投稿2020/03/11 11:50

ymneet

総合スコア154

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

punchan36

2020/03/13 23:27

ご回答有難うございます! 返答が遅くなり申し訳ありません。頂いたご回答を参考に試行錯誤しておりました。 コントローラで定義している「@anotherEntries = Entry.where」の部分も含め、「rooms#index.html.erb」は全体としてRoomをeachする方法で定義・表示をし直した方が良いと言う解釈で間違っておりませんでしょうか。 ご回答の様にモデル(UserモデルとRoomモデル)の関係性は既に明示しました。今多対多の関係について勉強しておりますが、データの取得が難しくつまづいております。
punchan36

2020/03/20 06:38

この度はご回答有難うございました。自己解決した為、その旨追記しておきました。 「has_many through」を使ったUserとRoomを直接結びつける方法でのビューの書き方は今回は分からず終いでしたが、多対多の考え方などが大変学びになりました!また何かありました際には宜しくお願い致します。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問