Как я могу отсортировать свои записи по среднему рейтингу?

Question

У меня есть таблица мест, которые я показываю на странице индекса мест как частичные. У меня также есть таблица отзывов, где одно место может иметь много отзывов, и каждый отзыв имеет рейтинг 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 звездный обзор показывает три частичные, один сверху, один посередине и один внизу списка.

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

Спасибо за любую помощь, она очень ценится!

редактировать

Модель объекта

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

Answer 1

Чтобы добавить к ответу 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, чтобы предотвратить деление на ноль.

При создании обзора необходимо добавить << или concatэто к ассоциации reviews объекта места, так что обратный вызов запускается.Например, это свяжет рецензию с местом, создаст рецензию (ВСТАВЬТЕ в БД) и вызовет обратный вызов:

@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
  ...

Чтобы исправить это целесообразно, вы можете просто добавить @review.venue.update_average_rating непосредственно перед строкой w / flash[:notice].

Answer 2

Если я правильно понимаю, у вас есть модель места проведения, у которой есть has_many: обзоры, и у каждого обзора есть столбец "рейтинг".

Я предлагаю альтернативный код, приведенный, например, Майклом, который должен быть НАМНОГО быстрее и готов к миллионам записей, но требует некоторой обработки, после чего добавляется обзор (рассмотренный в этом примере), который дает вам огромную производительность увеличить, когда записи выбраны, упорядочены и показаны:

Создайте миграцию, которая добавляет среднее_рейтинг в виде числа с плавающей запятой:

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

Answer 3

Предположим, что рецензии доступны для редактирования, тогда нет хороших ответов. Итак, я сделал, как показано ниже.

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

Answer 4

Вы можете создать метод average_rating для Venue, а затем просто:

@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

Это решение, вероятно, не оптимально, если имеется огромное количество записей или если производительность критична, но она очень проста и работает.