導軌;總計

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

你可以做到這一點，我渴望加載使用includes避免n + 1查詢問題

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

希望這有幫助！

Answer 4

您可以使用sum

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

Answer 5

有一個重構的同時，你可以做更多的事情。

在項目模型，你可以委託量的方法來itemable

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

現在你可以不用量操作.。

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

隨着prefix: false你可以寫，

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

Answer 6

你可以得到全額這種方式，我覺得這是最好的實踐和最高效的性能無論

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