Блог

У меня есть эта область в моей модели исполнителя, которая дает мне исполнителей в порядке их популярности в течение определенного периода времени. popularity в popularity_caches таблице вычисляется каждый день.

 scope :by_popularity, lambda { |*args|
  options = (default_popularity_options).merge(args[0] || {})

  select("SUM(popularity) AS popularity, artists.*").
from("popularity_caches FORCE INDEX (popularity_cache_group), artists FORCE INDEX (index_artists_on_id_and_genre_id)").
where("popularity_caches.target_type = 'Artist'").
where("popularity_caches.target_id = artists.id").
where("popularity_caches.time_frame = ?", options[:time_frame]).
where("popularity_caches.started_on > ?", options[:started_on]).
where("popularity_caches.started_on < ?", options[:ended_on]).
group("artists.id").
order("popularity DESC")
}
  

Кажется, это работает, за исключением случаев, когда я хочу получить количество : Artist.by_popularity.count . В ответ я получаю забавный хэш (вероятно, количество исполнителей, у которых есть popularity_caches за этот период):

 #<OrderedHash {295954=>1, 20143=>1, 157532=>1, 181291=>1, 300086=>1, 50100=>1, 262898=>1, 293888=>1, 130158=>2, 279943=>1, 336758=>1, 100201=>1, 134290=>2, 22726=>3, 144620=>2, 62497=>2 # snip
  

Это SQL, который я, вероятно, хочу взамен:

 SELECT COUNT(DISTINCT(artists.id)) AS count_all
FROM popularity_caches FORCE INDEX (popularity_cache_group), artists FORCE INDEX (index_artists_on_id_and_genre_id)
WHERE (popularity_caches.target_type = 'Artist')
  AND (popularity_caches.target_id = artists.id)
  AND (popularity_caches.time_frame = 'week')
  AND (popularity_caches.started_on > '2011-02-28 16:00:00')
  AND (popularity_caches.started_on < '2011-10-05')
ORDER BY popularity DESC
  

Чтобы получить count, мне пришлось создать отдельный метод, который в значительной степени делает то же самое, за исключением того, что SQL формируется по-другому. Это своего рода отстой, потому что, когда я хочу разбить на страницы, я должен передать две вещи:

 @artists = Artists.by_popularity(some args).paginate(
  :total_entries => Artist.count_by_popularity(pass in the same args here as in Artist.by_popularity),
  :per_page => 5,
  page => ...
)
  

Для меня это пахнет, потому что оно очень хрупкое.

Есть ли способ сделать это в ARel? Возможно, переопределить, как он подсчитывает вещи (разные artists.id ) и удаление group by , чтобы он не возвращал хэш для подсчета?

Спасибо!

Решаемая с помощью удивительного scuttle.io:

 PopularityCach.select(
  Arel::Nodes::Group.new(Artist.arel_table[:id]).count.as('count_all')
).where(
  PopularityCach.arel_table[:target_type].eq('Artist').and(
    PopularityCach.arel_table[:target_id].eq(Artist.arel_table[:id]).and(
      PopularityCach.arel_table[:time_frame].eq('week').and(
        PopularityCach.arel_table[:started_on].gt('2011-02-28 16:00:00').and(
          PopularityCach.arel_table[:started_on].lt('2011-10-05')
        )
      )
    )
  )
).order(:popularity).reverse_order