问题
在node的数据库模块中,运行后报错Error [ERR_HTTP_HEADERS_SENT]: Cannot set headers after they are sent to the client
前提
在node中,res.send()是不能被同时调用两次及以上的,也正好印证了报错信息中所说的Cannot set headers
情景
在用户注册接口中,逻辑为:
- 首先判断用户填写的用户名是否被占用
- 如果没有被占用,再将该用户信息插入到用户表中
以上逻辑需要用到两次数据库操作,即db.query()
最开始我想当然的把两次数据库操作并列书写,结果就报错了。
原因
实际上这里涉及到数据库操作异步的问题。据我推测,db.query()是异步的,也就是说两次数据库操作可能会同时发生,也就意味着可能同时发生两次res.send()操作,从而出现错误。
解决
知道上述原因之后就好解决了,直接把第二次db.query()放到第一次db.query()的处理函数中。
代码
// 用户注册
exports.userRegister = (req, res) => {
const userinfo = req.body
// 1.判断是否合法 (已经采用第三方包来进行验证,不用ifelse)
// if (!userinfo.username || !userinfo.password) {
// // return res.send({ status: 1, msg: '用户名或密码不合法' })
// return res.cc('用户名或密码不合法')
// }
// 2.检测用户名是否被占用
const sqlStr = 'select * from ev_users where username=?'
db.query(sqlStr, userinfo.username, (err, results) => {
// 执行sql语句失败
if (err) {
// return res.send({ status: 1, msg: err.message })
return res.cc(err)
}
// 判断用户名是否被占用
if (results.length > 0) {
// return res.send({ status: 1, msg: '用户名已被占用,请更换其他用户名!' })
return res.cc('用户名已被占用,请更换其他用户名!')
}
/**
* 注意这里存在一个异步的问题。如果两个db.query()并行写,就会存在两次res.send同时发送的情况。
* 所以要避免,就把第二个db.query放在第一个db.query里面
*/
// 3.调用bcrypt.hashSync对用户密码进行加密
userinfo.password = bcrypt.hashSync(userinfo.password, 10)
// 4.插入用户记录
const sql = 'insert into ev_users set ?'
db.query(sql, { username: userinfo.username, password: userinfo.password }, (err, results) => {
// 判断是否注册成功
if (err) return res.cc(err)
// 判断影响行数是否为1
if (results.affectedRows !== 1) return res.cc('注册用户失败,请稍后再试')
// 用户注册成功
return res.send({ status: 0, msg: '注册成功!' })
})
})
}