计算销售总额返回null/NaN的问题排查求助

我定义了Mongoose的Sale模型，包含数字类型的total字段，想要累加所有销售记录的total计算总销售额。之前用遍历累加的方法正常运行，但现在接口返回的total为null，控制台输出NaN，且确认所有销售记录的total都是数字类型，也没修改过代码，想知道问题原因。

Sale模型代码 #

const mongoose = require('mongoose');

const SaleSchema = new mongoose.Schema({
  tickets: [
    {
      type: mongoose.Schema.Types.ObjectId,
      ref: 'Ticket',
      required: true,
    },
  ],
  total: {
    type: Number,
    required: true,
  },
  createdAt: {
    type: Date,
    default: Date.now,
  },
});

const Sale = mongoose.model('Sale', SaleSchema);

module.exports = Sale;

销售额计算接口代码 #

//@desc      Get the total amount of all sales
//@method    GET
//@access    Private

exports.getTotalSales = asyncHandler(async (req, res, next) => {
  const sales = await Sale.find();

  if (!sales) {
    return next(new ErrorResponse(`No sales found`));
  }

  let total = 0;
  sales.forEach((sale) => {
    total += sale.total;
  });

  console.log(total);

  res.status(200).json({
    success: true,
    total,
  });
});

当前接口返回 #

{
  "success": true,
  "total": null
}

控制台输出total为NaN。

可能的原因分析 #

• 数据库存在未校验的脏数据：虽然Schema中定义了total为必填数字类型，但如果是在Schema添加约束前插入的记录，或是通过MongoDB客户端直接写入（绕开Mongoose校验），可能存在total为null、undefined或非数字的情况。Mongoose不会自动对已有数据做强制校验，这些脏数据会导致0 + null/undefined直接变成NaN。

• Mongoose文档字段序列化异常：Mongoose返回的是Document对象，sale.total可能是被包装的Mongoose Number实例，而非原始JS数字类型。在某些边界场景下，这种包装类型可能导致累加时出现类型转换错误，最终得到NaN。

• 空数组判断逻辑无效：Sale.find()即使没有数据也会返回空数组，而非null，所以代码中if (!sales)的判断永远不会触发。如果数据库中存在异常记录，这个逻辑无法拦截，会继续执行累加导致NaN。

• 中间件或查询操作干扰：asyncHandler或其他请求中间件可能意外修改了sales数据；或者查询时的populate等操作，导致total字段被覆盖或丢失（虽然概率较低，但需排查）。

验证与临时修复方法 #

1. 排查数据库脏数据：在MongoDB Shell中执行以下命令，检查是否存在异常记录：

# 查找total为null的记录
db.sales.find({ total: { $type: 10 } })
# 查找total不存在的记录
db.sales.find({ total: { $exists: false } })

2. 修改累加逻辑增加类型校验：

let total = 0;
sales.forEach((sale) => {
  // 确保只累加有效数字
  const validTotal = typeof sale.total === 'number' && !isNaN(sale.total) ? sale.total : 0;
  total += validTotal;
});

3. 改用Mongoose聚合查询（更可靠）：聚合查询直接在数据库层面计算，避免客户端遍历的潜在问题：

const aggResult = await Sale.aggregate([
  { $group: { _id: null, totalSales: { $sum: '$total' } } }
]);
const total = aggResult.length > 0 ? aggResult[0].totalSales : 0;

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