每个函数 jquery.这是正确的吗？

Question

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

Answer 1

考虑到jquery作为一个包装集（集合）工作，我认为你不需要each方法，只是

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

处理程序将应用于所有匹配的元素，而不需要each。

这将找到 .dragbox 项目内的所有 .close 和 .colpase 我认为这就是您想要的...

编辑以在中使用查找以获得轻微的性能提升。谢谢丹/亚历克斯。

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

如果您在“each”中将“parent”引用为实际的“.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);
}