Лучший способ реорганизовать это, не делая столько звонков, сколько я?

Question

Я пытаюсь перебрать несколько таких блоков. Они в основном сужают количество людей, которые выполняют множество атрибутов.

Я прошу прощения, если это кажется действительно грязным, но моя база данных действительно берет пошлину, обрабатывая это, и я знаю, что есть лучший способ. Я просто потерял стратегию прямо сейчас.

Мой код:

def count_of_distribution

#beginning with an array..
array_of_users = []

# any matching zip codes? ..
# zip_codes
@zip_codes = self.distributions.map(&:zip_code).compact
unless @zip_codes.nil? || @zip_codes.empty? 
  @matched_zips = CardSignup.all.map(&:zip_code) & @zip_codes
  @matched_zips.each do |mz| 
    CardSignup.find(:all, :conditions => ["zip_code = ?", mz]).each do |cs|
     array_of_users << cs.id
    end
  end
end

# any matching interests?..
# interest
@topics = self.distributions.map(&:me_topic).compact
unless  @topics.nil? || @topics.empty?
  @matched_topics = MeTopic.all.map(&:name) & @topics
  @matched_topics.each do |mt|
    MeTopic.find(:all, :conditions => ["name = ?", mt]).each do |mt2|
      mt2.users.each do |u|
        array_of_users << u.card_signup.id if u.card_signup
      end
    end
  end
end

# any matching sexes?..
# sex
@sexes = self.distributions.map(&:sex).compact
unless @sexes.nil? || @sexes.empty?
  @matched_sexes = CardSignup.all.map(&:sex) & @sexes
  @matched_sexes.each do |ms|
    CardSignup.find(:all, :conditions => ["sex = ?", ms]).each do |cs|
      array_of_users << cs.id
    end
  end
end

  total_number = array_of_users.compact.uniq

  return total_number
end

Это самый неприятный результат за всю историю:

Completed in 51801ms (View: 43903, DB: 7623) | 200 OK [http://localhost/admin/emails/3/distributions/new]

ОБНОВЛЕННЫЙ ОТВЕТ Это усечено, но все еще требует огромных потерь на DB

  array_of_users = []

  @zip_codes = self.distributions.map(&:zip_code).compact
  @sexes = self.distributions.map(&:sex).compact

  @zips_and_sexes = CardSignup.find(:all, :conditions => ["gender IN (?) OR zip_code IN (?)", my_sexes, my_zips])
  @zips_and_sexes.each{|cs| array_of_users << cs.id }

  @topics = self.distributions.map(&:me_topic).compact
  @all_topics = MeTopic.find(:all, :conditions => ["name IN (?)", @topics])
  array_of_users << CardSignup.find(:all, :conditions => ["user_id IN (?)", @all_topics.map(&:users)]).map(&:id)

Answer 1

Вы пытаетесь позволить рельсам выполнять все вычисления через серию циклов;Не удивительно, что это занимает так много времени.

Трудно следовать, но, возможно, вместо использования петель .each, попробуйте вытащить все, что вам нужно сразу же, а затем используйте .group_by(&:attribute)

ИЛИ, если ваш конечный результатЭто просто регистрация на карте.

Кажется, вы пытаетесь привлечь всех пользователей, у которых есть что-то нужное, почтовый индекс, тема или пол.Итак, пусть база данных сделает всю работу.

my_zips = @zip_codes = self.distributions.map(&:zip_code).compact.join(", ") my_sexes = @sexes = self.distributions.map(&:sex).compact.join(", ")

all_cards = CardSignup.find(:all, :conditions => ["sex IN (?) OR zip_code IN (?)", my_sexes, my_zips])

my_topics = @topics = self.distributions.map(&:me_topic).compact.join(", ") all_topics = MeTopic.find(:all, :conditions => ["name = ?", my_topics])

more_cards = all_topics.map{|x| x.users}.map{|n| n.card_signup} total_number = (all_cards + more_cards).flatten.uniq

Надеюсь, это лучший ответ.

Answer 2

Вот оно. Теперь он работает очень быстро:

array_of_users = []

# zips and sexes
@zip_codes = self.distributions.map(&:zip_code).compact
@sexes = self.distributions.map(&:sex).compact
@zips_and_sexes = CardSignup.find(:all, :conditions => ["gender IN (?) OR zip_code IN (?)", @sexes, @zip_codes])
@zips_and_sexes.each{|cs| array_of_users << cs.id }

# interest
@topics = self.distributions.map(&:me_topic).compact
@selected_topics = MeTopic.find(:all, :conditions => ["name in (?)", @topics]).map(&:id)
@matched_users = ActiveRecord::Base.connection.execute("SELECT * FROM `me_topics_users` WHERE (me_topic_id IN ('#{@selected_topics.join("', '")}') )")
@list_of_user_ids = []
@matched_users.each{|a| @list_of_user_ids << a[0] }
@list_of_user_ids.uniq!
array_of_users << CardSignup.find(:all, :conditions => ["user_id IN (?)", @list_of_user_ids]).map(&:id)

# age
@ages = self.distributions.map(&:age).compact
@ages_array = []
@ages.each{|a| @ages_array << how_old(a) }
@ages_array.each{|aa| array_of_users << aa.id}

array_of_users << CardSignup.all.map(&:id) if array_of_users.flatten.empty?
total_number = array_of_users.flatten.uniq

return total_number