Sequelize中offset值为NaN的数据库错误排查求助

你遇到的offset为NaN的问题，核心原因是parseInt(req.params.page)可能返回NaN。当req.params.page存在但不是有效数字（比如含非数字字符的字符串、空值）时，parseInt无法解析出合法整数，此时page的值变为NaN，计算limit * (page - 1)自然得到NaN，而Sequelize无法识别NaN作为offset参数，因此抛出数据库错误。

1. 严格校验分页参数 #

转换page参数时增加有效性校验，确保结果是合法正整数，无效时直接使用默认值1。

2. 限制分页范围 #

计算总页数后，强制page不超过总页数，避免offset超出数据总量。

3. 优化查询（可选） #

原代码两次查询Product表，可合并为一次findAndCountAll调用，减少数据库请求。

exports.getProducts = (req, res, next) => {
  const limit = 10; // 每页10个商品
  let offset = 0;

  Product.findAndCountAll()
    .then((data) => {
      // 处理page参数：确保是有效正整数，否则用默认值1
      let page = parseInt(req.params.page);
      if (!Number.isInteger(page) || page < 1) {
        page = 1;
      }

      const pages = Math.ceil(data.count / limit);
      // 限制page不超过总页数
      page = Math.min(page, pages);
      offset = limit * (page - 1);
      console.log(offset); // 现在应为合法数字
      console.log(typeof offset);

      return Product.findAll({
        limit: limit,
        offset: offset,
        include: [{
          model: Order,
          as: 'orders'
        }, {
          model: User,
          as: 'user'
        }]
      });
    })
    // 补充响应和错误处理
    .then(products => {
      res.status(200).json({ products });
    })
    .catch(err => {
      next(err);
    });
};

直接用findAndCountAll同时获取数据和总数，减少一次数据库调用：

exports.getProducts = (req, res, next) => {
  const limit = 10;
  let page = parseInt(req.params.page);
  if (!Number.isInteger(page) || page < 1) {
    page = 1;
  }

  Product.findAndCountAll({
    limit: limit,
    offset: limit * (page - 1),
    include: [{
      model: Order,
      as: 'orders'
    }, {
      model: User,
      as: 'user'
    }]
  })
  .then(result => {
    const { count: total, rows: products } = result;
    const pages = Math.ceil(total / limit);
    // 再次校验page，避免总数据量变化导致的越界
    page = Math.min(page, pages);
    res.status(200).json({ products, total, pages, currentPage: page });
  })
  .catch(err => {
    next(err);
  });
};

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