Ruby on Rails 会話一覧で最新のメッセージを一件だけ表示させたい

Question

###前提・実現したいこと
rails4 ローカル環境

現在メッセージ機能を実装してラインのようなメッセージのやりとりができるようにしたいと思っています。

機能自体は実装できたのですが、会話一覧ページ（誰とメッセージのやり取りをしているか表示するラインのトーク一覧のようなページ）で最新のメッセージを一件取得する方法が分かりません。

conversations : 会話情報を格納するテーブル
messages : 会話の個々のメッセージを格納するテーブル
として実装しました。

###該当のソースコード
conversations/index.html.erbここで最新のメッセージを一件だけ取得したい

<div class="container message">
  <div class="wrapper col-md-8 col-md-offset-2 col-sm-10 message-index">
      <h2>メッセージ一覧</h2>

      <% @conversations.each do |conversation| %>
      <div class="row conversation-list">
        <% if conversation.target_user(current_user).present? %>
          <%= link_to conversation.target_user(current_user).name, conversation_messages_path(conversation)%>
        <% end %>
      </div>
      <% end %>
  </div>
</div>

messages_controller.rb

class MessagesController < ApplicationController
  before_action :authenticate_user!
  
  before_action do
    @conversation = Conversation.find(params[:conversation_id])
  end

  def index
    @messages = @conversation.messages
    if @messages.length > 10
      @over_ten = true
      @messages = @messages[-10..-1]
    end

    if params[:m]
      @over_ten = false
      @messages = @conversation.messages
    end

    if @messages.last
      if @messages.last.user_id != current_user.id
       @messages.last.read = true
      end
    end

    @message = @conversation.messages.build
  end

  def create
    @message = @conversation.messages.build(message_params)
    if @message.save
      redirect_to conversation_messages_path(@conversation)
    end
  end

  private
    def message_params
      params.require(:message).permit(:body, :user_id)
    end

messageモデル

class Message < ActiveRecord::Base
  belongs_to :conversation
  belongs_to :user

  validates_presence_of :body, :conversation_id, :user_id
  def message_time
    created_at.strftime("%m/%d/%y at %l:%M %p")
  end
end

conversations_controller.rb

class ConversationsController < ApplicationController

  before_action :authenticate_user!

  def index
    @users = User.all
    @conversations = Conversation.all
  end

  def create
    if Conversation.between(params[:sender_id], params[:recipient_id]).present?
      @conversation = Conversation.between(params[:sender_id], params[:recipient_id]).first
    else
      @conversation = Conversation.create!(conversation_params)
    end

    redirect_to conversation_messages_path(@conversation)
  end

  private
    def conversation_params
      params.permit(:sender_id, :recipient_id)
    end
end

Conversationモデル

class Conversation < ActiveRecord::Base
  belongs_to :sender, foreign_key: :sender_id, class_name: 'User'
  belongs_to :recipient, foreign_key: :recipient_id, class_name: 'User'
  has_many :messages, dependent: :destroy
  validates_uniqueness_of :sender_id, scope: :recipient_id
  scope :between, -> (sender_id,recipient_id) do
    where("(conversations.sender_id = ? AND conversations.recipient_id =?) OR (conversations.sender_id = ? AND  conversations.recipient_id =?)", sender_id,recipient_id, recipient_id, sender_id)
  end

  def target_user(current_user)
    if sender_id == current_user.id
      User.find(recipient_id)
    elsif recipient_id == current_user.id
      User.find(sender_id)
    end
  end
end

データベーステーブル

  create_table "messages", force: :cascade do |t|
    t.text     "body"
    t.integer  "conversation_id"
    t.integer  "user_id"
    t.boolean  "read",            default: false
    t.datetime "created_at",                      null: false
    t.datetime "updated_at",                      null: false
  end
  add_index "messages", ["conversation_id"], name: "index_messages_on_conversation_id", using: :btree
  add_index "messages", ["user_id"], name: "index_messages_on_user_id", using: :btree

  create_table "conversations", force: :cascade do |t|
    t.integer  "sender_id"
    t.integer  "recipient_id"
    t.datetime "created_at",   null: false
    t.datetime "updated_at",   null: false
  end

どのように最新のメッセージを一件取得するばいいかアドバイス頂けると嬉しいです。

Answer 1

もうベストアンサーついてますが、こんなやり方もあるよという意味で回答します。

@messages.order(created_at: :desc).take

また、find_byとorderを組み合わせて使用することで、1件のみ取得することも可能です。

Comments

[riamk]

ご回答ありがとうございます。
サブクエリを色々調べていたのですが、うまく記述できず困っていたので、KaiShoyaさんの回答を参考にこちらも試してみます！

[退会済みユーザー]

横からすみません。
@mesagesに何が入っているのかわかりませんが、この方法だとすべてのmessageのうちの最新のものしか取得できないように見えます。
質問者さんはLINEのトーク一覧のようなものを作りたいそうなので、各conversationで最新のmessageを取得する必要がありそうに思いました。(間違ってたらすみません...)

[KaiShoya]

あー、確かにそうですね。
メッセージ機能は実装済みとのことなので、トーク別でのメッセージ取得はできている前提で話してました。
conversations/index.html.erbを見直しましたが、こんな感じでしょうか。
```
<% @conversations.each do |conversation| %>
<%= conversation.messages.order(created_at: :desc).take %>
<% end %>
```

[riamk]

実現したいのはchromeさんが仰るように「各conversationで最新のmessageを取得する」です。
私の説明が伝わりづらくてすみません。

教えて頂いた
<%= conversation.messages.order(created_at: :desc).take %>
で試してみたところ、
#<Message:0x007fdab6718200>
と表示されてしまい、メッセージの内容（body）を取得できません。
原因はどこにあるんでしょうか？

Answer 2

サブクエリを使えばいけると思います。

conversations_controller.rbのindex内で

@messages = Message.where(%"
        messages.created_at = (
        SELECT MAX(m_sub.created_at)
        FROM messages m_sub
        GROUP BY m_sub.conversation_id
        )")

適当に書いたコードなのでこれそのままは使えないと思いますが、たぶんこんな感じでサブクエリを使えば各conversationでcreated_atが最新のmessage一覧が取得できると思います。

あとはconversations/index.html.erbで

<% @conversations.each do |conversation| %>
  <%= @message.find{|m| m.conversation_id == conversation.id}.body %>      
<% end %>

みたいにすればいけそうな気がするのですがどうでしょうか。

Comments

[riamk]

ご回答ありがとうございます。
サブクエリはまだ使用したことがないので、色々と調べながら試してみます！
ありがとうございました。

[riamk]

ご回答ありがとうございます。
サブクエリは使用したことがなかったので色々と調べながら試してみます。
ありがとうございました！