健身房管理项目：通过用户邮箱查找会员资格验证到场问题求助

问题定位 #

你的代码存在几个核心问题导致查询会员资格返回空：

• 未正确获取用户ID：User.find()返回的是数组，你直接使用未定义的userId变量，导致后续查询条件错误
• 复用错误的查询对象：查询会员时沿用了带有email字段的query，但Membership模型没有email字段，导致查询条件无效
• 未添加有效会员的验证逻辑：没有过滤isActive: true且expiresAt未过期的会员
• 异常处理缺失：没有处理用户不存在的情况，也未统一异步代码风格

修复后的代码 #

exports.findMembership = async (req, res) => {
    try {
        // 从请求体获取邮箱
        const { email } = req.body;
        if (!email) {
            return res.status(400).json({ error: "邮箱不能为空" });
        }

        // 通过邮箱查找单个用户（邮箱唯一，用findOne而非find）
        const user = await User.findOne({ email }).select("_id");
        if (!user) {
            return res.status(404).json({ error: "未找到该用户" });
        }

        // 查询该用户的有效会员资格：isActive为true，且过期时间大于当前时间
        const validMembership = await Membership.findOne({
            user: user._id,
            isActive: true,
            expiresAt: { $gt: new Date() }
        }).populate('user');

        if (validMembership) {
            return res.json({
                status: "success",
                message: "会员有效，允许入场",
                membership: validMembership
            });
        } else {
            return res.status(403).json({
                status: "fail",
                message: "无有效会员资格，无法入场"
            });
        }
    } catch (err) {
        return res.status(500).json({ error: err.message });
    }
};

关键修改说明 #

• 使用User.findOne()替代find()：因邮箱是唯一字段，直接获取单个用户对象，无需处理数组
• 独立会员查询条件：创建全新查询对象，仅保留user、isActive和expiresAt相关条件，避免携带无关的email字段
• 添加有效会员验证：通过isActive: true和expiresAt: { $gt: new Date() }过滤出当前有效的会员
• 统一使用async/await：替换回调函数，让代码逻辑更清晰，同时添加完整的异常捕获
• 明确返回状态：给前端返回清晰的状态标识和提示信息，方便页面后续处理

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