在初始化期间调用实例方法是不是很糟糕？

Question

我正在考虑如何改进以下课程，我想到了：在初始化期间调用实例方法是不是很糟糕？

最初，我将方法 set_last_updated_on 作为“run”方法中的第一行，并且很可能我会将其保留在那里，但问题仍然如上所述。

class FinancialUpdateService
  def initialize(stock)
    @stock = stock
    set_last_updated_on
  end

  def run
    fetch_latest_financial_data
    remove_duplicate_data
    store_latest_financial_data  
  end

  private

  def set_last_updated_on
    @last_updated_on = @stock.financials.first.date.to_date
  end

  def fetch_latest_financial_data
    @latest_financial_data = StockDataApi.new(@stock.symbol, {start_date: @last_updated_on, end_date: Date.today-1}).financial_history
  end

  def remove_duplicate_data
    @latest_financial_data.delete_if { |data| data[:date].to_date <= @last_updated_on}
  end

  def store_latest_financial_data
    @latest_financial_data.each do |d| 
      @stock.financials.create(adj_close: d.fetch(:adj_close), close: d.fetch(:close),
      date: d.fetch(:date).to_time, high: d.fetch(:high), low: d.fetch(:low), open: d.fetch(:open),
      volume: d.fetch(:volume))
    end
  end
end

Answer 1

这不是类方法，而是实例方法。

不，这不是糟糕的形式。在 内部调用方法是完全正常且可取的initialize。