javascriptのクロージャでメモリリークが起こる場合

Question

javascriptのクロージャでメモリリークが起こる場合があると思います。

var hoge = function(){
    var x = 0;
    process.on('exit', function(){
        x++;
    });
    return function(){
        return x;
    }
}
var h = hoge();
console.log(h());
console.log(h());
console.log(h());

varのつく変数はDontDelete属性がつくため、delete演算子で解放できませんが、上記の例でvarをつけずに、

var hoge = function(){
    x = 0;
    process.on('exit', function(){
        x++;
    });
    return function(){
        return x;
    }
}
var h = hoge();
delete x;
console.log(h());
console.log(h());
console.log(h());

とするのはありですか？

Answer 1

varのつく変数はDontDelete属性がつくため、delete演算子で解放できませんが、上記の例でvarをつけずに、...(中略)...とするのはありですか？

グローバル変数を使うと内包関数が参照する「グローバル変数 x」が共有されてしまいます。
クロージャの優位性を失うことになると思うのですが、それはクロージャにする意味があるのでしょうか。
また、その例では Strict Mode で ReferenceError になります。

DontDelete 属性が存在するのは ES3 まで
DontDelete 属性が存在するのは ES3 までであり、ES5 では削除されています。

• 変数宣言のvarアリとナシの違いや使い分け(14790)｜teratail

クロージャがメモリ開放しないのは仕様
そもそも論としてクロージャがメモリを手放さないのは内包関数がローカル変数にスコープを持っているからであり、スコープを持つ変数をガベージコレクションで回収できない事に起因します。
つまり、仕様であってメモリリークではありません。

ガベージコレクション
メモリ開放が必要ならガベージコレクションによって開放されるようにスコープを調節してください。
ガベージコレクションは「対象がどこからも参照されなくなった時」に自動的にメモリを開放します。

'use strict';
(function () {
  var hoge = (function (){
    var x = 0;

    return function foo () { return ++x; };
  });

  function init () {
    var h1 = hoge();
    console.log(h1());  // 1
    console.log(h1());  // 2
    console.log(h1());  // 3

    var h2 = hoge();
    console.log(h2());  // 1
    console.log(h2());  // 2
    console.log(h2());  // 3
  }

  init();
}());

Re: tixure55 さん

Answer 2

上のコードは環境によっては動きません。ChromeとFirefoxのコンソールで（process.onを抜いて）試してみたら、ReferenceErrorとなってしまいました。

そもそも前提として、「varのつく変数はDontDelete属性がつくため、delete演算子で解放でき」ないのは間違いないですが、外側のクロージャ自体が解放されてアクセス不能になれば、ガベージコレクタが適切に回収していきます（Chromeでの話ですが、クロージャ内にイベントを仕掛けたものをデバッグしていた時に、イベントなど外に保存されている関数から参照されない変数が綺麗サッパリ消えていた、ということがありました）。