メッセージの未読・既読機能が実装出来ない。

Question

前提・実現したいこと

受信したメッセージの横に、未開封であればマークを表示するといった処理をしたいです。
メッセージ機能及び通知機能は既に実装出来ています。

未読・既読機能としては、以下の流れを考えています。

1. 通知テーブルにreadと言うboolean型のカラムを追加する。
2. /rooms/:idにアクセスがあった場合にreadの中身をfalseからtrueに変える処理を施し、未読かどうかを判定する。
3. trueの場合はマークを表示、/rooms/:idにアクセスがあった瞬間にマークを消す。

ただコードの記述の仕方が分からず足踏みしています。
「ログイン中のユーザーが抱えるmessageに関する通知（全件）を取得する」と言った通知機能は記述出来たのですが、「各room内のmessageに関する通知をそれぞれ取得する」記述の仕方が分かりません…。
ご助言を頂けますと有難いです。

schema.rb

  create_table "notifications", force: :cascade do |t|
    t.integer "visitor_id", null: false
    t.integer "visited_id", null: false
    t.integer "post_id"
    t.string "action", default: "", null: false
    t.boolean "checked", default: false, null: false
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
    t.integer "comment_id"
    t.integer "message_id"
    t.integer "room_id"
    t.boolean "read", default: false, null: false #未読・既読判定の為にカラム追加。
    t.index ["comment_id"], name: "index_notifications_on_comment_id"
    t.index ["message_id"], name: "index_notifications_on_message_id"
    t.index ["post_id"], name: "index_notifications_on_post_id"
    t.index ["room_id"], name: "index_notifications_on_room_id"
    t.index ["visited_id"], name: "index_notifications_on_visited_id"
    t.index ["visitor_id"], name: "index_notifications_on_visitor_id"
  end

messages_controller.rb

  def create    
      @room = @message.room
      if @message.save!
          @roommembernotme = Entry.where(room_id: @room.id).where.not(user_id: @current_user.id)
          @theid = @roommembernotme.find_by(room_id: @room.id)
          notification = @current_user.active_notifications.new(
              room_id: @room.id,
              message_id: @message.id,
              visited_id: @theid.user_id,
              visitor_id: @current_user.id,
              action: 'message'
          )
          if notification.visitor_id == notification.visited_id
              notification.checked = true
              notification.read = true # 自分がメッセージを送った場合は既読にする。
          end
          notification.save! 
      end 

rooms_controller.rb

  def show
      # ログイン中のユーザーが抱える「message」に関する相手からの通知を取得。
      @notifications = @current_user.passive_notifications.where(action: 'message').includes([:visitor], [:visited]).order(created_at: :desc).page(params[:page]).per(7)
      # ログイン中のユーザーが抱える相手からの通知のうち、該当のroom_idを取得し、showを開いた時点でreadをtrueにする。
      @current_user.passive_notifications.find_by(params[:id]).update(read: true)
      # 該当のroom_id内にある未読メッセージの数を取得。
      @unread = @notifications.where(room_id: params[:id], read: false).count
   end

   def index
      @rooms = Room.page(params[:page]).per(3)
      @user = @current_user
      @currentEntries = @current_user.entries
      myRoomIds = []
      @currentEntries.each do | entry |
        myRoomIds << entry.room_id
      end
      @anotherEntries = Entry.includes(:user, :room).where(room_id: myRoomIds).where('user_id != ?', @user.id).order(created_at: :desc)

      @notifications = @current_user.passive_notifications.where(action: 'message').includes([:visitor], [:visited]).order(created_at: :desc).page(params[:page]).per(7)
      @notifications.where(checked: false).each do |notification|
        notification.update_attributes(checked: true)
      end
      @unread = @notifications.where(room_id: params[:id], read: false).count
   end

