Mongoose基于ObjectId查询关联记录无法获取分组下文档的问题

• 核心错误是Group.find()的返回值类型判断错误：find()方法返回的是所有匹配查询条件的文档组成的数组，从你控制台打印的结果也能看到，返回值被[]包裹，是长度为1的对象数组，不是单个分组文档实例。你直接在数组上访问.id属性会得到undefined，最终执行Document查询时传入的查询条件是{group: undefined}，自然匹配不到任何文档，返回空数组。
• 潜在逻辑漏洞：现有写法只能处理单个分组的场景，实际业务中一个用户可能被添加为多个分组的查看者，只取单个分组id会漏掉其他分组下的共享文档。
• 额外细节注意：Mongoose文档默认主键字段为_id，虽然单个文档实例可以通过.id getter获取字符串格式的主键值，但该属性仅存在于单个文档实例上，数组对象本身不存在该属性。

方案1：修正分步查询逻辑（改动最小，适配现有代码） #

先提取返回分组数组中所有的分组ID，再通过$in操作符查询所有归属这些分组的文档，兼容单分组、多分组场景：

router.get("/shared-documents", async (req, res) => {
  try {
    // 查询当前用户作为查看者的全部分组
    const groups = await Group.find({
      viewer: req.session.userId
    })
    // 提取所有分组的_id组成查询数组
    const groupIds = groups.map(group => group._id);
    // 匹配所有关联分组在权限范围内的文档
    const documents = await Document.find({
      group: { $in: groupIds }
    });

    return res.render("documents/index", { documents });
  } catch (error) {
    // 原有错误处理逻辑
  }
});

如果业务确定一个用户最多只会被加入一个分组的查看者列表，可以把Group.find()替换为Group.findOne()，该方法会直接返回单个匹配的文档实例，可直接访问主键：

// 单分组场景简化写法
const group = await Group.findOne({
  viewer: req.session.userId
})
// 增加空值判断，避免用户无任何分组权限时抛出错误
const documents = group ? await Document.find({ group: group._id }) : [];

方案2：Mongoose关联查询（性能更优，减少数据库请求次数） #

可以直接从Document侧做关联匹配，一次查询完成权限过滤，不需要分两次手动关联ID：

router.get("/shared-documents", async (req, res) => {
  try {
    const documents = await Document.find()
      .populate({
        path: 'group',
        // 关联分组时仅匹配当前用户在查看者列表的分组
        match: { viewer: req.session.userId }
      })
      // 过滤掉关联分组为空（即无访问权限）的文档
      .then(list => list.filter(doc => doc.group !== null));

    return res.render("documents/index", { documents });
  } catch (error) {
    // 原有错误处理逻辑
  }
});

如果要从Group侧直接获取关联文档，可以在GroupSchema中添加虚拟字段映射关联关系：

GroupSchema.virtual('documents', {
  ref: 'Document',
  localField: '_id',
  foreignField: 'group'
});
// 开启虚拟字段导出，否则populate时无法拿到对应值
GroupSchema.set('toObject', { virtuals: true });
GroupSchema.set('toJSON', { virtuals: true });

添加后即可直接在查询Group时关联查出对应文档：

const groups = await Group.find({ viewer: req.session.userId }).populate('documents');
// 把所有分组下的文档拍平为一个数组即可
const documents = groups.flatMap(group => group.documents);

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