Rails 中間テーブルを経由してのカラムの値の取得

###前提・実現したいこと
railsでユーザーがツイッターのようにタイムラインに投稿できるものを作っています。さらに、ユーザーがグループを作り、そのグループページでメンバーの投稿を表示したいです。
アソシエーションは以下のように組んでいます。
Groups
has_many :user_groups
has_many :users, through: :user_groups
Users
has_many :reviews
has_many :user_groups
has_many :groups, through: :user_groups
UserGroups
belongs_to :user
belongs_to :group
Reviews（投稿）
belongs_to :user
UsersとGroupsが多対多の関係になっており、中間テーブル（UserGroups）でつないでいます。

ユーザーが所属しているグループ一覧をこのように表示させています。
例）
・太郎グループ
・花子グループ
そしてグループ名をクリックするとグループページに飛ぶようになっています。

UserGroupsテーブルのカラムは
id ｜ user_id ｜ group_id
の３つです。

####ここに質問したいことを詳細に書いてください
@group = Group.find(params[:id]).id
ここでクリックしたグループのidを取得できていると思います。
仮に1とします。
@users = UserGroup.where(group_id: @group)
そしてここでgroup_idカラムにクリックしたグループのid（ここでは１）を持つレコードを取得
@reviews = @users.user_id.reviews
group_idカラムに１をもったすべてのuser_idを取得し、そのuserがもっているreview（投稿）をとる

こういうつもりでコードを書いているのですがうまくいきません。
つたない説明ですが、回答していただけると幸いです。

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

undefined method `user_id'というエラーメッセージが出ます。

###該当のソースコード

@group = Group.find(params[:id]).id #クリックしたグループのid取得
@users = UserGroup.where(group_id: @group)#クリックしたグループのidを持つレコードを取得
@reviews = @users.user_id.reviews

###試したこと
課題に対してアプローチしたことを記載してください

###補足情報(言語/FW/ツール等のバージョンなど)
ruby
rails 4.2.6

行動規範の内容に同意します

回答1件

ベストアンサー

ruby
1@group = Group.find(params[:id]).id 
2# ここでクリックしたグループのidを取得できていると思います。

ええ。たしかにそうなんですが、おかしな事をやっていなければ Group.find(params[:id]).id == params[:id] なのであんまり意味がないですね。

という事で全面的に書き直した方がいいです。

Group#id (params[:id]) から、その Group に所属する User に関連する Review を取得したいのであれば

ruby
1reviews = Group.find(params[:id]).users.map(&:reviews)

です。

undefined method `user_id'というエラーメッセージが出ます。

これは恐らく

ruby
1@reviews = @users.user_id.reviews

の部分で出ているはずです。

この前の行で

ruby
1@users = UserGroup.where(group_id: @group)#クリックしたグループのidを持つレコードを取得

としているので @users は UserGroup::ActiveRecord_Associations_CollectionProxy という配列のようなもののインスタンスです。

そのインスタンスに対して #user_id というインスタンスメソッドを呼ぼうとしているので、そんなメソッドはないよ。という例外が発生しています。

ruby
1<% @reviews.each do |review| %>
2<%= review.title %>
3<%= review.text%>
4<% end %>

viewでこういう使い方をしたいのであればcontroller側では

ruby
1@reviews = Group.find(params[:id]).users.map(&:reviews).flatten

または

ruby
1@reviews = Group.find(params[:id]).users.flat_map(&:reviews)

あたりが例です。

投稿2016/11/07 09:43

編集2016/11/07 22:15

hana-da

総合スコア1728

tsubasa-i

2016/11/07 13:35 編集

回答ありがとうございます！回答にある reviews = Group.find(params[:id]).users.map(&:reviews) というように書き直して実行してみたところ undefined local variable or method `reviews'というエラーが出てしまいました。そこで、@reviewsにビューファイル共にと書き直して実行したところ、今度はreviewsテーブルの中のカラムに対してno method errorが出てしまいます。なぜでしょうか？

tsubasa-i

2016/11/07 13:40

<% @reviews.each do |review| %> <%= review.title %> <%= review.text%> <% end %> ビューファイルではこのようにeachで投稿の内容をとってきて表示しようとしています。