グループ参加人数０のグループも一覧に表示する

Question

ご回答いただけるとありがたいです。よろしくお願いします！

実現したいこと

・グループの参加人数が多い順にグループ一覧ページに表示されるようにしたい（実現済）
・グループの参加人数が０のグループもグループ一覧ページにに表示されるようにしたい

発生している問題

グループ一覧ページに、グループの参加人数が多い順に上から表示するように以下のようにコードを書きました。
※参加人数が同じ場合は、作成した順。

#groups
def list
    @groups = Group.joins(:group_users).group(:group_id).order('count(group_id) DESC').order('created_at ASC')
end

参加人数0のグループのidは中間テーブルのgroup_usersテーブルのgroup_idカラムにはありません（当たり前ですが）。
上記のように書いた結果、参加人数が０のグループのgroup_idが取得できず、表示されなくなります。
※ちなみに

.joins(:group_users).group(:group_id).order('count(group_id) DESC')

を書かなければ、グループ一覧に参加人数０のグループも表示されます。

該当のソースコード

#groups
def list
    @groups = Group.joins(:group_users).group(:group_id).order('count(group_id) DESC').order('created_at ASC')
end

#list.html.erb
<div class="group_lists">
  <h1>グループ一覧</h1>
  <div class="group_list">
    <% @groups.each do |group| %>
      <div class="group-container">
        <div class="upper-group-container">
          <div class="group-content-left">
            <div class="participation-button">
              <% if user_signed_in? %>
                <% unless GroupUser.exists?(group_id: group.id, user_id: current_user.id) %>
                  <div class="no_participation">
                    <%= link_to '参　加', join_group_path(group) %>
                  </div>
                <% else %>
                  <div class="participation">
                    <%= link_to '参加中', "#" %>
                  </div>
                <% end %>
              <% end %>
            </div>
            <div class="number_of_user">
              参加人数: <%= group.users.length %> #この行で参加人数を表示
            </div>
          </div>
(省略)

以上になります。ご助言いただけると幸いです。

##追加（マイグレーションとモデル）

#groups
class CreateGroups < ActiveRecord::Migration[6.0]
  def change
    create_table :groups do |t|
      t.string :name, null: false, unique: true
      t.string :content
      t.references :user, foreign_key: true
      t.timestamps
    end
  end
end

#group_users
class CreateGroupUsers < ActiveRecord::Migration[6.0]
  def change
    create_table :group_users do |t|
      t.references :group, foreign_key: true
      t.references :user, foreign_key: true
      t.timestamps
    end
  end
end

#Group
class Group < ApplicationRecord
  has_many :group_users
  has_many :users, through: :group_users, dependent: :destroy
  has_many :tweets, dependent: :destroy
  belongs_to :user

  validates :name, presence: true, uniqueness: true
end

#GroupUser
class GroupUser < ApplicationRecord
  belongs_to :group
  belongs_to :user
end

Answer 1

Group.select("groups.*, COUNT(group_users.id) users_count").left_joins(:group_users).group("groups.id").order("users_count desc")

これで期待通りに動くと思いますが、どうでしょうか。
select に COUNT(group_users.id) users_count を含め、
users_countでソートしています。

カウンターキャッシュ

上記のクエリは件数が増えると遅いです。
また、.count とするとエラーになります。そのためkaminariなどと使えない(未確認)。

railsにはカウンターキャッシュという機能があります。
これはgroup.users.countの結果をカラムに保存しておく機能です。

値の管理は勝手にやってくれます。

group.users_count # => 0
group.users << user
group.users_count # => 1

カウンターキャッシュを用いると、「０を含むusers.count順」のクエリは以下のように書けます。
.countできない問題もありません。

Group.order("users_count desc")

:counter_cache Railsガイド

Comments

[shawn_709]

いつもいつも本当にありがとうございます！！
期待していた形になりました！感動です！
カウンターキャッシュが何か全く理解できていないですが、しっかり勉強して理解できるようになりたいと思います！参考文献まで付けて頂き、ありがとうございます。

Answer 2

コンソールに出力されたSQLを確認されたらわかるかと思いますが、
joinsだと確かinner joinが実行されたはず。
inner joinだと合致しないデータは取得されません。
left_joinsだとleft joinなので合致しないデータがあってもnullで取得されます。

Comments

[shawn_709]

ご丁寧にありがとうございます！
@groups = Group.left_joins(:group_users)~~~~としたら、参加人数０のグループも一覧に表示することができました。
感動しました！ありがとうございます！！

[m.ts10806]

フレームワークからデータを操作するだけでなく、SQL自身も勉強するとより効果的に使えるようになると思います。私自身Railsはまともにやって２，３ヶ月ですが、他の言語やSQL自体の経験はそれなりにあるので割と「どうすれば取得したいデータが取得できるか」が分かるので調べもつきやすいです。

[shawn_709]

ありがとうございます！すみません。一つ問題が起きました。重ね重ねすみません。
left_joinsで参加人数０のグループも一覧に表示することができたんですが、
１グループ限定の表示になってしまっています。
参加人数０のグループ全てを取得するためにはどうすれば良いでしょうか？
申し訳ないです。。。

[m.ts10806]

その「１グループ限定の表示」が良く分かりません。

[shawn_709]

すみません。
参加人数が０のグループが複数あるのですが、
その内の１つのグループしかグループ一覧に表示されない
ということです。

[m.ts10806]

ん－
テーブル定義とリレーション次第かなぁと。
マイグレーションファイルとモデルを提示してください。

[m.ts10806]

ここが関係してたりしませんか？
<% unless GroupUser.exists?(group_id: group.id, user_id: current_user.id) %>

[shawn_709]

マイグレーションとモデルはここでは、マークダウンが使えないので、上を編集して書き足しました。

<% unless GroupUser.exists?(group_id: group.id, user_id: current_user.id) %>
に関しては、参加しているか否かを判断するために記述しています。

[m.ts10806]

たぶんこんな感じのSQLになっていたらいけそうには思います。
select
groups.id
groups.group_name
from
groups
left join
users on
users.id = groups.user_id
group by groups.group_id

あとそれなりに意図が分からないのが
count(group_id)　のこと。
「ユーザー数が多いグループの順」だったらもうちょっと複雑になりそうな気がします。

[shawn_709]

SQLを全然理解していないので、まずそこからやります！
すみません。手間を取らせてしまって。
count(group_id)のところは、はじめuser_idでやっていたのですが、エラーが出るので勘で色々変えてたら、group_idでエラーが出なくなり、参加人数の多い順に並んだのでそうしていました。

[m.ts10806]

現状でコンソールに出てきているSQLを質問本文に追記してもらって良いですか？
アソシエーションの結果にもなると思うので。
SQLは一番内側から実行されるのでorder byのところでcount()しても想定通りには動かなそうに思います。

[shawn_709]

本当に初心者すぎて理解ができていないところが多いです。
「現状でコンソールに出てきているSQL」というものがどれに該当しているのかがわかっていません。すみません。