Ruby on Railsで自分の所属する全てのグループの投稿を全て表示させたい

Question

やりたいこと

自分が所属するグループ（ここではUnion)の全ての投稿をpost index で表示したいのですがどう考えてもできません。

自分が所属するUnionは2つあります。

１、自分が作成したUnion。　これはUnionのuser_id　に　@current_userのidが保存されています。
２、他人が作成したUnionに参加している場合。　この場合は中間テーブルであるUnion_user の　user_id　に　@current_userの　id が保存されています。

自分が所属するある一つのUnionに投稿されているPostを表示するのは以下で可能かと思います。

    def posts_index
      @union = Union.find_by(id: params[:id])
      @user = @union.user
      @posts = Post.where(union_id: params[:id]).order(created_at: :desc)
    end
　　
　　<% @posts.each do |post| %>
      <div class="posts-index-item">
　　.
　　.
    <% end %>

しかし、自分が所属しているUnionが複数ある場合に全てのpostを表示させることができません。

やったこと

def index
    @user = User.find_by(id: @current_user.id)
    @unions = Union.where(user_id: @user.id).order(created_at: :desc)
  end

<% @unions.each do |union| %>         ←　自分が作成したUnion
   <div class="unions-index-item">
   .                              
   .
<% end %>

<% @user.unions.each do |union| %>     ←　他人が作成したUnion
   <div class="users-index-item">
   .
   .
<% end %>

以上のコードのようにして、自分が作成したUnion, 他人が作成したけど自分が所属するUnion index は表示できます。

そしてそのUnionsのポストを以下のように取り出そうとしてもうまくできません。

  def user_posts_index
    @user = User.find_by(id: @current_user.id)
    @unions = Union.where(user_id: @user.id).order(created_at: :desc)
  end

<% @unions.each do |union| %>
      <% post = Post.find_by(union_id: union.id)%>
      <div class="posts-index-item">
    .
    .
<% end %>

どうしたらできるでしょうか？

Comments

[winterboum]

UnionとUserの関連定義を載せてください。

あと、予感として
Unionの作成者はUnion_userには登録されていないのですね？
Unionのownerとメンバーの管理は分けておいたほうが（作成者もUnionUserに登録する）この処理は楽になるなぁ、、と

[atage517]

class Union < ApplicationRecord
has_many :union_users
has_many :users, through: :union_users
accepts_nested_attributes_for :union_users
end

class User < ApplicationRecord
has_many :union_users
has_many :unions, through: :union_users
end

class UnionUser < ApplicationRecord
belongs_to :user
belongs_to :union
end

Unionの作成者はUnionのuser__idに登録していますね。

そして他人の作ったUnionに参加した時はUnionUser テーブルを中間テーブルとしてuser_idに登録しています。

@posts = Post.where(union_id: @user.unions.ids)
@posts = Post.where(union_id: @unions.ids)
で自分が作ったUnionのpost index
参加しているUnionのpost index を表示することができたのですが二つとも表示させることができないので今方法を探しています

Answer 1

このケースは、Unionを作った本人も UserUnionに登録するのが一番簡単ですし、いろいろメリットもあります。
もしそうしてあれば
atageさんの
posts = Post.where(union_id: @user.unions.ids)（DBアクセス２回)
でも
posts = Post.joins(union: :users).where("users.id =?",@user.id)（DBアクセス１回)
でも取れます。

そうでない現状では
@posts = Post.where(union_id: @user.unions.ids) + Post.where(union_id: @unions.ids)
とか
@posts = Post.where(union_id: (@user.unions.ids + @unions.ids))
とかせざるを得ません。
現状では ４or３回のDBアクセスになり、そこもなんか気持ち悪いです

「union.user は『Unionを作った人』の意味だけ」にするのではだめですか？

Comments

[atage517]

すみません週末は忙しくて返信が遅れました。
いまwinterboumさんが書いたように　@posts = Post.where(union_id: (@user.unions.ids + @unions.ids)) でやると自分が作成したUnionと自分が所属しているUnionの投稿全てを取り出すことができました。
＋　を使って複数の条件を組み合わせることがでいるとは知りませんでした！ありがとうございます。

Unionの作成者もUnionUserに登録するとはどのようにすれば良いのでしょうか？
winterboumさんが言ったようにDBに何回もアクセスしていることはわかりました。確かにDBへのアクセスが少ない方がいいと思います。

自分がUnionを作成した際に@current_user.idがUnion のuser_idに保存されます。

中間テーブルであるUnionUserでは「Unionに参加する」ボタンをクリックするとクリックした時の@current_user.idがUnionUserのuser_idに保存されるようになっています。

作成者もUnionUserに登録するというメリットはわかったのですが作成した時にUnionUserにowner_id?のような者を作るやり方がわからないです

[winterboum]

@current_user.idをUnion のuser_idにしまった時に、「「Unionに参加する」ボタンをクリック」した時に行う処理をしておけばよいです。

[atage517]

ありがとうございました！