Node.js用户注册逻辑为何无法将admin字段设为true？

核心问题根源 #

你的代码逻辑存在两个关键错误，导致管理员字段设置失效：

1. 时序错误：在创建新用户之前执行User.findOne({email: process.env.ADMIN_EMAIL})，此时新用户还未存入数据库，查询结果必然是null，根本不会进入更新分支。
2. 实例更新用法错误：即便后续把查询移到用户创建后，你用admin.updateOne({ email: process.env.ADMIN_EMAIL }, { admin: true })的写法也不合理——拿到文档实例后，直接修改属性再保存才是正确做法，且原逻辑会误修改数据库中已存在的同邮箱用户（而非当前注册的新用户）。

修复方案一：创建用户时直接设置管理员字段（推荐） #

这是最高效的方式，无需额外数据库查询：

exports.register_post = asyncHandler(async (req, res, next) => {
    try {
        // 直接判断当前注册邮箱是否为预设管理员邮箱
        const isAdmin = req.body.email === process.env.ADMIN_EMAIL;

        // 创建用户时直接赋值admin字段
        const user = new User({
            firstName: req.body.firstName,
            lastName: req.body.lastName,
            email: req.body.email,
            password: hashedPassword,
            admin: isAdmin // 无需后续更新，一步到位
        });

        await user.save();
        res.redirect("/");
    } catch (err) {
        next(err);
    }
});

修复方案二：用户保存后再更新管理员字段（适合特殊场景） #

如果业务逻辑要求必须在用户创建后再执行更新，可采用以下写法：

exports.register_post = asyncHandler(async (req, res, next) => {
    try {
        // 创建并保存新用户
        const user = new User({
            firstName: req.body.firstName,
            lastName: req.body.lastName,
            email: req.body.email,
            password: hashedPassword,
        });

        await user.save();

        // 仅针对当前注册的用户判断并更新
        if (user.email === process.env.ADMIN_EMAIL) {
            try {
                // 方式1：修改实例属性后保存（更直观）
                user.admin = true;
                await user.save();

                // 方式2：使用Model的updateOne方法
                // await User.updateOne({ _id: user._id }, { admin: true });
            } catch (err) {
                console.error('Error updating admin status:', err);
                return res.status(500).send('Error updating admin status');
            }
        }

        res.redirect("/");
    } catch (err) {
        next(err);
    }
});

额外说明 #

原代码中错误处理未触发的原因很简单：因为findOne返回null，根本不会进入if (admin)分支，自然不会执行内部的try/catch逻辑。

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