各クラスインスタンスで新しいクラスを作成する代わりに、共通の空の関数を使用する方が効率的ですか？

Question

私は後でいくつかのコールバックを追加するように設計されたクラスを持っているとしましょう。

function myclass() { 
    this.onSomething = function() {}; 
    this.onOtherThing = function() {}; 

    this.something = function() { 
     // stuff 
     this.onSomething(); 
    }; 

    this.otherThing = function() { 
     // other stuff 
     this.onOtherThing(); 
    }; 
} 

私はthis.onSomethingとthis.onOtherThingは、それらがsomething()とotherThing()で呼び出されたときに、エラーがスローされますので、その種類は関数ではないことを示すundefinedまたはnullであることはできません。

空の関数が必要ですが、メモリを使用しているので、これを行うとクラスのメモリ効率が向上しますか？

function myclass() { 
    this.onSomething = empty; 
    this.onOtherThing = empty; 
    ... 
} 

function empty() { 

} 

このように、各クラスインスタンスのプロパティは、毎回新しい関数を作成するのではなく、同じ空の関数をポイントします。私は、空のメソッドを定義することは、多くのメモリを取らないと思うが、それでも...これは技術的に優れているのだろうか？

Answer 1

あなたは新しい機能があなたのクラスのすべてのインスタンスのために作成されているという事実について正しいです。すべてのインスタンス間で共有これを持つためには、クラスのプロトタイプにそれを宣言することができます。

var MyClass = function() { 
    this.something = function() { 
     // stuff 
     this.onSomething(); 
    }; 

    this.otherThing = function() { 
     // other stuff 
     this.onOtherThing(); 
    }; 
} 

MyClass.prototype.onSomething = function() {}; 
MyClass.prototype.onOtherThing = function() {}; 

この方法で、この方法は、すべてのインスタンスで共有されます。

Answer 2

なぜ空の関数を返すのではなく、return trueまたはreturn falseにしてみてください。 または最高のあなたが使用することができます。

function myclass() { 
    this.onSomething = false; 
    this.onOtherThing = false; 
    ... 
} 

あなたが試すことができますあなたのコメントのとおり：

function myclass() { 
    this.onSomething = empty(); 
    this.onOtherThing = empty(); 
     ... } 

function empty() { 
    //return something 
    return true; 
}