Найти условие нуждается в рефакторинге / оптимизации полиморфной ассоциации - PullRequest
Новая
       8

Найти условие нуждается в рефакторинге / оптимизации полиморфной ассоциации

1 голос
/ 05 апреля 2011

Я создаю метод рекомендаций для пользователей в моем проекте.Пользователи создают записи по интересам всякий раз, когда они просматривают, создают, комментируют или взаимодействуют с объектами (взвешенными в зависимости от действия).

Я написал метод find, который просматривает интересы пользователя, а затем находит пользователей, которые такжезаинтересованы в этих пунктах.Тем не менее, это ужасно неэффективно, делать столько вызовов дб, сколько интересует пользователь (до 50).

Вот урезанная версия того, что происходит: 

#User.rb
...
has_many :interests, :as => :interestable, :dependent => :destroy

def recommendations
  recommendations = []

  Interest.for(self).limit(50).each do |item|
    recommendations << Interest.other_fans_of(item)
  end

  user_ids = recommendations.flatten.map(&:user_id).uniq
end
...

#interest.rb

...

belongs_to :user
belongs_to :interestable, :polymorphic => true

named_scope :for, lambda { |user| { :conditions => { :user_id => user.id } } }
named_scope :limit, lambda { |num| { :limit => num } }
named_scope :other_fans_of, lambda { |interest| { :conditions => { :interestable_type => interest.interestable_type, :interestable_id => interest.interestable_id } } }
default_scope :order => "weight DESC"

...

Есть ликакие-нибудь гении sql, которые могут превратить это в один хороший чистый вызов БД?

1 Ответ

0 голосов
/ 04 ноября 2011

Нечто подобное должно делать эту работу. Там могут быть более красивые способы ... 

class User < ActiveRecord::Base
  #...
  def recommendations
    # get a list of the relevant interests
    the_interests = Interest.for(self).limit(50).map{|x| [x.interestable_type, x.interestable_id]}
    # make some sql
    conditions = the_interests.map{|x| "(`interestable_type`=? AND `interestable_id`=?)"}.join(" OR ")
    # use ruby magic to make a valid finder and get the other user_ids
    user_ids = Interest.all(:select => '`user_id`', :conditions => [conditions, *(the_interests.flatten)]).map(&:user_id).uniq
  end
  #...
end
...