如何在Rails模型中更新實例變量？

Question

在我的Rails應用程序中，我有users誰可以有很多payments。

class User < ActiveRecord::Base 

    has_many :invoices 
    has_many :payments 

    def year_ranges 
    ... 
    end 

    def quarter_ranges 
    ... 
    end 

    def month_ranges 
    ... 
    end 

    def revenue_between(range, kind) 
    payments.sum_within_range(range, kind) 
    end 

end 

---

class Invoice < ActiveRecord::Base 

    belongs_to :user 
    has_many :items 
    has_many :payments 

    ... 

end 

---

class Payment < ActiveRecord::Base 

    belongs_to :user 
    belongs_to :invoice 

    def net_amount 
    invoice.subtotal * percent_of_invoice_total/100 
    end 

    def taxable_amount 
    invoice.total_tax * percent_of_invoice_total/100 
    end 

    def gross_amount 
    invoice.total * percent_of_invoice_total/100 
    end 

    def self.chart_data(ranges, unit) 
    ranges.map do |r| { 
     :range   => range_label(r, unit), 
     :gross_revenue => sum_within_range(r, :gross), 
     :taxable_revenue => sum_within_range(r, :taxable), 
     :net_revenue  => sum_within_range(r, :net) } 
    end 
    end 

    def self.sum_within_range(range, kind) 
    @sum ||= includes(:invoice => :items) 
    @sum.select { |x| range.cover? x.date }.sum(&:"#{kind}_amount") 
    end 

end 

在我dashboard鑑於我取決於用戶選擇了GET參數列出了ranges總付款。用戶可以選擇years,quarters或months。

class DashboardController < ApplicationController 

    def show 
    if %w[year quarter month].include?(params[:by]) 
     @unit = params[:by] 
    else 
     @unit = 'year' 
    end 
    @ranges = @user.send("#{@unit}_ranges") 
    @paginated_ranges = @ranges.paginate(:page => params[:page], :per_page => 10) 
    @title = "All your payments" 
    end 

end 

的使用實例變量（@sum）大大降低SQL的數量在這裏查詢，因爲該數據庫將不會越過命中爲相同的查詢一遍又一遍。

但問題是，當用戶創建，刪除或更改他的某個payments時，這不會反映在@sum實例變量中。那麼我該如何重置它？還是有更好的解決方案呢？

感謝您的任何幫助。

Answer 1

也許你可以觀察員做到這一點：

# payment.rb 

def self.cached_sum(force=false) 
    if @sum.blank? || force 
    @sum = includes(:invoice => :items) 
    end 
    @sum 
end 

def self.sum_within_range(range) 
    @sum = cached_sum 
    @sum.select { |x| range.cover? x.date }.sum(&total) 
end 

#payment_observer.rb 

class PaymentObserver < ActiveRecord::Observer 
    # force @sum updating 

    def after_save(comment) 
    Payment.cached_sum(true) 
    end 

    def after_destroy(comment) 
    Payment.cached_sum(true) 
    end 

end 

你可以找到更多關於觀察員http://apidock.com/rails/v3.2.13/ActiveRecord/Observer

Answer 2

嗯，你@sum基本上是你所需要的值的緩存。像任何緩存一樣，如果所涉及的值發生變化，您需要使其無效。

您可以使用after_save或after_create過濾器調用設置爲@sum = nil的函數。還可以保存緩存覆蓋的範圍，並根據新的或更改的付款日期確定失效情況。

class Payment < ActiveRecord::Base 

    belongs_to :user 

    after_save :invalidate_cache 

    def self.sum_within_range(range) 
    @cached_range = range 
    @sum ||= includes(:invoice => :items) 
    @sum.select { |x| range.cover? x.date }.sum(&total) 
    end 

    def self.invalidate_cache 
    @sum = nil if @cached_range.includes?(payment_date) 
end 

Answer 3

而不是存儲的關聯作爲類Payment的一個實例變量，將其存儲爲一個user的實例變量的（我知道這聽起來混亂，我試圖下面解釋）

class User < ActiveRecord::Base 

    has_many :payments 

    def revenue_between(range) 
    @payments_with_invoices ||= payments.includes(:invoice => :items).all 
    # @payments_with_invoices is an array now so cannot use Payment's class method on it 
    @payments_with_invoices.select { |x| range.cover? x.date }.sum(&:total) 
    end 

end 

當您在類方法中定義@sum（類方法由self.表示）時，它成爲類Payment的實例變量。這意味着您可以以Payment.sum的身份訪問它。所以這與特定用戶和他/她的付款無關。 @sum現在是類Payment的一個屬性，Rails將以與緩存類的方法定義相同的方式對其進行緩存。

一旦@sum被初始化，它將保持不變，就像您注意到的那樣，即使在用戶創建新付款後或者其他用戶登錄該事件！它會在應用程序重新啓動時更改。

但是，如果您像上面顯示的那樣定義@payments_with_invoices，它將成爲User的特定實例的屬性，換句話說就是實例級別的實例變量。這意味着您可以將其作爲some_user.payments_with_invoices進行訪問。由於一個應用程序可以有很多用戶，所以這些用戶不會跨越請求持久存儲在Rails內存中所以無論何時用戶實例更改其屬性再次加載。

所以如果用戶創建更多付款，@payments_with_invoices變量將被刷新，因爲用戶實例被重新初始化。

Answer 4

這是附帶的問題，但不要使用#select和塊。

你在做什麼是選擇所有付款，然後過濾關係作爲一個數組。使用Arel來解決這個問題：

scope :within_range, ->(range){ where date: range } 

這將構建一個SQL BETWEEN語句。在生成的關係上使用#sum將構建一條SQL SUM（）語句，該語句可能比加載所有記錄更有效。