node.js - How to improve this code with memory leak in nodejs with zlib?

Question

I have problem with memory leak in nodejs when I added gzip support. I have wrote some code for tests. In code below I use zlib and I don't know why I have memory leak. How I can improve this code to avoid memory leaking ? Anybody can help me ?

var 
zlib = require('zlib'),
crypto = require('crypto');

var cacheList = {
  article: {},
};

  var timeoutId1, timeoutId2

  console.log('process.pid: '+ process.pid);

  clean = function()
  {
    var time = new Date().getTime();
timeoutId1 = setTimeout(function() { clean() }, 5000);

var countDeleted = 0;

for (id in cacheList.article) {
  if (cacheList.article[id] && cacheList.article[id].timeExpire + 5000 < time) {
    delete cacheList.article[id];
countDeleted++;
  }
}

console.log('deleted: ' + countDeleted);

}

  run = function()
  {
var time = new Date().getTime();
timeoutId1 = setTimeout(function() { run() }, 5);

var md5 = crypto.createHash('md5');
md5.update('' + time);
var id = md5.digest('hex');

//console.log('id: ' + id);
var text = id+id+id+id+id+id+id+id +id+id+id+id+id+id+id+id +id+id+id+id+id+id+id+id +id+id+id+id+id+id+id+id +id+id+id+id+id+id+id+id +id+id+id+id+id+id+id+id +id+id+id+id+id+id+id+id +id+id+id+id+id+id+id+id;

zlib.gzip(text, function(err, result) {
  if (!err) {
      cacheList.article[id] = {
      timeExpire: time + 10000,
      data: text,
      datagzip: result,
    };
  }
});

}

  timeoutId1 = setTimeout(function() { run() }, 3000);
  timeoutId2 = setTimeout(function() { clean() }, 5000);

Answer 1

将最后两行更改为

timeoutId1 = run()
timeoutId2 = clean()

由于您已经拥有调用 setTimeout 的函数，通过使用另一个 setTimeout，您是在告诉它运行该函数的另一个实例（它反过来调用自己，所以现在您让它运行两次，依此类推）。