rooms_index.html.erb

    <div class="message-items">
      <% @anotherEntries.each do |e| %>
        <div class="message-item">
          <div class="message-left">
            <%= link_to room_path(e.room.id) do %>
              <img src="<%= "/user_images/#{e.user.image_name}" %>">
            <% end %>
          </div>
          <div class="message-center">
            <div class="message-center-username">
              <%= link_to room_path(e.room.id) do %
                <%= e.user.name %>
                <%= @unread %>                  
              <% end %>
            </div>
            <div class="message-center-message">
              <%= link_to room_path(e.room.id) do %>
                <% dm = Message.find_by(id: e.room.message_ids.last).try(:content) %>
                <%= truncate(dm, length: 9) %>
              <% end %>
            </div>
          </div>
          <div class="message-right">
            <div class="message-date" style="color: #C0C0C0;"><%= e.updated_at.strftime("%Y/%m/%d") %></div>
            <div class="message-time" style="color: #C0C0C0;"><%= e.updated_at.strftime("%H:%M") %></div>
          </div>
        </div>
      <% end %>
    </div>

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

ruby 2.6.4p104
RubyGems 3.0.3
Rails 5.2.3

【追記】ER図

【追記】解決したコード

rooms_controller.rb

  def index
    #以下、メッセージ機能に関する記述。変更なし。
    @rooms = Room.page(params[:page]).per(3)
    @user = @current_user
    @currentEntries = @current_user.entries
    myRoomIds = []
    @currentEntries.each do | entry |
      myRoomIds << entry.room_id
    end
    @anotherEntries = Entry.includes(:user, :room).where(room_id: myRoomIds).where('user_id != ?', @user.id).order(created_at: :desc)

    # 既読機能についてはコントローラでは特に記述せず。ビューにて記述。
  end

   def show
    #以下、メッセージ機能に関する記述。変更なし。
    @rooms = Room.page(params[:page]).per(3)
    @user = @current_user
    @currentEntries = @current_user.entries
    myRoomIds = []
    @currentEntries.each do | entry |
      myRoomIds << entry.room_id
    end
    @anotherEntries = Entry.includes(:user, :room).where(room_id: myRoomIds).where('user_id != ?', @user.id).order(created_at: :desc)

    # ここから既読機能。ログイン中のユーザーが抱える「message」に関する「相手からの通知」のうち、該当のroom_idを取得し、showを開いた時点でreadをtrueにする。
    @current_user.passive_notifications.where(action: 'message', read: false, room_id: params[:id]).each do |notification|
      notification.update_attributes(read: true)
    end
   end

rooms_index.html.erb と rooms_show.html.erb（既読機能に関する内容は同じ）

   <!-- ログイン中のユーザーが保持している各Entryの中で、未読メッセージがある場合countを繰り返し表示。-->
   <% message_count = Notification.where(action: "message", visited_id: @current_user.id, visitor_id: e.user.id, room_id: e.room.id, read: false).count %>
   <% if message_count > 0 %>
     <div class="message-count date-and-count"><%= message_count %></div>
   <% end %> 

Answer 1

この質問は「追記・修正依頼」で解決しました。

他に書くことが思い当たらないため、rooms_controller.rbをリファクタリングしたものを載せておきます。
動かなかったらすみません。

before_action :set_room, only: [:index, :show]
before_action :set_another_entries, only: [:index, :show]

def index
end

def show
  @current_user
    .passive_notifications
    .where(action: 'message', read: false, room_id: params[:id])
    .update_all(read: true)
end

private

def set_room
  @rooms = Room.page(params[:page]).per(3)
end

def set_another_entries
  my_room_ids = @current_user.entries.pluck(:room_id)
  @another_entries = Entry.includes(:user, :room).where(room_id: my_room_ids).where('user_id != ?', @current_user.id).order(created_at: :desc)
end

Comments

[punchan36]

有難うございます！！
こちらの記述でも、room作成・message送受信・通知機能と全て問題なく作動致しました（質問文には記載していなかった既存の変数はいくつか残す必要あり）。

配列のくだりはpluckと言うメソッドでこんなにも簡略化出来るのですね…まさに目から鱗が落ちる思いでした。大変勉強になりました。
親身にご助言も頂きまして有難うございました！

