我已经用我的node.js应用程序做了一些测试,寻找我的代码应该做的内存泄漏。我运行的脚本在我看来应该会泄漏内存,但我对结果感到惊讶。
redisClient.on('message', initRequest);
function onSuccess(self, json){
console.dir(json);
}
function initRequest(channel, message){
var request = new RequestObject({
redisMessage: message
});
request.on('success', onSuccess);
}
redisClient每秒会发出几个“消息”事件。这意味着initRequest
函数经常被调用。每次在内存中创建request
对象时,都会将函数onSuccess
绑定到它的“成功”事件。
我假设(但这里我可能错了),只要listener (本例中为onSuccess
)绑定到这个对象,它就不能被垃圾收集。然后我想,内存使用量将会增长,因为内存不会被释放。
作为这个潜在泄漏的解决方案,我想使用.once
而不是.on
,因为这将解除侦听器的绑定,并且对象可能会被垃圾回收。
我已经使用pmap测试了这两个场景(比较.on
和.once
以及另一个场景,这里不值得一提),我没有发现很大的区别。
总结一下,我有两个问题:
.on
的示例代码应该泄漏内存,这在内存消耗图上看不到?发布于 2014-02-25 22:55:39
1:是:-)
2:一般来说,使用事件侦听器时的内存泄漏是因为正在发出的对象保留了对它的引用,所以不会对正在侦听的对象进行垃圾回收。
因此,在您的代码中,request
对象将引用onSuccess
函数。但是,onSuccess
只是一个被重用为所有请求对象侦听器的函数,这样就不会导致内存积累。
旁注:我不知道redisClient
和RequestObject
的内部结构,但在我看来,一旦initRequest
函数完成,也就是在调用任何侦听器之前,request
就会准备好进行垃圾回收。
发布于 2014-02-25 23:03:12
据我所知,request对象应该只存在于initRequest函数中,因此应该在函数终止时标记为垃圾回收。
https://stackoverflow.com/questions/21162969
复制相似问题