在 Node.js 下处理 MongoDB 查询结果时,不少开发者都会踩到一个经典“陷阱”:明明用 map() 遍历了一遍,重复项却依然存在?输出一看,竟然还多出不少 undefined。别担心,这背后其实是一个常见的逻辑误解——我们来拆解一下你的代码到底哪里出了问题,以及如何用一行代码轻松解决。
先看这段有问题的写法:
let resultArray = []; // 初始为空 []
allRecipes.map(recipe => {
const tag = recipe.tag;
if (resultArray.includes(tag)) { // ❌ 始终为 false —— resultArray 还没被赋值!
// 跳过
} else {
return tag; // ✅ 每次都执行,但 map 不会“累积”到 resultArray
}
});
核心问题其实就一句话:map() 并不会修改原数组,而且你在执行 map 的过程中,反复检查的 resultArray 从头到尾都是空的。因此 resultArray.includes(tag) 永远返回 false,每个标签都被返回了,最终得到一个包含重复项的稀疏数组(还带着 undefined)。更重要的是,你压根没把 map 返回的新数组赋值给 resultArray——所以 console.log(resultArray) 打印出来的其实是 map 返回的原始数组,里面的重复项一个都没少。
这才是你看到 [ Breakfast, Lunch, Lunch, Breakfast ] 的根本原因。
那么,正确的做法是什么?很简单:别在 map 里做状态判断,换用 ES6 的 Set。Set 天生保证元素唯一性,插入和查询都是 O(1),比反复调用 includes()(O(n))高效得多,代码也更为清晰:
const getUniqueTags = async (req, res) => {
try {
const allRecipes = await Recipe.find({}).select("tag");
// ✅ 一步提取所有 tag 并自动去重
const uniqueTags = [...new Set(allRecipes.map(recipe => recipe.tag))];
console.log(uniqueTags); // 示例输出: [ 'Breakfast', 'Lunch', 'Dinner' ]
res.status(200).json(uniqueTags);
} catch (error) {
console.error(error);
res.status(500).json({ error: "Failed to fetch unique tags" });
}
};
这里的关键步骤:
allRecipes.map(r => r.tag)先提取所有标签(可能重复);new Set(...)自动去重;[...set]或Array.from(set)把 Set 转回数组,方便 HTTP 响应和前端使用。
还有几个小细节值得注意:
- 如果
recipe.tag可能为null或undefined,Set 会保留这些值。更健壮的做法是先过滤:allRecipes.map(r => r.tag).filter(tag => typeof tag === 'string' && tag.trim() !== ''),再丢进 Set。 - Set 默认按插入顺序迭代,通常能满足需求。如需排序,可在最后加个
.sort()。 - 千万别在循环里用
push+includes手动去重——不仅逻辑容易出错,时间复杂度还会飙升到 O(n²)。
说到底,处理唯一性需求,Set 就是最简洁、最可靠的标准实践。既避免了手动状态管理的陷阱,也显著提升了性能。下次遇到类似场景,直接上 Set 就对了。
