Ruby on Rails メッセージ機能でのメッセージの取得について

Question

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

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

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

メッセージ機能は無事に実装できて、メッセージのやり取りをしているページで『相手のメッセージは左側』『自分のメッセージは右側』という表示の仕方にしようと思ったのですが、相手のメッセージがCouldn't find User with 'id'=とエラーになり表示できません。

最初に試してみたときは正常に表示できたのですが、機能テストのためユーザーを増やして試してみたところ、先ほどのCouldn't find User with 'id'=というエラーになってしまいました。

あと、

<% if @over_ten %>
   <%= link_to '以前のメッセージをすべて表示', '?m=all' %>
<% end %>

とこのように10件以上の表示はリンクをクリックして表示させているのですが、正常に表示さえているユーザーとのメッセージもこのリンクをクリックするとCouldn't find User with 'id'=と同じようにエラーになってしまいます。

###該当のソースコード
問題があると思われるviewのコード。
また、今回メッセージ機能は会話「conversations」、メッセージ「messages」として関連づけて作成したので、そちらのコントローラーとモデルなども記載します。

views/messages/index.html.erb

    <% if @over_ten %>
      <%= link_to '以前のメッセージをすべて表示', '?m=all' %>
    <% end %>

    <div class="ui segment">
      <% @messages.each do |message| %>
        <% if message.user_id == current_user.id %>
          <!-- 自分のメッセージ -->
          <div class="item-right">
            <% user = User.find(message.user_id) %>
            <strong><%= user.name %></strong>
            <%= message.message_time %>
            <%= message.body %>
          </div>
        <% else %>
          <!-- 相手のメッセージ -->
          <div class="item-left">
            <% user = User.find(message.user_id) %>
            <strong><%= user.name %></strong>
            <%= message.message_time %>
            <i class="right triangle icon"></i>
            <%= message.body %>
          </div>
        <% end %>
      <% end %>
    </div>

    <!-- メッセージの送信 -->
    <%= form_for [@conversation, @message], html: {class: "ui reply form"} do |f| %>
      <div class="field">
        <%= f.text_area :body, class: "form-control" %>
      </div>
      <%= f.text_field :user_id, value: current_user.id, type: "hidden"  %>
      <div>
        <%= f.submit "メッセージを送る" %>
      </div>
    <% end %>

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

errorは

<% else %>
          <!-- 相手のメッセージ -->
          <div class="item-left">
            <% user = User.find(message.user_id) %> 
            <strong><%= user.name %></strong>

<% user = User.find(message.user_id) %>
ここで発生しているようです。
おそらくmessage.user_idが正しく入っていないmassageがあるということでしょう。
つまり名無しによって投稿されたメッセージがあるということです。

<% else %>
          <!-- 相手のメッセージ -->
          <div class="item-left">
            <% user = User.find_by(id: message.user_id) %> 
            <strong><%= user.try(:name) %></strong>

とすれば、とりあえず、エラーメッセージは出なくなると思いますが。
誰の発言かわからないメッセージを許容するということで
ソフトウェア的にはバグだということに注意してください

もし、名無しメッセージが増えるようでしたら、messages_controller.rbとviews/_form.html.erb views/new.html.erbも晒してください。

Comments

[riamk]

ご回答ありがとうございます。
さっそく教えて頂いたコードのように試してみましたが、
undefined method `strftime' for nil:NilClass
とエラーが出てしまいました。
なぜ、strftimeがエラーになってしまうのか分からず苦戦しております。

コントローラーやモデル、テーブルなど、現在のコードを追記しましたので、再度アドバイス頂けると嬉しいです。

[moke]

massage.created_at
も登録されていないようですね
これも
def message_time
created_at.try(:strftime,"%m/%d/%y at %l:%M %p")
end
とすればとりあえずエラー回避は可能です。
これ以上解説が必要でしたら
さらに
massages_controllers.rbの
def message_params
と
app/views/massages/_form.html.erb app/views/massages/new.html.erb
も見せてください

[riamk]

ありがとうございます！
とりあえずは無事に表示できました。

massages_controllers.rbのdef message_paramsは追記しました。

app/views/massages/_form.html.erb
app/views/massages/new.html.erb
この2点に関しては現時点では作成していません。

現状は正常に表示できていますが、何か修正した方がいいポイントなどご指摘があれば教えて頂けると嬉しいです。

[moke]

Massage controller のindex内の
@message = @conversation.messages.build
を
message = @conversation.messages.build(user: current_user)にしてみてはいかがでしょう？

[riamk]

ご指摘ありがとうございます。
message = @conversation.messages.build(user: current_user)
にしてみたところ、
First argument in form cannot contain nil or be empty
とエラーになったので
@message = @conversation.messages.build(user: current_user)
としました。

表示は正常にできていますが問題はありますでしょうか？

[moke]

@が抜けてましたねそれで大丈夫です。それでうまくいくと思います。

[riamk]

はい、現状はこれでうまくいっています。
ありがとうございました。

私はまだ初心者で全然分からないことばかりですが、機会があればまたよろしくお願いします。