jquery - 各関数jquery. それが正しいか？

Question

 $('.dragbox').each(function(){
        $('.close').click(function(){
            $(this).parent().hide();
        }),
        $('.colpase').click(function(){
            $(this).siblings('.dragbox_content').toggle();
        })
     });    

Answer 1

jqueryがラップされたセット（コレクション）として機能することを考えると、各メソッドが必要だとは思いません。

$('.dragbox').find('.close').click(function(){
    $(this).parent().hide();
})
$('.dragbox').find('.colpase').click(function(){
    $(this).siblings('.dragbox_content').toggle();
})

ハンドラーは、それぞれを必要とせずに、一致したすべての要素に適用されます。

これにより、.dragboxアイテム内のすべての.closeと.colpaseが検出されます。

パフォーマンスをわずかに向上させるために、findを使用するように編集しました。ダン/アレックスに感謝します。

Answer 2

いいえ。おそらく、各ドラッグボックス内の一致する要素にクリック ハンドラーを適用する必要があります。あなたはそれを行うことができます：

 $('.dragbox').each(function(){
        $('.close', this).click(function(){
            $(this).parent().hide();
        }),
        $('.colpase', this).click(function(){
            $(this).siblings('.dragbox_content').toggle();
        })
     });    

ハンドラーをグローバルに追加したいだけなら、each は必要ありません。

Answer 3

そこに each() 関数が必要なようには見えません。イベント ハンドラーをオブジェクトに複数回適用している可能性があります。ただ：

    $('.close').click(function(){
        $(this).parent().hide();
    });
    $('.colpase').click(function(){
        $(this).siblings('.dragbox_content').toggle();
    });

トリックを行う必要があります。

Answer 4

「親」を「各」の実際の「.dragbox」に言及している場合

$('.dragbox').each(function(){

  var self = this;

  $(self).find('.close').bind("click", function(){

    $(self).hide();

  }); // <-- ","?

  $(self).find('.colpase').bind("click", function(){

    //This line confuses me because you could do the same in the selector for the click event
    //unless you do have more things in the function
    $(this).siblings('.dragbox_content').toggle();

  });

  /*
  $(self).find('.colpase').siblings('.dragbox_content').bind("click", function(){
    $(this).toggle();
  });
  */

}); 

Answer 5

function set_colors(pick_color)
{
    var color_code=pick_color;
    $('#'+ color_owner).parent().css('background-color',color_code);
}