logo标签列表

如何按平均评分对我的记录进行排序？

ruby-on-railsruby-on-rails-3sorting
5

我有一个场馆表格，我正在展示它们作为局部视图在场馆索引页面上。我还有一个评论表格，其中一个场馆可以有多个评论，每个评论都有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
4个回答
8

除了NoICE的答案外，通过钩入:after_add和:after_remove关联回调，您无需记住调用特殊的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，以避免除以零。
创建评论时，必须将其附加到场馆对象的“reviews”关联中并使用<<或concat进行连接，以触发回调。例如，这将将评论与场馆相关联，创建评论（插入到数据库中）并触发回调：
@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。请注意，不要删除任何HTML标签。
嘿，谢谢！我不知道有 :after_add 和 :after_remove 回调函数，非常有用！:) （附注：零除检查也很方便:D） - Dalibor Filus
PS：我过去在评分模型中使用 after_save 解决了这些问题，但这种方法更加简洁。 - Dalibor Filus
@NoICE 不用谢。像这样的技巧有无数种。我已经全职使用Rails一年多了，仍在学习中。 - Jonathan Tran
@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
@Dave 更新了回调函数的触发示例。不要在控制器中定义update_average_rating，因为它与数据模型有关，应该放在模型类中。如果需要手动调用它，请执行Venue.find(params[:id]).update_average_rating。 - Jonathan Tran
@Jonathan Tran，我不确定如何将回调请求与我的创建评论方法结合起来。如果我将其添加到我的问题中，它可以正常工作，但是它无法保存作者的user_id。 - Dave
4

如果我理解正确，你有一个场馆模型（model venue），它具有has_many :reviews关联，并且每个评论都有一个"rating"列。

我提供了一个替代代码，可以比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
PS. 当然也有通过纯 SQL 实现的方法，但是这需要使用子查询和分组语句... 这将在 SQL 服务器端创建“临时表”，非常缓慢，并且当您有大量记录时可能会导致服务器崩溃。 - Dalibor Filus
@NoICE，感谢您的出色回答，我下班后会尝试一下。 - Dave
@NoICE，我在average_rating字段上遇到了问题，它始终保持默认值0.0不变。我可以从控制台添加新场馆并为它们设置平均评分，这很好用，但是当添加评论时该字段就无法更改。我已将您编写的代码添加到我的应用程序中（我是超级新手，可能犯了一个愚蠢的错误）。 - Dave
你好，在我的示例中，您必须通过调用 @venue.add_rating(options_for_review) 来创建评价。虽然我承认这不是最佳解决方案（方法的命名也不是最佳），但正如 @Jonathan Tran 建议的那样，您可以像他展示的那样通过 :after_add 和 :after_remove 调用此方法，这确实非常有用！ :) - Dalibor Filus
@NoICE，我已经添加了:after_add调用，但仍然无法更新记录中的average_rating字段。我尝试将update_average_rating方法移动到控制器并更改最后一行为Venue.find(params[:id]).update_attribute(:average_rating, c == 0 ? 0.0 : s / c.to_f)，但仍然没有效果。我已经在我的问题中添加了开发日志以记录添加评论时的情况。 - Dave
0
假设评论是可编辑的，那么没有答案能够很好地工作。因此我已经按照以下方式完成了。
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
0
你可以在 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

如果记录数量很大或性能至关重要，那么这种解决方案可能不是最优的，但它非常简单并且有效。

感谢查看，这正是我需要的，但它返回了一个“#ActiveRecord::Relation:0x41095b0”的未定义方法`include'”错误。 - Dave
抱歉，includes 是带有一个 s 的。我修正了答案。但它是不必要的：它只会减少查询的数量。 - Michaël Witrant
网页内容由stack overflow 提供, 点击上面的
可以查看英文原文，
原文链接