Ruby on Rails汇总属性的最佳方法

Question

在我当前正在创建的Web应用程序中，我必须为每个用户计算一个分数。 我目前正在以这种方式计算分数：

class Opinion < ActiveRecord::Base
  belongs_to :hero
  def score_value
    self.note * (self.end_date - self.start_date)
  end
end

class User < ActiveRecord::Base
  has_many :opinions
  def update_score
    self.score = 0
    self.opinions.each { |opinion| self.score += opinion.score_value }
    self.save
  end
end

但是我觉得这不是最好的方法。 有什么更好的办法吗？

Answer 1

好吧，我建议在#update_score使用所有的红宝石功能：

class User < ActiveRecord::Base
  has_many :opinions
  def update_score
    self.score = self.opinions.map(&:score_value).inject(0, &:+)
    self.save
  end
end

---

您可以做的另一件事是将一些计算移至数据库：

class User < ActiveRecord::Base
  has_many :opinions
  def update_score
    self.score = opinions.pluck(:note, 'DATEDIFF(end_date, start_date)').map do |note, days|
      note * days 
    end.inject(0, &:+)
    self.save
  end
end

这只是性能上的改进，我认为它并不优雅。

Answer 2

您可以在此处进行多项改进，我要做的是

属性更改时重新计算分数

class Opinion
  before_save :update_score

  def update_score
    return unless note_changed? || end_date_changed? || start_date_changed?
    self.note * (self.end_date - self.start_date)
  end
end

更新用户模型中分数的计算方式

class User
  def update_score
    self.score = self.opinions.sum(:score)
  end
end