Question

我正在使用异步功能来确认用户是否存在于具有相同用户名或电子邮件的数据库中。

问题是我在单独的文件中具有异步功能。我正在将文件导入特定端点的路由。

当我发送删除后的注册用户请求

await

它通过User.CreateNewUser和User.ValidateUserExists进行，但是当我再次发送相同的请求时，会将同一用户添加到数据库中，而不验证用户是否存在于数据库中。

这是我的路线

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((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) 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;

这是用户存在的支票

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

async function ValidateUserExists(username, email) {
    if(!username || !email ) throw new Error('Not enough args provided')
    let ValidUsername = null;
    let ValidEmail = null;

    if(username) {
        ValidUsername = await UserNameExists(username);
    }
    if(email) {
        ValidEmail = await EmailExists(email);
    }
    if(ValidUsername && ValidEmail) return ValidUsername;
    return null;
}

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

module.exports = {
    ValidateUserExists,
    CreateNewUser
}

我希望它能引发这个回应

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`});

我使用await和async错误的方式吗？有人可以启发我

Answer 1

您的问题是您发送给asyncHandler(的函数回调未标记为异步，因此您在内部使用的任何函数都不能使用await。

假设您的asyncHandler希望兑现诺言，那么asyncHandler(async (req,res) => {应该可以解决这个问题。

作为旁注，查看您的代码，您可能要考虑一个基本的逻辑问题。理论上，在多用户环境中，先执行UserNameExists / EmailExits然后执行CreateUser，仍然可以存储相同的用户和电子邮件。大多数数据库都有一种称为事务锁的东西，可以防止这种情况的发生。

在等待时出错，该等待仅在使用异步功能时

1 个答案: