使用Express+Sequelize创建用户时bcrypt哈希密码的重复邮箱异常问题

你的问题核心是邮箱唯一性校验后未正确终止请求流程：当请求带password时，校验邮箱存在后仅返回错误提示但未停止后续代码执行，导致用户创建逻辑仍被触发；而不带password时，bcrypt.hash处理undefined密码会抛出异常，直接进入错误捕获分支，跳过了创建步骤，所以不会重复创建。

1. 调整逻辑顺序，提前拦截无效请求 #

把邮箱唯一性校验放在最前面，既可以提前终止无效请求，也避免不必要的密码哈希计算：

app.post('/users', async (req, res) => {
  try {
    // 第一步：校验邮箱是否已存在，存在则直接返回错误并终止流程
    const existingUser = await User.findOne({ where: { email: req.body.email } });
    if (existingUser) {
      return res.status(400).json({ errCode: 1, message: 'Your email is already exist!' });
    }

    // 第二步：校验密码是否必填，提前返回错误
    if (!req.body.password) {
      return res.status(400).json({ errCode: 2, message: 'Password is required!' });
    }

    // 第三步：哈希密码
    const hashedPassword = await bcrypt.hash(req.body.password, 10);

    // 第四步：创建用户
    const user = await User.create({
      email: req.body.email,
      password: hashedPassword
    });

    res.status(201).json({ errCode: 0, data: user });
  } catch (err) {
    res.status(500).json({ errCode: 500, message: err.message });
  }
});

2. 添加数据库级唯一约束（双重保障） #

仅靠代码校验无法避免并发场景下的重复创建（比如两个请求同时校验同一邮箱，都通过后执行创建），需要在Sequelize模型中给email字段添加唯一约束：

// User 模型定义
const User = sequelize.define('User', {
  email: {
    type: DataTypes.STRING,
    allowNull: false,
    unique: {
      msg: 'Your email is already exist!'
    },
    validate: {
      isEmail: true
    }
  },
  password: {
    type: DataTypes.STRING,
    allowNull: false
  }
});

同时在捕获异常时处理唯一约束冲突：

catch (err) {
  if (err.name === 'SequelizeUniqueConstraintError') {
    return res.status(400).json({ errCode: 1, message: 'Your email is already exist!' });
  }
  res.status(500).json({ errCode: 500, message: err.message });
}

3. 确保错误响应后终止函数执行 #

所有返回错误响应的语句必须添加return，否则Node.js会继续执行后续代码，导致重复创建用户。

1. 逻辑顺序：先做邮箱唯一性校验，再处理密码哈希和用户创建；
2. 数据库约束：给邮箱字段加唯一约束，覆盖并发场景；
3. 流程终止：错误响应后必须用return停止函数执行。

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