Rails查找条件…其中属性不是数据库列

display_ready_products = Product.all.select(&:display_ready?)

这是非常低效的,特别是如果你有大量的产品,可能是不合格的。

更好的方法是为照片设置计数器缓存,并在上载文件时设置标志:

class Product < ActiveRecord::Base
  has_many :photos
end

class Photo < ActiveRecord::Base
  belongs_to :product, :counter_cache => true
end

add_column :products, :photos_count, :default => 0

这将给你一个列与照片的数量。有一种方法可以在开始时用正确的数字而不是零来预填充这些计数器,但是这里不需要讨论这个问题。

添加列以记录文件标志:

add_column :products, :file_exists, :boolean, :null => false, :default => false

保存时触发:

class Product < ActiveRecord::Base
  before_save :assign_file_exists_flag

protected
  def assign_file_exists_flag
    self.file_exists = File.exist?(self.file.path)
  end
end

由于这两个属性被呈现到数据库列中,您现在可以直接查询它们:

Product.find(:all, :conditions => 'file_exists=1 AND photos_count>0')

您可以通过编写两个将封装该行为的命名作用域来清除这个问题。

1) 从数据库中选择所有可能的行。这种情况发生在数据库中。

2) 在Ruby中,从所有行中选择有效行。如

possible_products = Product.find(:all, :conditions => {:featured => true})
products = possible_products.select{|p| p.display_ready?}

补充:

products = Product.find(:all, :conditions => {:featured => true}).select {|p|
               p.display_ready?}

第二个选择是数组对象的选择方法。选择是一个非常方便的方法,连同检测。(Detect来自Enumerable,并与数组混合在一起。)