Answer 2

visitor　なのか visited なのかどちらかわかりませんが、それとroom で絞ってください
ああ、visitorの方ですね。そちらは絞れていますから
@notifications.find_by(room_id: params[:id]).update(read: true)
show すべき room が　params[:id] で来ている前提です

Comments

[punchan36]

有難うございます！
ご助言頂いた記述で、下記の2番は恐らく出来るようになったかと思います。

1. 各ルーム内における未読メッセージ（read: false）の数を各ルームにそれぞれ表示する。
2. 相手から届いたmessageに関する通知のうち、該当のroom_idを取得し、showを開いた時点でreadをtrueにする。

1番なのですが、rooms_controller.rb内に以下の記述をしました。
@unread = @notifications.find_by(room_id: params[:id]).where(read: false).count
（該当のroom_id内にある未読メッセージの数を取得。）

しかし以下のエラーが出てしまい前に進めません。
NoMethodError in RoomsController#index
undefined method `where' for nil:NilClass

コントローラの書き方など間違っておりますでしょうか…？
（質問文は今現在のコードに書き直しました。）

[winterboum]

やるなら
@notifications.where(room_id: params[:id], read: false).count,read: false).count
ですが、
@notifications.find_by(room_id: params[:id])
がnilってことは、２番もできていないですね。
RoomsController#index を呼んだ時のパラメータを確認してください。
room.idがどういう形で来ているのか、そもそも来ているのか

[punchan36]

有難うございます。
@unread = @notifications.where(room_id: params[:id], read: false).count
に記述し直してからログを追ってみました。

詳細は追記致しましたが、rooms_index.html.erbは表示されるもののrooms_show.html.erbを開くとエラーが出ます。
このような状態になってからの見極めがいつも苦手（難しい）なのですが、どのように見て行けば良いのでしょうか…。

[winterboum]

@notifications.find_by(room_id: params[:id]).update
これがそういうエラーを出しているとすると、そういうnotificationができていないということです。notificationを作る所を疑ってください。
なお、
エラーメッセージの載せ方がはんば。どの行か、もでてません？

[neko_daisuki]

@current_user.passive_notifications.where(action: 'message').includes([:visitor], [:visited]).order(created_at: :desc).page(params[:page]).per(7)
として絞り込んだものを .find_by(room_id: params[:id]) してるので見つからないんだと思います。

@current_user.passive_notifications の中には params[:id] のがあるはずなので
これを find_by して update ですね。

[winterboum]

です
あ、その idが何のIDかを確認してくださいね

[punchan36]

返答が遅くなり申し訳ありません。
また、ご助言頂き有難うございます。

先ほど質問文の内容を編集してしまったのですが、その後に自分のコードの間違いに気づきました。

romms_controller.rb内の記述を以下の様に変更したりと試行錯誤しております。
@notifications.find_by(room_id: params[:id]).update(read: true)
↓
@current_user.passive_notifications.find_by(params[:id]).update(read: true)

まだ上手く機能はしておりませんが、エラーは出なくなりました。また行き詰りましたらコメント致します。

[punchan36]

何度も申し訳ありません。
先ほど申しましたようにrooms_controller.rb内の記述を変更したところ、rooms_index.html.erb, rooms_show.html.erbのどちらを表示してもエラー文は出なくなりました。

しかしread: falseのメッセージがあるにも関わらず、countが0と表示されます。定義した「@unread」が上手く機能していないようです。
rooms_index.html.erbにあります「@unread」周りの記述も追記致しましたが、中身が上手く渡っていないのでしょうか…。

[neko_daisuki]

find_by の使い方が間違ってます。

#show の @unread はper(7)となってるので最大で7にしかなりません。
(どうして 0 になるのかは分かりません)
#index の @unread は @notifications が見つかりません。

@unread = @notifications.where(room_id: params[:id], read: false) は複数あるはずなのですか？
だとすると、update(read: true) したい notification ってどう決まるのでしょうか。
なんか混乱してきました。

[punchan36]

引き続き有難うございます！

申し訳ありません。
#index 内にも #show 同様 @notifications は記述しておりましたが、質問文に書き忘れていました（追記致しました）。@notificationsの記述があった上で、countが0になりました。

また、そうですね。@notifications の定義には per(7) の記述も入ってましたので、@read の定義は少し長くなりますが以下の様に書き換えようと思います。
@unread = @notifications.where(room_id: params[:id], read: false).count
↓
@unread = @current_user.passive_notifications.where(action: 'message', room_id: params[:id], read: false).includes([:visitor], [:visited]).count

更に@unreadの定義の意味ですが、各room_id内にある未読メッセージの数をそれぞれ取得したいです。ですので、room_idは各ルームにつきもちろん一つですが、未読メッセージの数は2つ以上ある可能性もありますのでwhereを使い記述致しました。

[neko_daisuki]

#index
@unread = @notifications.where(room_id: params[:id], read: false).count
としてますが、#index には params[:id] がないのでは。

#show
未読メッセージの数が2つ以上ある可能性があるのなら、
２つあったとして、どちらを既読にするのでしょうか。

どちらも既読にするのなら @unread は 0 になりますよね。

[punchan36]

有難うございます。
なるほどです…。言っていただいて初めて気が付きました。

#index には params[:d] が存在しない為、各ルームの未読メッセージの count も 0 になる。

#show では、rooms/id を開いた時点で params[:d] 内にある全ての未読メッセージを read:trueにしたいです。
しかし rooms_show.html.erb も rooms_index.html.erb と同じように記述してしまっているため、各ルームの部分に「今開いている rooms/id で受け取った未読メッセージの count」が表示されてしまっています（今開いている rooms/id の count が 0 になる → 他のルームの count 表示 も 0 になる）。

私がこれまで実装した物の中では一番複雑かもしれず、これ以上どうすれば良いのか見当がつかないのですが、根本的にコードを見直さないと実装は難しそうでしょうか…。

[neko_daisuki]

#index と #show でどの通知を表示してどんな時に何を既読にするのか仕様がよく分かりません。
仕様がはっきりすればそんなに難しいコードは必要ないと思いますよ。

[punchan36]

その辺りを上手く説明出来ておらず、申し訳ありませんでした。
仕様と致しましてはLINEやFaceBookのMessengerと同様です。

#index: ログイン中のユーザーが抱えている各 room を画面左側に一覧で表示する（ユーザー名、ユーザーのプロフィール画像、最新メッセージの冒頭数文字、最新メッセージの送信時間など）。
ここまでは実装出来ており、ここから更に、それぞれの room に未読メッセージが何件あるか、各ユーザー名の横にその数字を表示したい（Aさん=2, Bさん=3, Cさん=0など）。

#show: indexで表示していた画面左側部分（room一覧）は引き続き表示したままにし、クリックされたユーザーとのメッセージのやり取りの詳細を画面右側に表示する。
ここまでは実装出来ており、ここから更に、rooms/id がクリックされた瞬間に、 該当の rooms/id にあった未読メッセージを「全て」 read:true にしたい。
またその処理により、画面左側に表示していた、該当 room における未読メッセージの数も0(表示なし)にしたい。
他の room に未読メッセージがある場合は、該当 rooms/id をクリックしない限り、もちろんその count が表示され続けるようにしたい（#index の説明の最後に書いた例で、仮に今回Aさんとの room をクリックした場合、表示される count は「Aさん=0, Bさん=3, Cさん=0」とAさんの count のみ0に変更したい）。

このような仕様となっております。
中々解決にならず申し訳ありませんが、出来ましたら引き続きご助言頂けますと有難いです。

[neko_daisuki]

index.html.erb の #show へのリンクを
<%= link_to room_path(e.room.id, visitor_id: e.user.id) do%>
として、#show で params[:visitor_id] を受け取れば良いのではないでしょうか。

#show では current_user と params[:id](roomのid) と params[:visitor_id] で絞り込んだ
notification を read: true にする。

#index はモデルの関連付けがどうなっているのか分からないので
なんとも言えませんが、できないことはなさそうです。

[punchan36]

有難うございます。
まだ上手く機能しませんが、根本的に色々と見落としている気がしてきました。

#show
@unread = @notifications.where(room_id: params[:id], read: false).count

こちらの定義も、今表示されている params[:id] の情報を取得している訳ですので、そこでの未読メッセージの count が仮に3であれば、他のルームの未読メッセージ数も全て3と表示されてしまいます。

ですので、rooms_show を表示していなくても、各ルームの未読メッセージの数を取得+表示出来るような変数を定義出来れば良いのかなと思いました。
つまりは、#index で count を正しく表示出来れば、#show でも表示出来るのかなと思います。

ただ #index を表示している段階では room_id: params[:id] を取ってくる事も出来ません。各ルームの情報は以下の変数を繰り返し処理して表示させている訳ですが、ここに他テーブルの情報である Notification の count の情報を組み込む方法も複雑でよく分かりません…。

#index
@anotherEntries = Entry.includes(:user, :room).where(room_id: myRoomIds).where('user_id != ?', @user.id).order(created_at: :desc)

最初に質問させて頂いた時から内容が少し変わってきておりますが、ER図等も含め、別で質問を設けた方が望ましいでしょうか…。
貴重なお時間を割かせてしまい申し訳ありません。

[neko_daisuki]

僕は構いませんが、winterboumさんに通知でご迷惑をおかけしていないかは気になります。

Entryというのは user と room の中間テーブルでしょうか？
@anotherEntries は自分が参加してるroomで、
かつ自分以外が参加しているroomを取得しているんですよね。

N+1 とかは無視して取り合えずこんな感じでダメなんでしょうか
@anotherEntries.each do |entry|
entry.user.name
# 未読のカウント
Notification.where(actoin: "message", visited: current_user, visitor: entry.user, room: entry.room, read: false).count
end

または、rails c で実行してみて欲しいのですが、
Notification.where(visited_id: 1, read: false).group(:visitor_id).count
とすると、{ (visitor_id => (count), (visitor_id => (count), ...} というのが得られます。
これを @unread として、ビューで @unread[e.user.id] としても未読を表示できるんじゃないかと思います。

[punchan36]

有難うございます！無事理想通り動くようになりました。
最終的に動くようになったコードは追記致しました。

ただ一点だけ理解に至らない点がありまして、
rooms/controller.rbの#showにあります以下の行ですが、room_id: params[:id]は1つのみを取得したいのに、where（複数を取得するメソッド）できちんと動く事が少し不思議でした。

@current_user.passive_notifications.where(action: 'message', read: false, room_id:params[:id]).each do |notification|
notification.update_attributes(read: true)
end

また「Entry」や「@anotherEntries」の定義の意味は仰る通りでございます。全体のER図も載せておきました。

解決致しましたので質問を閉じようと思うのですが、こちらのご回答をそのままベストアンサーにしても宜しいのでしょうか…？

[neko_daisuki]

おめでとうございます。
ループの中でユーザーの数だけSQLが実行されてしまう点はご留意ください。

条件に当てはまるレコードが本当にひとつなら下のでも良いです。
@current_user.passive_notifications.find_by(action: 'message', read: false, room_id: params[:id]).update_attributes(read: true)

ベストアンサーはお任せします。

[winterboum]

nekoさん回答まとめてくださいよ。そちらにベストを

[punchan36]

なるほどです。お二方とも有難うございました！
winterboumさま、親身にご助言頂いたにも関わらずなかなか解決まで辿り着けず申し訳ありませんでした。
neko_daisukiさまからご回答があるようであれば、そちらをベストアンサーに選ばさせて頂ければと思います。

[neko_daisuki]

了解しました。
お二方、お気遣いいただきありがとうございます。