このシングルトンパターンには何か問題がありますか？

Question

私は初心者ですので、私と一緒に、私はシングルトンパターンを見て、それを実装する方法を理解するのは非常に難しいです、と私はいくつかの人々がほとんどの時間を避けることを提案するだろう知っているので、私はそれが理解しやすいと思うバリエーションしかし、何とか私はこれがこのパターンの最高の実装ではないと感じます、あなたはこのパターンのより良い形を提案することができます。

var Foo = (function() { 
 
    
 
    var instance; 
 
    var _priVar = 2; 
 
    var log = function() { 
 
     console.log("Hello"); 
 
    }; 
 
    
 
    function Singleton(x, y) { 
 
     if (instance) { 
 
      return instance; 
 
     } 
 
     
 
     this.name = x; 
 
     this.age = y + _priVar; 
 
     this.log = log; 
 
     
 
     instance = this; 
 
    } 
 
    Singleton.getInstance = function() { 
 
     return instance || new Singleton(); 
 
    } 
 
    return Singleton; 
 
}());

と 私の目標は、私たちが

var a = new Foo("Bob", 24); var b = new Foo(); var c = Foo(); var d = Foo.getInstance();

を次の手順を実行したときに、我々はまだ取得することである

a == b // true 
a == c // true 
a == d // true 
a.name // 'Bob' 
b.age // 26 
c.log // 'Hello' 
d.name // 'Bob' 

もモジュールパターンとして知られている最も簡単なシングルトンは、オブジェクトリテラルで構成さのおかげで、

Answer 1

：

var foo = (function() { 
    var x = "Bob", 
     y = 24, 
     _priVar = 2; 

    function log() { 
     console.log("Hello"); 
    } 

    return { 
     name: x, 
     age: y + _priVar, 
     log: log 
    }; 
}()); 

あなたが怠惰な初期化を紹介したい場合は、あなたがあなたのように余分なgetInstance機能を使用することができます実装：

var getFoo = (function() { 
    var instance = null; 
    return function getFooInstance() { 
     if (instance) return instance; 

     var x = "Bob", 
      y = 24, 
      _priVar = 2; 

     function log() { 
      console.log("Hello"); 
     } 

     return instance = { 
      name: x, 
      age: y + _priVar, 
      log: log 
     }; 
    }; 
}()); 

シングルトンは、コード内のようなコンストラクタを使用しないでください。これは単なる不要です。あなたが初期化の引数を渡す必要があると感じたら、はそれをシングルトンにしないでください。