使用async/await遇报错及用户注册校验失效问题求助

你遇到的核心问题有两个：参数传递不匹配导致校验逻辑完全失效，以及移除await破坏异步流程导致校验未完成就创建用户，另外还有一处容易忽略的语法问题，我们一步步拆解解决：

1. 最关键的问题：参数接收不匹配 #

在路由代码中，你调用ValidateUserExists时传递的是一个对象：

const existingUser = await User.ValidateUserExists({username, email});

但在user-exists-check.js里，ValidateUserExists的函数定义是接收两个独立参数：

async function ValidateUserExists(username, email) { ... }

这直接导致函数内部的username和email都是undefined，校验逻辑完全跳过，直接返回null，自然无法拦截重复用户。

修正参数接收 #

把ValidateUserExists改成接收对象参数，和路由的调用逻辑匹配：

// user-exists-check.js
async function ValidateUserExists({ username, email }) {
  if(!username || !email ) throw new Error('Not enough args provided');
  
  let existingUser = null;
  
  // 先检查用户名是否存在
  const usernameUser = await UserNameExists(username);
  if (usernameUser) {
    existingUser = usernameUser;
  } else {
    // 用户名不存在时，再检查邮箱
    const emailUser = await EmailExists(email);
    if (emailUser) {
      existingUser = emailUser;
    }
  }
  
  return existingUser;
}

2. 移除await的致命问题 #

当你移除await时，User.ValidateUserExists({username, email})会直接返回一个Promise对象，而不是实际的用户数据。此时if(existingUser)会判断为true（因为Promise是真值），但后续的username === existingUser.username会因为existingUser是Promise而非用户对象而不成立，代码会跳过错误返回，直接执行创建用户逻辑，导致重复数据入库。

必须保留await，它的作用是等待异步校验完成，拿到真实的用户数据后再判断是否创建新用户。

3. 容易忽略的语法错误 #

你的路由回调函数没有标记async，但内部使用了await，这会直接触发语法错误！await只能在async函数内部使用，所以必须修正路由的回调函数定义：

router.post('/', asyncHandler(async (req,res) => { // 这里添加async
  // ... 原有逻辑
}));

4. 额外优化：校验错误后终止流程 #

原代码中参数校验失败后没有加return，会导致后续的存在性校验和创建用户代码仍然执行，引发不必要的数据库请求：

// 原代码
if(validation) res.status(400).json({error: validation});

// 修正后
if(validation) {
  return res.status(400).json({error: validation});
}

修正后的完整代码 #

路由代码 #

const express = require("express");
const router = express.Router();
const asyncHandler = require('../helpers/asyncHandler');
const validate = require('validate.js');
const User = require('../controllers/user-exists-check');

//Route for creating user
router.post('/', asyncHandler(async (req,res) => {
 const constraints = {
 username: { presence: true, length:{minimum:4,maximum:10} },
 password: { presence: true, length:{minimum:6, maximum:20} },
 email: { presence: true, email:true }}
 const username = req.body.username;
 const email = req.body.email;
 const password = req.body.password;
 const permission_id = req.body.permission_id;
 const validation = validate({username, email, password}, constraints);
 if(validation) {
   return res.status(400).json({error: validation});
 }
 const existingUser = await User.ValidateUserExists({username, email});
 if(existingUser) {
 if(username === existingUser.username) return res.status(400).json({error:`Username ${username} is already taken`});
 if(email === existingUser.email) return res.status(400).json({error:`Email is already taken`});
 }
 const newUser = await User.CreateNewUser({username, email, password, permission_id});
 return res.status(200).json({user: newUser})
}));
module.exports = router;

user-exists-check.js #

const {UserNameExists, EmailExists, CreateUser} = require('./user-db-check');

async function ValidateUserExists({ username, email }) {
  if(!username || !email ) throw new Error('Not enough args provided');
  
  let existingUser = null;
  
  const usernameExists = await UserNameExists(username);
  if (usernameExists) {
    existingUser = usernameExists;
  } else {
    const emailExists = await EmailExists(email);
    if (emailExists) {
      existingUser = emailExists;
    }
  }
  
  return existingUser;
}

async function CreateNewUser(args){
 return await CreateUser(args);
}

module.exports = { ValidateUserExists, CreateNewUser };

总结 #

• 确保函数参数传递和接收的格式一致（对象传参就用对象接收）
• 永远不要移除异步操作前的await，否则会破坏异步流程，导致校验逻辑失效
• 内部使用await的函数必须明确标记async
• 参数校验失败后要终止后续代码执行，避免不必要的数据库操作

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