レール：私は次のコードを持っているアレイ

Question

に中括弧を削除します。

@temp = SalesOrder.where("status > ?", 0).ids 
items = SalesOrderItem.where(sales_order_id: @temp).where.not(product_id: nil) 

total = items.to_a.group_by(&:product_id).map do |product_id, quantity| 
    {:product_name => Product.find(product_id.to_i).name, 
    :quantity => quantity.sum { |q| q.quantity.to_f } } 
end 

@top_five = total.sort_by { |h| h[:quantity] }.reverse! 

@top_fiveすると、この配列を生成します。

[{:product_name => "Six", :quantity => 1300.0}, 
{:product_name => "Plastic Tumbler 620ml", :quantity => 1205.0}, 
{:product_name => "Product Four", :quantit y=> 1110.0}, 
{:product_name => "Product Five", :quantity => 510.0}, 
{:product_name => "Fiber Optic Cable", :quantity => 200.0}] 

私はそれがこの形式で

{"Football" => 10, "Basketball" => 5} 

する必要がありますまたはこれは

[["Football", 10], ["Basketball", 5]] 

Answer 1

@temp = SalesOrder.where("status > ?", 0).ids 
items = SalesOrderItem.where(sales_order_id: @temp).where.not(product_id: nil) 
total = items.to_a.group_by(&:product_id).each_with_object({}) do |(product_id, quantity), total| 
    total[Product.find(product_id.to_i).name] = quantity.map(&:quantity).map(&:to_f).sum 
end 
@top_five = total.sort_by { |k, v| v }.reverse! 

これを確認してください。それは動作するはずです。何かエラーがありましたら、私にpingしてください。私はそれを更新します

PS：あなたのコードはまったく最適化されていません。これらはすべて単一のSQLクエリで行うことができますが、ロジックはデバッグなしでこのクエリを書くのはかなり難しいです