MERN项目基于邮箱设置用户角色遇500错误，求解决方法

错误原因 #

你在创建UserModel实例时误用了MongoDB聚合操作符$cond——Mongoose模型实例的字段需要直接赋值具体值，而非数据库查询/聚合语法，这种写法会导致Mongoose无法解析字段值，最终触发500错误。

快速修复代码 #

把角色判断逻辑放在Node.js层面完成，直接给role字段赋值具体字符串：

app.post("/register", async (req, res) => {
    bcrypt
    .hash(req.body.password, 10)
    .then((hashedPassword) => {
      // 用正则判断邮箱是否含bsi（可根据需求调整正则规则）
      const isAdmin = /bsi/.test(req.body.email);
      const user = new UserModel({
        email: req.body.email,
        password: hashedPassword,
        role: isAdmin ? "Admin" : "User"
      });
      user
        .save()
        .then((result) => {
          res.status(201).send({
            message: "User Created Successfully",
            result,
          });
        })
        .catch((error) => {
          res.status(500).send({
            message: "Error creating user",
            error,
          });
        });
    })
    .catch((e) => {
      res.status(500).send({
        message: "Password was not hashed successfully",
        e,
      });
    });
});

更严谨的正则匹配（可选） #

如果只想匹配@bsi.后缀的邮箱（避免前缀含bsi的普通用户被误判），可以把正则改成：

const isAdmin = /@bsi\./i.test(req.body.email);
// i表示不区分大小写，支持example@BSI.com、test@bsi.org这类邮箱

其他实现逻辑 #

1. 使用Mongoose Pre-Save钩子（解耦逻辑） #

把角色判断逻辑放到用户模型的前置钩子中，所有创建/更新用户的操作都会自动处理角色，无需在路由里重复写逻辑：

// 在UserModel定义文件中添加pre-save钩子
const userSchema = new mongoose.Schema({
  email: { type: String, required: true },
  password: { type: String, required: true },
  role: { type: String, default: "User" }
});

userSchema.pre("save", function(next) {
  // 仅当邮箱字段被修改时才重新判断角色
  if (!this.isModified("email")) return next();
  this.role = /@bsi\./i.test(this.email) ? "Admin" : "User";
  next();
});

const UserModel = mongoose.model("User", userSchema);

对应的路由代码可以简化：

app.post("/register", async (req, res) => {
    bcrypt
    .hash(req.body.password, 10)
    .then((hashedPassword) => {
      const user = new UserModel({
        email: req.body.email,
        password: hashedPassword
      });
      user
        .save()
        .then((result) => {
          res.status(201).send({
            message: "User Created Successfully",
            result,
          });
        })
        .catch((error) => {
          res.status(500).send({
            message: "Error creating user",
            error,
          });
        });
    })
    .catch((e) => {
      res.status(500).send({
        message: "Password was not hashed successfully",
        e,
      });
    });
});

2. 用Async/Await简化异步代码 #

原代码的then/catch嵌套可以换成async/await，让代码更简洁易读：

app.post("/register", async (req, res) => {
  try {
    const hashedPassword = await bcrypt.hash(req.body.password, 10);
    const isAdmin = /@bsi\./i.test(req.body.email);
    const user = new UserModel({
      email: req.body.email,
      password: hashedPassword,
      role: isAdmin ? "Admin" : "User"
    });
    const result = await user.save();
    res.status(201).send({
      message: "User Created Successfully",
      result,
    });
  } catch (error) {
    // 区分密码哈希错误和用户创建错误
    const message = error.message.includes("hash") 
      ? "Password was not hashed successfully" 
      : "Error creating user";
    res.status(500).send({ message, error });
  }
});

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