有没有更有效的方法来获取这个数组？

Question

我有一个模型项目，用户可以通过创建新的 UpVote 或 DownVote 对其进行投票。创建 UpVote 或 Downvote 时，它​​会记录用户的 ip 地址和 Item 的 id。我想列出当前用户的ip没有投票的前100个Item的数组。

到目前为止，这是我所拥有的：

架构

create_table "up_votes", force: true do |t|
  t.string   "ip"
end

create_table "down_votes", force: true do |t|
  t.string   "ip"
end

型号

class Item < ActiveRecord::Base

  def up_votes_array
    self.up_votes.map(&:ip).to_a
  end

  def down_votes_array
    self.down_votes.map(&:ip).to_a
  end

  def up_voted?(ip)
    self.up_votes_array.include? ip
  end

  def down_voted?(ip)
    self.down_votes_array.include? ip
  end

控制器

@not_voted = Item.where(show: true).select { |item| !item.up_voted?(request.remote_ip) }.select { |thing| !item.down_voted?(request.remote_ip) }.sort_by(&:alphabetical).reverse.first(100).shuffle

---

它可以工作，但有些东西似乎不必要地复杂，我担心随着数据库的增长，它可能会变得低效。有没有更有效的方法来获取这个数组？

我正在使用 Rails 4 和 Sqlite3。

Answer 1

我认为理想的查询应该是这样的：

Item.where(show: true).joins(:up_votes).joins(:down_votes).where('up_votes.ip != ?', request.remote_ip).where('down_votes.ip != ?', request.remote_ip).limit(100)

这将产生一个最佳查询并在您的内存中加载 100 行，这与给定的查询相反，它为Item 加载整个表，并且所有迭代都在它的赞成票和反对票上进行检查。

您也可以将查询组合在一起： Item.where(show: true).joins(:up_votes).joins(:down_votes).where('up_votes.ip != ? and down_votes.ip != ', request.remote_ip, request.remote_ip).limit(100)