これは私の目的のために閉鎖する正しい方法でしたか？

Question

私は徐々に閉鎖を理解しており、次のコードが機能します。私がここでやろうとしていることを達成するためのより簡単な方法があれば、私は思っています。

以下のコードでは、thisは、単にオブジェクトであり、配列であるsectionsというプロパティを参照しています。私はこの配列をループし、各プロパティ（DOM要素）のホバーイベントにそれぞれsectionのものをバインドします。閉鎖の出番私はhover()方法に供給するコールバックです。メインthisオブジェクトには二つの方法.sectionMouseenter()と（私のコードでe.typeによって表される）イベントタイプがmouseenterかmouseleaveであるかに応じて呼び出されます.sectionMouseleave()を持っています。引数として、現在のセクションがこれらのメソッドに渡されます。もちろん、forループの反復にはJavaScriptでは可変スコープがないため、section変数の参照をカプセル化するにはクロージャが必要です。

for (var i = this.sections.length - 1; i >= 0; i--) { 
     var section = that.sections[i]; 
     section.el.hover(
      (function(section){ 
       return function(e){ 
        that['section' + e.type.capitalize()](section); 
       } 
      })(section) 
     ); 
    }; 

この閉包を書いたのは「正しい」方法ですか、それとも良い方法ですか？

Answer 1

オンザフライでは関数を構築しませんが、forループの外側に配置してください。

var sections = this.sections; 

function dummy(section) { 
    return function(e) { 
     that['section' + e.type.capitalize()](section); 
    } 
} 

for (var i = sections.length - 1; i >= 0; i--) { 
    section.el.hover(dummy(sections[i])); 
}