0

2 つのプライベート メソッドを持つクラス コンストラクターがあります。

var deleteOption = function(){...};
var removeOption = function(){...};

いくつかの操作を行った後の最初のものは、メソッドを呼び出して DOM から HTML 要素を削除しますremoveOption()。これは、このメソッドが呼び出される唯一の場所です。私は今日、そのメソッドをプライベート インターフェイスから削除し、メソッド deleteOptionを関数として内部に配置することをお勧めします。

var deleteOption = function(){
    ... //do some operations here

    removeOption();
    function removeOption() {
        ...
    }
};

1) Is it a common practice in JavaScript?

2) Is it a good desicions from the standpoint of class architecture?

4

1 に答える 1

1

その関数でのみ呼び出し、要素の配列をまったく渡さない (たとえば、ループがない) 場合は、関数全体ではなく関数コードのみを実装することもできます。removeOption が非常に大きい場合は、必要に応じてコメントしてください。同じように。

関数内に関数を配置することは、JavaScript 内で一般的に使用される方法ですが、常に役立つとは限りません。私には、他の1つの関数でのみ実行される別の外部関数を呼び出したように見えるので、できる最善のことは、それらの関数を単にマージすることです。

利点は、コードが既にネイティブ関数に含まれているため、呼び出す関数が 1 つ少なくなるため、わずかに高速になることです。

于 2013-10-23T07:05:17.543 に答える