Rails Arel выбирает отдельные столбцы

Я ударил небольшой блок с помощью новых методов scope (Arel 0.4.0, Rails 3.0.0.rc)

В основном я:

A topics, модель has_many :comments и a comments (с столбцом topic_id), которая belongs_to :topics.

Я пытаюсь получить коллекцию "Горячие темы", то есть темы, которые были недавно прокомментированы. Текущий код выглядит следующим образом:

# models/comment.rb
scope :recent, order("comments.created_at DESC")

# models/topic.rb
scope :hot, joins(:comments) & Comment.recent & limit(5)

Если я выполняю Topic.hot.to_sql, запускается следующий запрос:

SELECT "topics".* FROM "topics" INNER JOIN "comments"
ON "comments"."topic_id" = "topics"."id"
ORDER BY comments.created_at DESC LIMIT 5

Это работает отлично, но он потенциально возвращает повторяющиеся темы. Если тема № 3 была недавно прокомментирована несколько раз, она будет возвращена несколько раз.

Мой вопрос

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

Любые советы/предложения очень ценятся - я добавил 100 репрессий в надежде скоро прийти к элегантному решению.

Ответ 1

Решение 1

Это не использует Arel, но синтаксис Rails 2.x:

Topic.all(:select => "topics.*, C.id AS last_comment_id, 
                       C.created_at AS last_comment_at",
          :joins => "JOINS (
             SELECT DISTINCT A.id, A.topic_id, B.created_at
             FROM   messages A,
             (
               SELECT   topic_id, max(created_at) AS created_at
               FROM     comments
               GROUP BY topic_id
               ORDER BY created_at
               LIMIT 5
             ) B
             WHERE  A.user_id    = B.user_id AND 
                    A.created_at = B.created_at
           ) AS C ON topics.id = C.topic_id
          "
).each do |topic|
  p "topic id: #{topic.id}"
  p "last comment id: #{topic.last_comment_id}"
  p "last comment at: #{topic.last_comment_at}"
end

Убедитесь, что вы указали столбец created_at и topic_id в таблице comments.

Решение 2

Добавьте столбец last_comment_id в свою модель Topic. Обновите last_comment_id после создания комментария. Этот подход намного быстрее, чем использование сложного SQL для определения последнего комментария.

например:

class Topic < ActiveRecord::Base
  has_many :comments
  belongs_to :last_comment, :class_name => "Comment"
  scope :hot, joins(:last_comment).order("comments.created_at DESC").limit(5)
end

class  Comment
  belongs_to :topic

  after_create :update_topic

  def update_topic
    topic.last_comment = self
    topic.save
    # OR better still
    # topic.update_attribute(:last_comment_id, id)
  end
end

Это намного эффективнее, чем запуск сложного SQL-запроса для определения горячих тем.

Ответ 2

Это не так элегантно в большинстве реализаций SQL. Один из способов - сначала получить список пяти последних комментариев, сгруппированных по topic_id. Затем получите комментарии comments.created_at путем выбора sub с помощью предложения IN.

Я очень новичок в Arel, но что-то вроде этого может работать

recent_unique_comments = Comment.group(c[:topic_id]) \
                                .order('comments.created_at DESC') \
                                .limit(5) \
                                .project(comments[:topic_id]
recent_topics = Topic.where(t[:topic_id].in(recent_unique_comments))

# Another experiment (there has to be another way...)

recent_comments = Comment.join(Topic) \
                         .on(Comment[:topic_id].eq(Topic[:topic_id])) \ 
                         .where(t[:topic_id].in(recent_unique_comments)) \
                         .order('comments.topic_id, comments.created_at DESC') \
                         .group_by(&:topic_id).to_a.map{|hsh| hsh[1][0]}

Ответ 3

Для этого вам нужно иметь область с GROUP BY, чтобы получить последний комментарий для каждой темы. Затем вы можете заказать эту область с помощью created_at, чтобы получить последние комментарии по темам.

Следующие работы для меня используют sqlite

class Comment < ActiveRecord::Base

  belongs_to :topic

  scope :recent, order("comments.created_at DESC")
  scope :latest_by_topic, group("comments.topic_id").order("comments.created_at DESC")
end


class Topic < ActiveRecord::Base
  has_many :comments

  scope :hot, joins(:comments) & Comment.latest_by_topic & limit(5)
end

Я использовал следующий seeds.rb для генерации тестовых данных

(1..10).each do |t|
  topic = Topic.new
  (1..10).each do |c|
    topic.comments.build(:subject => "Comment #{c} for topic #{t}")
  end
  topic.save
end

И следующие результаты теста

ruby-1.9.2-p0 > Topic.hot.map(&:id)
 => [10, 9, 8, 7, 6] 
ruby-1.9.2-p0 > Topic.first.comments.create(:subject => 'Topic 1 - New comment')
 => #<Comment id: 101, subject: "Topic 1 - New comment", topic_id: 1, content: nil, created_at: "2010-08-26 10:53:34", updated_at: "2010-08-26 10:53:34"> 
ruby-1.9.2-p0 > Topic.hot.map(&:id)
 => [1, 10, 9, 8, 7] 
ruby-1.9.2-p0 > 

SQL, сгенерированный для sqlite (переформатированный), чрезвычайно прост, и я надеюсь, что Arel предоставит другой SQL для других движков, поскольку это, конечно же, потерпит неудачу во многих механизмах БД, поскольку столбцы в теме не входят в "Групповой список". Если это создало проблему, вы могли бы, вероятно, преодолеть ее, ограничив выделенные столбцы просто комментариями. Topic_id

puts Topic.hot.to_sql
SELECT     "topics".* 
FROM       "topics" 
INNER JOIN "comments" ON "comments"."topic_id" = "topics"."id" 
GROUP BY  comments.topic_id 
ORDER BY  comments.created_at DESC LIMIT 5

Ответ 4

Поскольку вопрос касался Arel, я думал, что добавлю это, поскольку Rails 3.2.1 добавляет uniq в QueryMethods:

Если вы добавите .uniq в Arel, он добавит DISTINCT в оператор select.

например. Topic.hot.uniq

Также работает в области:

например. scope :hot, joins(:comments).order("comments.created_at DESC").limit(5).uniq

Поэтому я бы предположил, что

scope :hot, joins(:comments) & Comment.recent & limit(5) & uniq

также должен работать.

См. http://apidock.com/rails/ActiveRecord/QueryMethods/uniq