在紅寶石中添加塊

Question

我發佈了一個similar question這個不久之前，關於格式化MySQL查詢使用塊，並得到了非常好的迴應，但他們是非常具體的問題。這一次，我正在處理在表格中獲取行的.sum()。這裏是我現在得到的：

def balance 
    balance = 0 
    items.each do |item| 
    balance = balance + item.charges.sum(:revenue, :conditions => ['created_at >= ?', Time.now.beginning_of_month]) 
    end 
    balance 
end 

我的目標是獲取給定用戶的本月所有費用的總和。費用屬於屬於用戶的項目。我相信在Ruby/Rails中有更好的方法來做到這一點。

你會怎麼做？

Answer 1

直轉換可以做到：

def balance 
    conds = ["created_at > ?", Time.now.beginning_of_month] 
    items.inject(0) do |total, item| 
    total + item.charges.sum(:revenue, :conditions => conds) 
    end 
end 

有可能是取決於你的關係是如何映射出更優化的方法。舉例來說，你可以做一些事情，如：

def balance 
    Charge.sum :revenue, 
    :conditions => ["charges.item_id IN (?) AND created_at > ?", 
     items.map { |item| item.id }, 
     Time.now.beginning_of_month] 
end 

對於這類情況，map，inject，select，等等，是非常寶貴的工具。 Here's a lengthy discussion on inject，有關詳細信息，請參閱RDoc's for the Enumerable模塊。

Answer 2

沒有理由不這樣做在SQL查詢本身，e.g：

Charges.sum :revenue, :conditions => [ "created_at >= ?, items.user_id = ?", 
             Time.now.beginning_of_month, some_user_id ], 
         :joins => :items 

編輯：這是從文檔sum是否會採取一個符號不清楚：加入像find一樣。如果沒有，你的：加入行應該是這樣，而不是：

:joins => "JOIN items ON charges.item_id = items.id" 

Answer 3

只是一對夫婦的一般性評論：

• 我建議把你的代碼模型，而不是一個控制器，查看或助手，遵循Skinny Controller, Fat Model的想法。
• 而不是寫balance = balance + item.charges.sum(:revenue, :conditions => ['created_at >= ?', Time.now.beginning_of_month])你可以使用+=和做：balance += item.charges.sum(:revenue, :conditions => ['created_at >= ?', Time.now.beginning_of_month])。
• 這在技術上並不是一個問題，但它有一個與您的方法同名的變量（即「平衡」）似乎很糟糕。
• 我會將Time.now.beginning_of_month存儲在循環外的變量中，因此每次都不會重新計算。

Answer 4

這是一個非常常見的遞歸模式。它被稱爲catamorphsim in category theory，a fold in mathematics and functional programming，它有時也被稱爲減少而在Smalltalk中它被稱爲inject:into:。在Ruby中，它被稱爲inject或reduce（這兩個方法都是別名）。

這個想法是，你有一個值的集合，你想「減少」或「摺疊」多個值的集合成一個單一的值。 （Smalltalk名稱inject:into:來自這樣一個事實，即您將初始值插入到爲集合中的每個元素調用的塊中。）

def balance 
    this_month = Time.now.beginning_of_month 
    items.reduce(0) { |balance, item| 
    balance + item.charges.sum(:revenue, :conditions => ['created_at >= ?', this_month]) 
    } 
end