提高地图命令的效率？

Question

我的 rails 程序中有 4 个数据库，产品、产品、产品和订单

样品产品：中型套件

Medium Kit 包含 2 个产品（这是在 OfferingProduct 数据库中设置的） 1 小 2 中等

客户订购 2 个中号套件

此程序应占此订单中的 2 个小型和 4 个中型以及许多类似的订单

我正在尝试建立一个库存系统来跟踪每周售出的每种产品的数量，以便我可以重新订购产品。要计算这个，我有：

def get_trend
  product = Product.first
  customer_orders =[]
  product.offerings.each do |o|
    customer_orders = customer_orders + o.orders
  end
  o=customer_orders.group_by { |t| t.date.beginning_of_week }
  y = []
  op=self.offering_products;
  o.sort.each do |week, orders|
    y << orders.map { |o| o.quantity*op.find_by_offering_id(o.offering).quantity }.sum
  end
  return y
end

这似乎可行，但计算大约需要 20 秒。有什么办法可以让这更快吗？我知道大部分时间都花在了

y << orders.map { |o| o.quantity*op.find_by_offering_id(o.offering).quantity }.sum

计算给定一周内订购的产品数量的行。有什么想法吗？

Answer 1

问题是，当您最好使用声明（即数据库中的联接）时，您正在使用迭代（即应用程序代码中的循环）。循环永远不会像连接一样快，将数据集移动到应用程序的内存中也无济于事。

我猜你正在使用 ActiveRecord？如果是这样，这可能有助于如何使用连接进行查询：http://guides.rubyonrails.org/active_record_querying.html

Answer 2

感谢 Iain 提供的信息，这就是我最终所做的事情

def get_trend
    sql = ActiveRecord::Base.connection()
    if Rails.env == "production"
        d=sql.execute("SELECT SUM(orders.quantity * offering_products.quantity), EXTRACT(ISOYEAR FROM orders.date) AS year, EXTRACT(WEEK FROM orders.date) AS week " +
        " FROM orders " +
        " INNER JOIN offerings ON offerings.id = orders.offering_id " + 
        " INNER JOIN offering_products ON offering_products.offering_id = offerings.id " +
        " INNER JOIN products ON products.id = offering_products.product_id " +
        " WHERE (products.id = #{self.id}) GROUP BY year, week ORDER BY year, week ")
        y=d.map { |a| a["sum"].to_i }
    else
        d=sql.execute("SELECT SUM(orders.quantity * offering_products.quantity), strftime('%G-%V', orders.date) AS year " +
        " FROM orders " +
        " INNER JOIN offerings ON offerings.id = orders.offering_id " +
        " INNER JOIN offering_products ON offering_products.offering_id = offerings.id " + 
        " INNER JOIN products ON products.id = offering_products.product_id " +
        " WHERE (products.id = #{self.id}) GROUP BY year")
        y=d.map { |a| a[0] }
    end
    return y
end

更新：需要区分生产和本地代码，因为 heroku 使用 postgresql，并且日期函数的工作方式不同:( 这也意味着结果不完全相同，因为函数对一年中的第一周的处理方式不同。