初始化中計算實例變量的最簡潔方法是什麼？

Question

假設我有一個類定義了我的日子集合以及它們的古怪程度。它是更好地初始化我@scores在初始化函數變量，像這樣：

class WackyDayScorer 
    attr_reader :scores 
    def initialize(start = Time.now, max = 90.days) 
    @start = start 
    @limit = start - max 
    @scores = get_scores 
    end 

private 
    def get_scores 
    result = [] 
    t = @start.clone 
    until t < max 
     result << score_wackiness(t) 
     t -= 1.day 
    end 
    result 
    end 
end 

或初始化它在get_scores方法，像這樣：

class WackyDayScorer 
    attr_reader :scores 

    def initialize(start = Time.now, max = 90.days) 
    @start = start 
    @limit = start - max 
    get_scores 
    end 

private 
    def get_scores 
    @scores = [] 
    t = @start.clone 
    until t < max 
     @scores << score_wackiness(t) 
     t -= 1.day 
    end 
    end 
end 

Answer 1

第一種是完全錯誤的。你是說

@scores = get_scores 

但get_scores沒有返回一個有用的值，所以這是堅果。相反，get_scores設置@scores直接和內部作爲副作用。

第二個至少是連貫的。

就個人而言，雖然，我不會做的事情你做要麼。我會說，你說什麼第一次：

@scores = get_scores 

但是我執行get_scores不願觸及的伊娃。如果你想處理在初始化的東西，創建來處理計算一個私有方法，然後設置在#initialize實例變量

def get_scores 
    scores = [] 
    # do stuff to scores 
    return scores # or simply, scores 
    end 

Answer 2

：這將返回的實際值。如果你想在不刪除實例變量的情況下在類的其他地方重新使用該計算，我不會在方法中設置它。

class WackyDayScorer 
    attr_accessor :scores 

    def initialize(start = DateTime.now, max = 90.days) 
    @start = start 
    @limit = start - max 
    @scores = calculate_scores 
    end 

private 

    def calculate_scores 
    (@limit..@start).to_a.map { |date_time| score_wackiness(date_time) } 
    end 
end 

在#calculate_scores，我們正在創建的限定日期和開始之間的範圍（倒數，當你在做），每個結果發送到#score_wackiness方法。

Answer 3

我建議你明確地定義爲得分，吸氣，memoizing計算並跳過所有的儀式：

class WackyDayScorer 
    def initialize(start = Time.now, max = 90.days) 
    @start = start 
    @limit = start - max 
    end 

    def scores 
    @scores ||= begin 
     result = [] 
     t = @start.clone 
     until t < max 
     result << score_wackiness(t) 
     t -= 1.day 
     end 
     result 
    end 
    end 
end 

這基本上是你在做什麼，無論如何，但它更直接。