これはjavascript Closureの候補ですか？

Question

このコードは動作します。しかし、私はこれが閉鎖を実装するための候補者であるかどうか考えてきました。私を聞いてください。

私はこれをしたい：

$(".trigger").click(function() {  
    $(this).parents(".heading").next(".list").slideToggle("slow", function() { 
     $(this).toggleClass("close"); 
     $(this).parents(".heading").toggleClass("close"); 
    }); 

});  

実行が側slideToggle機能thisにある問題はtriggerを指していないようです。私はこれを行うことで仕事を見つけた：

$(".trigger").click(function() {  
    var self = $(this); 
    $(this).parents(".heading").next(".list").slideToggle("slow", function() { 
     $(self).toggleClass("close"); 
     $(self).parents(".heading").toggleClass("close"); 
    }); 
}); 

それは動作します。しかし、私は閉鎖を実行するための良い候補者かどうか疑問に思っていますか？そして、コード品質を向上させます。ありがとう。

Answer 1

クロージャは、その目的が異なるため、ここでは特に役立ちません。バインディング矢印関数は任意のコンテキストの無知であり、ちょうどthisを意味字句文脈に固執：、thisが変更されることはありません今

$(".trigger").click(function() {  
    $(this).parents(".heading").next(".list").slideToggle("slow",() => { 
     $(this).toggleClass("close"); 
     $(this).parents(".heading").toggleClass("close"); 
    }); 
}); 

：しかし、あなたは字句コンテキストを使用する矢印関数の動作を使用することができますclickコールバック関数と同じthisです。