Express JS配合Mongoose查询 中间件排序与limit未生效问题

你的代码有4个核心逻辑错误，导致预设的排序、条数限制不生效：

• 提前执行了数据库查询：过滤阶段的Item.find()前错误添加了await，这一步就已经把查询发到数据库返回了结果数组，后续追加的排序、字段筛选逻辑都是基于新生成的查询对象执行，之前的配置完全丢失。
• 查询链式调用断裂：排序、字段筛选、分页阶段你每次都重新调用Item.find()生成全新的查询对象，没有基于上一步的Query实例追加条件，后面的逻辑会直接覆盖前面的配置。
• limit触发逻辑错误：你把条数限制limit的逻辑写在了if (req.query.page)判断块内，只有传入page参数时才会应用limit，你的别名路由只设置了limit没传page，所以3条的限制根本不会执行。
• 路由API笔误：你用了express.Route()创建路由，正确的API是express.Router()，否则路由实例本身就无法正常工作。

1. 修正路由创建API #

// 把express.Route()改成express.Router()
express.Router().route('/top-5-menu').get(aliasTopMenu, getAllItems);

你的aliasTopMenu中间件逻辑没有问题，不需要修改。

2. 重写getAllItems的查询拼接逻辑 #

核心原则是：先全程基于同一个Mongoose Query对象拼接所有查询条件，所有条件拼接完成后再统一await执行查询，不要中途重新生成查询对象，也不要提前加await。

const getAllItems = AsyncWrapper(async (req, res) => {
  //1) 基础过滤
  const queryObj = { ...req.query };
  const excludField = ['page', 'sort', 'limit', 'fields'];
  excludField.forEach((el) => delete queryObj[el]);

  // 高级比较符过滤
  let queryStr = JSON.stringify(queryObj);
  queryStr = queryStr.replace(/(gt|gte|lte|lt)/g, (matched) => `$${matched}`);
  // 注意：这里不要加await，保留可链式调用的Query对象
  let query = Item.find(JSON.parse(queryStr));

  //2) 排序：基于已有query追加，不要重新调用Item.find()
  if (req.query.sort) {
    const sortBy = req.query.sort.split(',').join(' ');
    query = query.sort(sortBy);
  }

  //3) 字段限制：基于已有query追加
  if (req.query.fields) {
    const fields = req.query.fields.split(',').join(' ');
    query = query.select(fields);
  }

  //4) 分页与条数限制：不管有没有page参数，都应用limit配置
  const page = req.query.page * 1 || 1;
  const limit = req.query.limit * 1 || 100;
  const skip = (page - 1) * limit;
  query = query.skip(skip).limit(limit);

  // 只有传了page参数时才校验页码合法性
  if (req.query.page) {
    const numItems = await Item.countDocuments();
    if (skip >= numItems) {
      throw new Error('this page do not exist');
    }
  }

  // 所有条件拼接完成，最后统一执行查询
  const items = await query;
  res.status(200).json(items);
});

访问/top-5-menu接口时，会自动应用你预设的参数：

• 按价格降序排序
• 只返回name、price、rating三个字段
• 最多返回3条数据
  不会再出现返回全量数据、排序不生效的问题。

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