JavaScriptのベストプラクティスのプロトタイプ

Question

私は、コンストラクタのプロトタイプで関数を持っている：

function Animal(name) { 
    this.name = name 
} 

Animal.prototype.generateToys = function(numberOfToys) { 
    if(numberOfToys == 1) { 
    this.createToys(); 
    } 
    else { 
    this.createToys(); 
    } 
} 

createToysは、まだ宣言する必要があります。そして、それは私の質問が指しているところです。

function Animal(name) { 
    this.name = name 
} 

Animal.prototype.generateToys = function(numberOfToys) { 
    if(numberOfToys == 1) { 
    this.createToys(); 
    } 
    else { 
    this.createToys(); 
    } 

    function createToys() { 
    ... 
    ... 
    ... 
    } 
} 

それとも、以下のような方法（プロトタイプ）として、それを作成します。：generateToysが、そうのような方法generateToys内createToysを作成する方が良いだろう、createToys()を呼び出します唯一の方法になると仮定すると

Animal.prototype.createToys = function() { 
    ... 
    ... 
    ... 
} 

何が良いのでしょうか、なぜですか？ :)

Answer 1

このメソッドをgenerateToys()メソッドの中に挿入すると、そのメソッドを呼び出すたびに再宣言され、メソッドが完了するとスコープから削除されます。ほとんどの場合、これはあなたが望むものではないので、別の方法として作成することをお勧めします。

Answer 2

これはアーキテクチャによって異なります。

Animalのインスタンスをたくさん用意したい場合は、プロトタイプにメソッドを追加する方が良いでしょう。それ以外の場合、プライベート関数がたくさん作成され、コストパフォーマンスが高くなります。プロトタイプの内部関数を宣言

Answer 3

は、その範囲外の視界を制限し、機能を使用すると、プロトタイプにそれを宣言するAnimal.prototype.generateToys()（メモリの無駄）

Animal.prototype.generateToys = function(numberOfToys) { 
    var createToys = function createToys() { 

    } 

}) 

呼び出すたびに宣言することになりつまり、あなたのインスタンスの各それを直接呼び出すことができます。プロトタイプ自体は、すべてのインスタンスで共有される単一の参照であるため、その宣言を持つメモリ内に1つの場所しかありません。