Ruby on Rails视图中Helper返回Active Record数组而非计算整数值问题

问题原因 #

Ruby 方法默认返回最后一行执行语句的结果，你写的all_time_gross方法中，当存在销售数据时，分支的最后一行执行语句是sales.each do ... end迭代，而each方法的返回值就是它遍历的原始数组（也就是你看到的Active Record集合），你计算好的累加值没有被作为方法返回值输出，所以视图才会渲染出数组内容。
而gross_income_upload方法执行正常的原因是，它的分支最后一行语句是销售收入的计算赋值操作，返回值就是计算得到的数值，符合预期。

修复方案 #

方案1：修改原有逻辑，主动返回累加值

只需要在each迭代结束后，显式把累加变量作为方法最后一行返回即可：

def all_time_gross
  if Sale.any?
    all_time_gross = 0
    sales = Sale.all
    sales.each do |sale|
      all_time_gross += sale.purchase_count * sale.item_price
    end
    all_time_gross # 新增这一行，返回累加结果
  else
    0
  end
end

方案2：更高效的Rails原生写法（推荐）

不需要把所有销售记录加载到内存遍历计算，可以直接调用Active Record的sum方法让数据库层面完成求和计算，性能更好、代码更简洁：

def all_time_gross
  Sale.sum("purchase_count * item_price")
end

这个写法自动兼容没有销售数据的场景，没有数据时会直接返回0，不需要额外判断。

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