如何按平均评分对记录进行排序？答案

【问题标题】：How can I sort my records by average rating?如何按平均评分对记录进行排序？
【发布时间】：2011-05-15 11:10:16
【问题描述】：

我有一张场地表，我在场地索引页面上显示为部分场地。我还有一张评论表，其中一个场地可以有很多评论，每条评论的评分为 1-5。

我正在尝试让场地显示在索引页面上，平均评分最高的场地位于顶部并降序排列。

控制器代码如下所示：

场地管理员

def index
    if
      @venues = Venue.with_type(params[:venuetypes]).with_area(params[:areas]).joins(:reviews).order("reviews.rating DESC")
    else
      @venues = Venue.all
    end
  end

这给出了这样的结果：

如果场地 1 有 5 星评价在顶部显示场地部分列表。
如果场地 2 的评价为 5 星，并且 1 星评论显示两个部分，一个在顶部，一个在
在列表的底部。
如果场地 3 有 5 星评价，3 星评价和 1 星评论显示三个部分，一个在顶部，一个在中间，一个在在列表的底部。

我只希望每个场馆有一个部分展示，但按平均评级在列表中定位，我觉得有一个 .average 或某处缺少什么我怎么能做到这一点？

非常感谢您的帮助！

编辑

场地模型

class Venue < ActiveRecord::Base
  attr_accessible :name, :addressline1, :addressline2, :addressline3, :addressline4, :postcode, :phonenumber, :about, :icontoppx, :iconleftpx, :area_id, :venuetype_id, :lat, :long, :venuephotos_attributes
  belongs_to :area
  belongs_to :venuetype
  has_many :reviews
  has_many :venuephotos

  accepts_nested_attributes_for :venuephotos, :allow_destroy => true

  scope :with_type, lambda { |types|
    types.present? ? where(:venuetype_id => types) : scoped }

  scope :with_area, lambda { |areas|
    areas.present? ? where(:area_id => areas) : scoped }

  def to_param
    "#{id}-#{name.gsub(/\W/, '-').downcase}"
  end

  def add_rating(rating_opts)
    @venue.add_rating(:rating => rating, :reviewer => params[:rating][:reviewer])
    self.reviews.create(rating_opts)
    self.update_rating!
  end

  def update_rating!
    s = self.reviews.sum(:rating)
    c = self.reviews.count
    self.update_attribute(:average_rating, s.to_f / c.to_f)
    self.save(:validate => false)
  end
end

添加评论的开发日志

