ROR：.eachとmultipltテーブルから選択し、私は3つのテーブル

Question

を持っています：ユーザー、イベント、およびuser_eventsを、私は、コントローラで次の操作を行います。

@temp = UserEvent.where(['user_id = ?', session[:uid]]).all 
    @temp.each do |t| 
     @event_names = Event.where(['id = ?', t.event_id]).all 
    end 

をそれだけで最後にマッチしたイベントを保存して終了。どこが間違っていたのですか？

Answer 1

まず、短期的な解決策：今すぐ

@event_names = UserEvent.where(['user_id = ?', session[:uid]]).all.map do |t| 
    Event.where(['id = ?', t.event_id]).all 
end.flatten 

、長期的な解決策：

@user = User.find(params[:id]) 
@user.events 

が動作するように上記を取得するには、2つのことをしなければならない。

1. あなたの関連付けを正しく設定してください。それはあなたが必要とするようになりますhas_many :through association：nested resource routes設定

   class User 
       has_many :user_events 
       has_many :events, :through => :user_events 
   end 
   
   
   class UserEvent 
       belongs_to :user 
       belongs_to :event 
   end 
   
   
   class Event 
       has_many :user_events 
       has_many :users, :through => :user_events 
   end 
   

2. config/routes.rb中：

   resources :users do 
       resources :events 
   end 
   

私は強くあなたが長期的な解決策を見て促します。短期的な解決策を採用すると、レールが提供しなければならない多くのメリットが失われてしまいます。

Answer 2

ではなく、それぞれのマップ、実行します。

@event_names = @temp.map do |t| 
    Event.where(:id => t.event_id).all 
end 

をしかし、右@bowserseniorです：レールが正しいことをやらせるために適切な関係を使用します。 N + 1ではなく2つのクエリしか実行されないため、効率的です。

class UserEvent < ActiveRecord::Base 
    belongs_to :user 
    belongs_to :event 
end 

class User < ActiveRecord::Base 
    has_many :user_events 
    has_many :events, :through => :user_events 
end 

@events = User.find(session[:uid]).events 
@event_names = @events.map(&:name) # for example 

Answer 3

何が問題になりましたか？

@temp.each do |t| 
    @event_names = Event.where(['id = ?', t.event_id]).all # will reassign @event_name each time 
end 

あなたは、変数@event_namesあなたがループを通って行くので、@event_namesが保持する値のみが最終パスからそのループをされるたびに再割り当てされています。

は、あなたがやるべきこと：

@temp.each do |t| 
    @event_names.push(Event.where(['id = ?', t.event_id]).all) or 
end  

また、他の人が述べたように、これはこれを行うための最良/良い方法は実際にはありません。