Question

function Redis(){

this.redis=require('redis-node');
this.client =this.redis.createClient(6377, '127.0.0.1', {detect_buffers: true});
this.client.auth("pwd");

}

module.exports=Redis;

Redis.prototype.setKeyValue=function(key,value){

var obj=this;
this.client.get(key,function(err,res){

    if(res==null){


        obj.client.set(key,value,function (err, result) {
            console.log(result); 
            obj.client.quit();//here im getting error as client doesn't have method quit
         });

    }
    else{

        console.log('Sorry!!!key is already exist');
    }
});


  };

Answer 1

最后我才知道

https://github.com/bnoguchi/redis-node

在上面的库中，客户端没有名为client.quit（）的方法，而不是我们可以使用client.close（）。

https://github.com/mranney/node_redis

这里有一个名为quit的方法来关闭连接。

Answer 2

NodeJS的首选库是in-node_redis。此外，您的代码不受race condition保护（get和set之间的密钥可能是另一个进程的set）希望Redis为此提供命令： setnx

最后，您的代码可以简化为：

var redis = require("redis"); function Redis() { this.client = redis.createClient(6377, '127.0.0.1', { detect_buffers: true, auth_pass: "pwd" }); } /** * @param {String} key key name * @param {String} value key value * @param {Function} f(err, ret) */ Redis.prototype.setKeyValue = function(key, value, f) { this.client.setnx(key, value, f); }; module.exports = Redis;

但是我不明白为什么你不直接使用redis client api而不是将它包装在Redis函数中？

退出命令在node-redis中给出错误

2 个答案: