如何简化totalMonth函数？求更优雅的JavaScript实现方案

你的代码功能正常但存在大量重复逻辑，这里提供两种更简洁高效的实现方式，核心思路是避免重复声明变量，通过一次遍历完成月份总和的计算：

方式一：清晰易懂的分步实现 #

totalMonth() {
  // 统一维护月份属性名的顺序数组
  const monthKeys = ['jan', 'feb', 'mar', 'apr', 'may', 'jun', 'jul', 'aug', 'sep', 'oct', 'nov', 'dec'];
  // 初始化12个月份的总和为0
  const monthSums = monthKeys.map(() => 0);

  // 遍历每个数据项，直接累加对应月份的值
  this.overview.forEach(subkpi => {
    monthKeys.forEach((key, index) => {
      // 用||0处理可能的undefined值，避免总和变成NaN
      monthSums[index] += subkpi[key] || 0;
    });
  });

  return monthSums;
}

方式二：更紧凑的reduce写法 #

如果喜欢更简洁的代码，可以用reduce合并初始化与遍历逻辑：

totalMonth() {
  const monthKeys = ['jan', 'feb', 'mar', 'apr', 'may', 'jun', 'jul', 'aug', 'sep', 'oct', 'nov', 'dec'];
  
  return this.overview.reduce((sums, subkpi) => {
    monthKeys.forEach((key, i) => sums[i] += subkpi[key] || 0);
    return sums;
  }, monthKeys.map(() => 0));
}

优化点说明 #

• 用monthKeys集中管理月份属性名，避免硬编码重复，后续修改月份顺序或属性名只需修改这一处
• 直接在遍历数据时完成累加，省去了创建12个中间数组再求和的步骤，减少内存占用和计算步骤
• 添加||0处理，防止某个月份数据缺失（值为undefined）时导致总和变为NaN，提升代码鲁棒性

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