Ruby on Rails - 每月最高票数答案

【问题标题】：Ruby on Rails - Monthly top vote getterRuby on Rails - 每月最高票数
【发布时间】：2013-03-11 00:52:07
【问题描述】：

我需要一些关于 Rails 投票系统的建议，该系统可以每月识别出票数最高的人。我有一个可以工作的系统，但对 Rails 不熟悉，我确信有更有效的方法可用。以下是我当前设置的简化版本（控制器代码省略）：

class Charity < ActiveRecord::Base
  has_many :votes
end

class Vote < ActiveRecord::Base
  belongs_to :charity
end

我的架构如下：

ActiveRecord::Schema.define(:version => 20130310015627) do
  create_table "charities", :force => true do |t|
    t.string   "name"
    t.text     "description"
    t.date     "last_win"
    t.datetime "created_at",  :null => false
    t.datetime "updated_at",  :null => false
  end 
  create_table "votes", :force => true do |t|
    t.integer  "charity_id"
    t.datetime "created_at", :null => false
    t.datetime "updated_at", :null => false
  end
end

我将使用 'whenever' gem 来运行 cron 作业以确定每月获胜者并更新 charities 表的 'last_win' 列。以下代码是我质疑我的效率的地方：

vote_counts = Vote.count(:group => "charity_id")
most_votes = vote_counts.values.max
winning_ids = vote_counts.map{|k,v| v == most_votes ? k :nil }.compact
charities = Charity.find(winning_ids)
charities.each {|charity| charity.update_attributes(:last_win => Date.today)}

我相信有很多方法可以更好地做到这一点，并希望能得到一些建议。如果您对设置投票表/关联的更好方法有任何建议，我们也将不胜感激。

提前致谢， CRS

【问题讨论】：

可以只有一个获胜者吗？看起来你的代码处理了关系。
可以有多个获奖者。 cron 作业将在该月的第一天运行。我会将其调整为仅计算上个月的投票。

标签： ruby-on-rails ruby ruby-on-rails-3.2 voting favorites

【解决方案1】：

类似这样的：

如果只有一个获胜者，我认为这会奏效

winner_id = Vote.group(:charity_id).order("count(*) desc").pluck(:charity_id).first
Charity.find(winner)id).update_attribute!(:last_win => Date.today)

你可以修改它的关系：

most_votes = Vote.group(:charity_id).order("count(*) desc").count.first[1]
winners = Vote.group(:charity_id).having("count(*) = ?", most_votes).pluck(:charity_id)

Charity.where(:id => winners).update_all(:last_win => Date.today)

确保所有内容都在您的数据库中正确编入索引，

您可能可以更精简它，但 SQL 会变得更复杂。

【讨论】：

感谢您的帮助。感谢您提醒我有关索引的信息。

【解决方案2】：

最后两行可能是：

Charity.where(id:winning_ids).update_all(last_win:Date.today)

这将转化为单个 SQL 更新命令，而不是为每个获奖慈善机构发出更新命令。

您确定获胜慈善机构的第一部分看起来不错，因为您将其作为一项 cron 工作运行，您可能不在乎是否需要几分钟。

但是，如果您想实时显示这些值，您可以在 Vote 上添加一个 after_create 挂钩以更新其所有者慈善机构的计数器（可能在另一个表中）：

class Vote < ActiveRecord::Base
  belongs_to :charity
  after_create :increment_vote_count
  CharityVote.where(year:Time.now.year, month:Time.now.month, 
    charity_id:self.charity_id).first_or_create.increment!(:counter)
end

【讨论】：