Мне нужно показать рейтинг песни текущего пользователя в index.

У меня есть связанные модели:

class Song < ActiveRecord::Base
  has_many :votes, dependent: :destroy

  def user_rate(user)
    votes.find_by(user_id: user).try(:rate)
  end
end

class Vote < ActiveRecord::Base
  belongs_to :song, counter_cache: true
  belongs_to :user
end

Таблица голосов:

  create_table "votes", force: :cascade do |t|
    t.integer  "rate"
    t.integer  "user_id"
    t.integer  "song_id
  end

И действие index в SongsController:

def index
  respond_with @songs = Song.includes(:votes).all
end

В песнях / index.json.rabl я ищу голосование текущего пользователя (каждый пользователь может проголосовать только один раз)

collection @songs
extends "songs/base"
attributes :thumb_cover
node(:current_user_rate){ |song| song.user_rate(current_user) }

Но song.user_rate(current_user) генерирует n + 1 запросов :(

  User Load (0.9ms)  SELECT  "users".* FROM "users"  ORDER BY "users"."id" ASC LIMIT 1
  Song Load (0.6ms)  SELECT "songs".* FROM "songs"
  Vote Load (0.8ms)  SELECT  "votes".* FROM "votes" WHERE "votes"."song_id" = ? AND "votes"."user_id" = 1 LIMIT 1  [["song_id", 1]]
  Vote Load (0.3ms)  SELECT  "votes".* FROM "votes" WHERE "votes"."song_id" = ? AND "votes"."user_id" = 1 LIMIT 1  [["song_id", 2]]

Есть ли способ присоединить столбец vote.rate текущего пользователя к каждой песне в одном запросе?

0
Szymon Rut 29 Янв 2016 в 00:15

2 ответа

Лучший ответ

Лучше всего вам погрузиться в ActiveRecord Association Extensions:

#app/models/song.rb
class Song < ActiveRecord::Base
  has_many :votes, dependent: :destroy do
     def user_rate(user)
       find_by(user_id: user.id).try(:rate)
     end
  end
end

Это позволит:

@song   = Song.find params[:id]
@rating = @song.votes.user_rate current_user #-> 5 or false

Если вы не хотите выполнять другой запрос, вы можете попробовать свои силы в объекте proxy_association, доступном через расширение:

#app/models/song.rb
class Song < ActiveRecord::Base
  has_many :votes, dependent: :destroy do
     def user_rate(user)
       ratings = proxy_association.target.map {|f| [f.user_id, f.rate]}
       rating = ratings.select { |user_id, rate| user_id == user.id }
       rating.first[1] if rating.first
     end
  end
end

Вы сможете использовать:

@song   = Song.find params[:id]
@rating = @song.votes.try(:by, current_user) #-> 5 or false

if @rating
 ...
end

Не тестировалось, но раньше я использовал proxy_association аналогичным образом.

1
Szymon Rut 28 Янв 2016 в 23:44

Изменить на;

def self.user_rate(user)
  self.votes.find_by(user_id: user.id).try(:rate)
end
0
Richard Peck 28 Янв 2016 в 22:07