使用Node.js util模块promisify方法时出现报错（附代码）

运行报错的根源是promisify封装Redis方法时的上下文绑定顺序错误，导致sadd方法内部的this指向丢失。
你当前的写法promisify(redisDb.sadd).bind(redisDb)存在逻辑问题：

• 直接取出redisDb.sadd作为参数传入promisify时，该方法已经脱离了redisDb实例上下文，promisify内部调用原sadd方法时，无法访问Redis实例的内部属性（连接配置、连接池实例等），直接触发类型错误或内部调用报错。
• 后续的.bind(redisDb)是绑定在promisify返回的包装函数上，不会修正已经传入promisify的原方法的this指向，绑定完全不生效。

只需要调整上下文绑定和promisify的执行顺序即可，以下方案二选一即可：

• 通用兼容写法：先给原方法绑定实例上下文，再传入promisify做Promise转换，适配所有版本的Redis客户端。
• 简化写法：如果你使用的是ioredis客户端，库本身所有方法默认返回Promise，不需要手动做promisify封装，直接await redisDb.sadd()调用即可。

const { promisify } = require('util')


module.exports = (db,redisDb) =>{
    // 修正：先绑定原方法的redisDb上下文，再做promisify封装
    const saddAsync = promisify(redisDb.sadd.bind(redisDb))
    
    return async(request, response) =>{
   
        const result = await db.collection('polls').updateOne({
            _id: request.params.poll,
            'choices._id': request.body.choice
        },{
            $inc:{
                'choices.$.count':1
            }
        })
        await saddAsync(request.params.poll, request.body.ip)

        return response.json({
            message: 'vote has been registered.'
        })
    }
}

你当前用sadd存储IP是为了做投票去重防刷，建议给对应Redis key设置合理的过期时间，避免无效数据长期占用内存。

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