Started POST "/venues/44-rating-test-5/reviews" for 127.0.0.1 at 2011-05-18 09:24:24 +0100
  Processing by ReviewsController#create as JS
  Parameters: {"utf8"=>"âœ“", "authenticity_token"=>"GZWd67b5ocJOjwKI6z9nJInBXxvQahHrjUtUpdm9oJE=", "review"=>{"rating"=>"5", "title"=>"5 star review"}, "venue_id"=>"44-rating-test-5"}
  [1m[36mVenue Load (1.0ms)[0m  [1mSELECT `venues`.* FROM `venues` WHERE (`venues`.`id` = 44) LIMIT 1[0m
  [1m[35mUser Load (0.0ms)[0m  SELECT `users`.* FROM `users` WHERE (`users`.`id` = 3) LIMIT 1
  [1m[36mSQL (0.0ms)[0m  [1mBEGIN[0m
  [1m[35mSQL (2.0ms)[0m  describe `reviews`
  [1m[36mAREL (0.0ms)[0m  [1mINSERT INTO `reviews` (`title`, `created_at`, `updated_at`, `venue_id`, `user_id`, `rating`) VALUES ('5 star review', '2011-05-18 08:24:24', '2011-05-18 08:24:24', NULL, 3, 5)[0m
  [1m[35mSQL (27.0ms)[0m  COMMIT
  [1m[36mSQL (0.0ms)[0m  [1mBEGIN[0m
  [1m[35mAREL (0.0ms)[0m  UPDATE `reviews` SET `venue_id` = 44, `updated_at` = '2011-05-18 08:24:24' WHERE (`reviews`.`id` = 90)
  [1m[36mSQL (23.0ms)[0m  [1mCOMMIT[0m
  [1m[35mSQL (1.0ms)[0m  SELECT COUNT(*) FROM `reviews` WHERE (`reviews`.venue_id = 44)
  [1m[36mUser Load (0.0ms)[0m  [1mSELECT `users`.* FROM `users` WHERE (`users`.`id` = 3) LIMIT 1[0m
Rendered reviews/_review.html.erb (9.0ms)
Rendered reviews/create.js.erb (22.0ms)
Completed 200 OK in 220ms (Views: 56.0ms | ActiveRecord: 54.0ms)

编辑 创建审核方法（审核控制器）

def create
    @review = current_user.reviews.create!(params[:review])
    @review.venue = @venue
    if @review.save
      flash[:notice] = 'Thank you for reviewing this venue!'
      respond_to do |format|
        format.html { redirect_to venue_path(@venue) }
        format.js
      end
    else
      render :action => :new
    end
  end

【问题讨论】：

标签： ruby-on-rails ruby-on-rails-3 sorting

【解决方案1】：

要添加到NoICE's answer，通过挂钩:after_add 和:after_remove association callbacks，您不必记得调用特殊的add_rating 方法。

class Venue < ActiveRecord::Base
  has_many :reviews, :after_add => :update_average_rating, :after_remove => :update_average_rating

  def update_average_rating(review=nil)
    s = self.reviews.sum(:rating)
    c = self.reviews.count
    self.update_attribute(:average_rating, c == 0 ? 0.0 : s / c.to_f)
  end

end

此外，您还需要检查计数是否为 0，以防止被零除。

创建评论时，您必须将<< 或concat 附加到场地对象的reviews 关联，以便触发回调。例如，这会将评论与场地相关联，创建评论（INSERT 到数据库中），并触发回调：

@venue = Venue.find(params[:venue_id])
@venue.reviews << Review.new(params[:review])

这将创建评论，但不会触发回调，即使 venue_id 是参数：

Review.create(params[:review])

如果你真的想让你的动作触发回调，你可以将你的代码更改为：

def create
  @review = Review.new(params[:review].merge({ :user => current_user, :venue => @venue })
  if @review.valid? and @venue.reviews << @review
  ...

不过，为了方便地修复它，您可以在带有 flash[:notice] 的行之前添加 @review.venue.update_average_rating。

【讨论】：

嘿，谢谢！不知道 :after_add 和 :after_remove 回调，非常有用！ :) （ps 除以零检查也很方便：D）
附注我过去在 Rating 模型中使用 after_save 解决了这些问题 :) 但这要干净得多
@NoICE 不客气。像这样的把戏不计其数。我已经全职做 Rails 一年多了，而且还在学习。
@Jonathan Tran，我添加了 :after_add 调用，但它仍然没有更新场地记录中的 average_rating 字段我尝试将 update_average_rating 方法移动到控制器并将最后一行更改为 Venue .find(params[:id]).update_attribute(:average_rating, c == 0 ? 0.0 : s / c.to_f) 但仍然没有，我已经为我的问题添加评论时添加了开发日志，可以这与使用 javascript 函数的 add review 方法有关吗？
@Dave 更新了答案，其中包含触发回调的示例。不要在你的控制器中定义update_average_rating；它属于模型类，因为它与您的数据模型有关。如果您需要手动调用它，请执行Venue.find(params[:id]).update_average_rating。

【解决方案2】：

如果我理解正确，您有模型场地，其中有 has_many :reviews，每个评论都有“评分”列。

我正在提供替代代码，例如 Michael 给出的代码，它应该更快并且准备好数百万条记录，但它需要一些处理然后添加审查（在本示例中介绍），这会给您带来巨大的性能选择、排序和显示记录时提升：

创建一个将 average_rating 添加为浮点数的迁移：

add_collumn :venues, :average_rating, :float, :default => 0.0, :null => false
add_index :venues, :average_rating

现在，在您的控制器中：

# perhaps add paginate at the end instead of .all ...
@venues = Venue.with_type(params[:venuetypes]).with_area(params[:areas]).order("average_rating DESC").all

模型更新：

class Venue < ActiveRecord::Base
  has_many :reviews

  # you'll need to create ratings for this venue via this method, so everything is atomic
  # and transaction safe
  # parameter is hash, so you can pass as many review parameters as you wish, e.g.
  # @venue.add_rating(:rating => rating, :reviewer => params[:rating][:reviewer])
  # or
  # @venue.add_rating(params[:rating])
  # :)
  def add_rating(rating_opts)
    # you can of course add as 
    self.reviews.create(rating_opts)
    self.update_rating!
  end

  # let's update average rating of this venue
  def update_rating!
    s = self.reviews.sum(:rating)
    c = self.reviews.count
    self.average_rating = s.to_f / c.to_f
    self.save(:validate => false)
    # or you can use .update_attribute(:average_rating, s.to_f / c.to_f)
  end

end

希望这会有所帮助。如果您有任何问题，请询问。

问候，NoICE

【讨论】：

附言。也应该有办法通过纯 SQL 来做到这一点，但是你需要子查询和 group by 子句......这将在 SQL 服务器端创建“临时表”，这非常慢并且可以杀死你的服务器你有很多记录。
@NoICE，感谢您的出色回答，我下班后会试一试
@NoICE，我的 average_rating 字段有问题，它没有从默认的 0.0 更改，我可以添加新场地并从控制台为它们设置 average_rating，效果很好，添加评论时，字段不会改变。我已将您编写的代码添加到我的问题中，因为我已将其放在我的应用程序中（我对此非常陌生，所以可能犯了一个愚蠢的错误）
您好，在我的示例中，您必须通过调用 @venue.add_rating(options_for_review) 来创建评论......好吧，我承认这不是最佳解决方案（也不是方法的命名），但作为 @ Jonathan Tran 建议，您可以通过 :after_add 和 :after_remove 调用此方法，如他所示，确实非常有用！ :)
@NoICE，我添加了 :after_add 调用，但它仍然没有更新记录中的 average_rating 字段我尝试将 update_average_rating 方法移动到控制器并将最后一行更改为 Venue.find (params[:id]).update_attribute(:average_rating, c == 0 ? 0.0 : s / c.to_f) 但仍然没有，我已经添加了开发日志，以便在我的问题中添加评论时

【解决方案3】：

您可以在 Venue 上创建一个 average_rating 方法，然后简单地：

@venues = Venue.with_type(params[:venuetypes]).with_area(params[:areas]).includes(:reviews).sort_by(&:average_rating).reverse

方法：

class Venue
  def average_rating
    ratings = reviews.map(&:rating)
    ratings.sum.to_f / ratings.size
  end
end

如果有大量记录或性能很关键，此解决方案可能不是最佳解决方案，但它非常简单且有效。

【讨论】：

感谢您的观看，这只是我所追求的那种东西，但它为 #<:relation:0x41095b0> 错误返回了一个未定义的方法 `include'
抱歉，includes 带有 s。我确定了答案。但这不是必需的：它只会减少查询的数量。

【解决方案4】：

让我们假设评论是可编辑的，那么没有答案可以正常工作。所以我做了如下的事情。

class Venue < ActiveRecord::Base
    has_many :reviews, :dependent => :delete_all
end

现在评论模型如下。

class Venue < ActiveRecord::Base
    belongs_to :venue

    # If reviews are editable
    after_save :update_average_rating 

    # If reviews are deletable
    before_destroy :update_average_rating 

    private
      def update_average_rating
       s = self.venue.reviews.sum(:rating)
       c = self.venue.reviews.count
       self.venue.update_attribute(:average_rating, c == 0 ? 0.0 : s / c.to_f)
  end
end

【讨论】：