Rails：根据多个参数显示产品

Question

我正在学习 Rails 并尝试建立一个产品库，其中的产品将根据三个元素进行显示：位置、类别和到期日期（产品可以有多个位置和类别，但只有一个到期日期）。只要产品尚未过期，就会显示产品，并且可以通过下拉菜单选择位置和类别。

我开始写这个问题时，在合并位置和类别选择标准时遇到了困难，我找到了解决方案，但非常感谢任何有关可以做得更好的帮助。

我通过连接使用 has_many 来创建产品、位置和类别之间的连接。

这是模型：

class Product < ActiveRecord::Base  
  has_many :categorizations  
  has_many :categories, :through => :categorizations  
  has_many :localizations  
  has_many :locations, :through => :localizations  
end  

class Categorization < ActiveRecord::Base
  belongs_to :product
  belongs_to :category
end  

class Category < ActiveRecord::Base
  has_many :categorizations
  has_many :products, :through => :categorizations
end  

class Localization < ActiveRecord::Base
  belongs_to :product
  belongs_to :location
end 

class Location < ActiveRecord::Base
  has_many :localizations
  has_many :products, :through => :localizations
end 

这是我的控制器。地点及地点类别 ID 作为参数传递，并且产品的到期日期必须大于当前时间：

class LibraryController < ApplicationController
   def index
     @products = Product.find(:all, include => [ :locations, :categories ],
                              :conditions => ['expiry_date > ? AND locations.id = ? AND categories.id = ?',
                              Time.now, params[:location_id],params[:category_id]])
   end
end

因此，通过在 URL 中传递 location_id 和 Category_id 参数，我可以通过两者的组合列出产品。

有没有更好的方法来实现我想要做的事情？

Answer 1

这也将实现您想要的功能：

@products = Product.find_all_by_category_id_and_location_id(params[:category_id], params[:location_id])

您还可以使用 Product.where ，据说这比 find 更好。

欲了解更多信息，请谷歌“动态查找器”。

Answer 2

好的。不，我认为在这种情况下没有“更好”的方法。当然有“不同”的方式来做你想做的事，但从表面上看，你所做的很好，并且不会尖叫“这段代码太糟糕了！”或任何东西。

这里很难回答建议/风格的问题，因为最终的答案是，“在网络上搜索其他人在你的情况下正在做什么，如果你的解决方案看起来传统/合乎逻辑，你自己评估/做出决定”，或者这类问题可以通过研究该主题的相关书籍来回答。

回答这样的定性问题几乎是不可能的，因为：

1. 解决每个问题都有多种方法，其中许多方法既不是“正确”也不是“错误”
2. 总是存在人们违反“规则”的边缘情况，在这种情况下甚至是非常规的解决方案绝对是做某事的最佳方式
3. 。您是开发人员，是构建该事物的人。在某种程度上，你应该发挥领导作用，并决定什么是最好的。

我要求你定义“更好”的原因主要是因为#1 - 除非你给我们一个你想要实现的具体结果，否则你所有的你会得到 (a) 充满意见的答案，而不是针对特定目标，或 (b) 只是采取不同的做事方式，可能对你有帮助，也可能没有帮助。因此，它们在实际中并不是很有用。

Answer 3

您还可以通过使用“Product.where”（优于 Rails 3.1 中的 find）来改进您的解决方案，并将它们转换为 Rails 中的named_scopes，并根据需要链接它们。

scope :not_expired, where('expiry_date > ?', Time.now)