MongoDB中邮箱与手机号重复校验问题排查及代码修复方案

我帮你排查到问题的核心了——你在findOne方法里的查询写法完全错了！当前代码里的第二个参数是投影字段（指定要返回哪些数据），不是查询条件，所以你的代码其实只在检查邮箱是否存在，根本没对手机号做校验。

下面是具体的修改步骤和修正后的代码：

1. 修正注册接口的查询逻辑 #

把路由里的findOne调用改成用$or操作符，同时匹配邮箱或手机号已存在的情况：

router.post("/register", async (req, res) => {
  const { name, email, phone, work, password, cpassword } = req.body;
  if (!name || !email || !phone || !work || !password || !cpassword) {
    return res.status(422).json({ error: "Please fill your details" });
  }
  try {
    // 这里是关键：用$or同时检查邮箱和手机号
    const userExist = await User.findOne({ 
      $or: [
        { email: email }, 
        { phone: phone }
      ] 
    });
    if (userExist) {
      return res
        .status(422)
        .json({ error: "Email or Phone number already exists" });
    }
    const user = new User({ name, email, phone, work, password, cpassword });
    const userRegister = await user.save();
    if (userRegister) {
      res.status(201).json({ message: "User registered successfully" });
    }
  } catch (err) {
    // 新增：处理数据库唯一约束的错误（如果后面加了unique的话）
    if (err.code === 11000) {
      const duplicateField = Object.keys(err.keyPattern)[0];
      return res.status(422).json({ 
        error: `${duplicateField.charAt(0).toUpperCase() + duplicateField.slice(1)} already exists` 
      });
    }
    console.log(err);
    res.status(500).json({ error: "Server error" });
  }
});

2. 给Schema添加数据库层面的唯一约束（可选但推荐） #

虽然代码逻辑能解决问题，但在数据库层面添加唯一约束能提供双重保障，防止因为代码漏洞导致重复数据。修改你的UserSchema：

const mongoose = require('mongoose');
const userSchema = new mongoose.Schema({
  name: {
    type: String,
    required: true,
  },
  email: {
    type: String,
    required: true,
    unique: true, // 新增唯一约束
    trim: true, // 可选：自动去除前后空格
    lowercase: true // 可选：统一转小写，避免大小写重复（比如test@a.com和TEST@a.com）
  },
  phone: {
    type: Number,
    required: true,
    unique: true // 新增唯一约束
  },
  work: {
    type: String,
    required: true,
  },
  password: {
    type: String,
    required: true,
  },
  cpassword: {
    type: String,
    required: true,
  }
})

// 注意：如果之前已经有数据，添加unique约束后需要先确保现有数据没有重复，否则mongoose启动会报错
const User = mongoose.model('USER', userSchema);
module.exports = User;

为什么原来的代码无效？ #

再给你解释下原代码的问题：

// 原错误写法
const userExist = await User.findOne({ email: email }, {phone: phone });

这里的第二个参数{phone: phone}是告诉MongoDB：查询到匹配email的文档后，只返回phone字段，而不是“查询phone等于指定值的文档”。这完全不是你想要的逻辑，所以手机号重复时根本不会被检测到。

用$or操作符后，MongoDB会查询邮箱等于输入值 或者 手机号等于输入值的所有文档，只要存在就返回，这样就能实现双重校验了。

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