javascriptのコードについて

質問

以下のコードなのですが、
エンクロージャー関数の
ローカル変数hogeをさらに、エンクロージャー内部で定義された
関数ででクロージャーとして保持させたいメソッドのコードですが

var Method = function (){

var hoge = "初期値";
var getter = function (){
return hoge;
}
var setter = function (param){
hoge = param;
return hoge;
}
return {"set" : setter,"get" : getter}
}

var obj = Method();
console.log(obj);
console.log(obj . get());
obj.set("初期値変更");

console.log(obj.get());

obj . set("更に変更");
console.log(obj.get());

この場合、メソッドの返り値として、一般的な文献に乗っている関数（関数オブジェクト）を返すのではなく
オブジェクトリテラルとして返しています。
この場合でも、動きとしてはクロージャーの動きをしているのでhogeという変数の保持はできているっぽいんですが
クロージャーって関数内で定義された関数であれば、どういう返り値の返し方でも
クロージャーになるのですかね？
また、この方法は、一般的にjsで関数コンストラクタ呼び出しをしてインスタンスを作る際privateメンバを実現する方法として紹介されていますが、
これはクロージャーとして生成するたんびに内部の変数を保持するためメモリ食い虫になるらしいのですが
まず間違いなく、このクラス（便宜上そう呼びます）のインスタンスはひとつしかつくることはない！！という仕様だとしても
いけないのでしょうか？
というかもう現状javascripのバッドノウハウ的なものになっているのでしょうか？

回答No.2

b0a0a · ベストアンサー

別に値を返さないとクロージャじゃないというわけではありませんよ
要は変数を保持する関数がクロージャだと自分は理解しています

var getter = function (){
return hoge;
}
の時点で１つクロージャはできています

このクロージャは１つ上のスコープにMethod関数の変数、例えば
hoge = "初期値"
を持っており、Method関数が終わってもその繋がりを持ち続けているのです

ですから関数をreturnするときに何か特別なことが行われているわけでなく、関数そのものが特殊なので、返し方には影響されません

また、メモリリーク等通常であれば気にすることはありません
そもそもクロージャとは直接関係ありません

DOMと循環参照が起きた場合GCによる不要なメモリの回収が困難になるという問題があって、
その処理を書くときにクロージャが便利でよく使われるので、気をつけましょうということであって、クロージャを避けようと受け取ってしまったのなら大変な誤りです

それに仮にメモリリークで１回に数KBメモリが消費されたところで、１０００回やっても数MBです
非常に長時間動作するWebアプリケーションでないのなら杞憂というものです

回答No.1

notnot

＞クロージャーって関数内で定義された関数であれば、どういう返り値の返し方でもクロージャーになるのですかね？

返しているオブジェクトの要素が関数ですからね。で、その返された関数を呼び出している。
何の不思議もないと思いますけど。

＞これはクロージャーとして生成するたんびに内部の変数を保持するためメモリ食い虫になるらしいのですが

普通の言語だと、オブジェクトの生成に相当しますから、そのオブジェクト用のメモリが確保されます。
解放せずに、生成を繰り返せばどんどん使用メモリは増えるし、適宜オブジェクトを解放すれば(例だと、obj=null; すれば)、メモリも解放されてガベージコレクションされます。

＞いけないのでしょうか？

必要ならばすればいいので全く問題ありません。