Node+Express+Mongoose注册接口POST请求500错误排查求助

你的代码存在几个关键错误，直接导致了500状态码和不符合预期的响应逻辑：

1. Mongoose查询方法参数错误 #

User.findOne(username) 不符合Mongoose的API规范，findOne必须接收查询对象作为参数，正确写法应为：

const uniqueCheck = await User.findOne({ username });

2. 必填字段检查顺序颠倒 #

你先执行了密码哈希、用户名唯一性校验，再检查字段是否为空。如果未传入password，bcrypt.hash(password, salt)会因password为undefined抛出错误，直接进入catch块返回500，根本无法触发400的参数校验逻辑。必须优先检查必填字段，再执行后续操作。

3. Mongoose创建文档参数错误 #

User.create(username, { password: hash }) 用法错误，create方法仅接收一个对象参数，需将所有字段整合到同一对象中：

const user = await User.create({ username, password: hash });

4. 响应后未终止函数执行 #

返回响应（如403、400）后，代码会继续向下执行，可能导致后续逻辑报错或尝试重复发送响应。每个响应语句后必须添加return终止函数。

修复后的完整代码 #

const jwt = require('jsonwebtoken')
const bcrypt = require('bcrypt')
const User = require('../models/userModel');

const registerUser = async (req, res) => {
    try {
        const { username, password } = req.body;
        
        // 优先检查必填字段
        if (!username || !password) {
            return res.status(400).json('Please fill in all fields')
        }

        // 校验用户名唯一性
        const uniqueCheck = await User.findOne({ username });
        if (uniqueCheck) {
            return res.status(403).json('Username already exists');
        }

        // 哈希密码
        const salt = await bcrypt.genSalt(10);
        const hash = await bcrypt.hash(password, salt);

        // 创建用户
        const user = await User.create({ username, password: hash }); 
        res.status(201).json(user);
        
    } catch (error) {
        // 打印错误便于调试
        console.error(error);
        res.status(500).json({ error: 'Problem registering user', detail: error.message });        
    }
}

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