在初始化中计算实例变量的最简洁方法是什么？

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 的实现不会触及任何ivar。它会返回一个实际值：

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

Answer 2

如果要在初始化中处理某些事情，请创建一个私有方法来处理计算，然后在#initialize 中设置实例变量。我不会在方法中设置它，以防你想在类的其他地方重用该计算而不清除你的实例变量。

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

我建议明确定义分数的 getter，记住计算并跳过所有仪式：

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

这基本上就是你正在做的事情，但它更直接。