Rails模型中获取过去10周销售数据的最佳实践咨询

问题背景 #

你需要复用一段获取过去10周销售数据的代码，考虑将其放入Sales模型，但不确定是否符合Rails最佳实践，同时觉得当前循环查询的实现不够优雅。

当前实现 #

def past_10_sales(yw)
  past_sales = []
  year_week = yw

  10.times do
    sales = Sales.where(year_week: year_week.previous)

    past_sales << sales unless sales.empty?
    year_week = year_week.previous
  end
  past_sales.flatten
end

补充信息 #

• YearWeek模型以年份加周数作为ID（如202244），调用year_week.previous可获取上一周的实例。
• Sales模型通过year_week_id关联YearWeek，表结构如下：

create_table "sales" do |t|
  t.datetime "created_at", null: false
  t.datetime "updated_at", null: false
  t.bigint "year_week_id"
  # 其他字段...
end

最佳实践判断 #

把这个方法放到Sales模型里完全符合Rails最佳实践。Rails推崇"胖模型、瘦控制器"，数据查询和业务逻辑就应该封装在模型中，这样既能提高代码复用性，也能让控制器更专注于请求处理和响应返回。你担心的"在模型中调用Sales.where"是完全合理的，模型本身就是负责封装数据操作的载体。

更优雅的优化实现 #

当前循环查询的问题在于会发起10次独立的数据库请求，效率较低。我们可以先一次性获取过去10周的YearWeek实例ID，再通过一次查询拿到所有对应销售数据，大幅减少数据库交互次数。

方案1：封装YearWeek的周数获取逻辑 #

先在YearWeek模型中添加一个获取指定周数前n周的方法：

# app/models/year_week.rb
class YearWeek < ApplicationRecord
  def self.past_n_weeks(start_week, count = 10)
    weeks = []
    current = start_week
    count.times do
      current = current.previous
      weeks << current if current.present? # 避免边界情况（如已无更早周）
    end
    weeks
  end
end

然后在Sales模型中实现查询方法：

# app/models/sales.rb
class Sales < ApplicationRecord
  belongs_to :year_week

  def self.past_10_weeks(start_year_week)
    target_weeks = YearWeek.past_n_weeks(start_year_week)
    where(year_week_id: target_weeks.pluck(:id))
  end
end

方案2：直接在Sales模型中实现（更简洁） #

如果不需要复用周数获取逻辑，也可以直接在Sales模型中完成：

# app/models/sales.rb
class Sales < ApplicationRecord
  belongs_to :year_week

  def self.past_10_weeks(start_year_week)
    week_ids = []
    current_week = start_year_week
    10.times do
      current_week = current_week.previous
      week_ids << current_week.id if current_week.present?
    end
    where(year_week_id: week_ids)
  end
end

优化后的优势 #

1. 减少数据库请求：从10次查询变为1次，大幅提升性能。
2. 返回ActiveRecord::Relation：结果保留Relation特性，可以继续链式调用（如排序、筛选），比原方法返回的数组更灵活。
3. 自动过滤空周：原方法中unless sales.empty?的逻辑可以省略，因为where只会返回有对应销售记录的数据。

如果需要和原方法一样返回数组，只需在方法末尾添加.to_a即可。

内容的提问来源于stack exchange，提问作者marcHoll90