Rails;総計

Question

Rubyのように以下のコードをリファクタリングしたいと思います。

def cal_total 
    total = nil 
    items.each do |item| 
     total = total + item.itemable.amount 
    end 
    total 
end 

基本的にオブジェクトを反復し、各量をフェッチして合計を返します。

更新

total = item.itemable.amountは正しいです。

Answer 1

あなたは次のことを試すことができます。

items.map(&:amount).inject(0, &:+) 

正確な構文は、あなたのコードに依存します。必ずしもRuby Wayではありませんが、より簡潔で機能的なスタイルです。

Answer 2

ことはあなたの関連性のためにこれを試してみてください：

items.map(&:itemable).sum(&:amount) 

OR

items.map { |i| i.itemable.amount }.sum 

Answer 3

あなたがこれを行うことができ、私は

items = Item.includes(:itemable) 
total = items.map{ |item| item.itemable.amount }.sum 

n + 1クエリの問題を回避するために希望をincludesを使用して積極的なロードしていますそれは助ける！

Answer 4

あなたはリファクタリング中に行うことができますもう一つはありsum

items.map{ |item| item.itemable.amount }.sum 

Answer 5

を利用することができます。アイテムのモデルでは

、あなたは今、あなたは.オペレータなしで量を得ることができ

class Item 
    delegate :amount, to: :itemable, prefix: true, allow_nil: true 
end 

をitemableする金額メソッドを委任することができます。あなたが書くことができますprefix: falseで

items.map{ |item| item.itemable_amount }.sum 

、

items.map{ |item| item.amount }.sum 

Answer 6

あなたは合計をこのように取得し、私はこれがベストプラクティスとパフォーマンスの問題の中で最も効率的であると考えることができます

items.map(:&itemable).pluck(:amount).reduce(:+)