javascript - jQuery Each() function not working - Stack Overflow

admin管理员组

文章数量:1425200

I'm trying to add a class to each div.banner inside of my #destaques, but isn't working. What's happening?

JS:

$(document).ready(function() {
  bannerRotator("#destaques");
});



function bannerRotator(element) {

  // Conta quantos banners existem:

  i = 0;

  $(element).find(".banner").each(function() {
    i++;
    $(this).addClass("test");
  });

  alert(i);

  //

}

HTML:

<div id="destaques">
<div class="banner"><img src="images/001.jpg"/></div>
<div class="banner"><img src="images/002.jpg"/></div>
<div class="banner"><img src="images/003.jpg"/></div>
</div>

I'm trying to add a class to each div.banner inside of my #destaques, but isn't working. What's happening?

JS:

$(document).ready(function() {
  bannerRotator("#destaques");
});



function bannerRotator(element) {

  // Conta quantos banners existem:

  i = 0;

  $(element).find(".banner").each(function() {
    i++;
    $(this).addClass("test");
  });

  alert(i);

  //

}

HTML:

<div id="destaques">
<div class="banner"><img src="images/001.jpg"/></div>
<div class="banner"><img src="images/002.jpg"/></div>
<div class="banner"><img src="images/003.jpg"/></div>
</div>

• javascript
• jquery
• html

Share Improve this question Follow edited Mar 22, 2011 at 18:58 Jakub Hampl 40.6k1010 gold badges7979 silver badges111111 bronze badges asked Mar 22, 2011 at 18:54 Lucas VeigaLucas Veiga 1,79377 gold badges2727 silver badges4848 bronze badges 2

• 1 var i = $('#destaques .banner').addClass('test').length, – Josiah Ruddell Commented Mar 22, 2011 at 18:57
• Working for me. – Brad Christie Commented Mar 22, 2011 at 18:58

Add a ment  | 

2 Answers 2

Sorted by: Reset to default 6

addClass will work on a collection automatically.

$("#destaques").find(".banner").addClass("test");

Example on jsfiddle.

side note: this could be also be simplified to

$("#destaques .banner").addClass("test");

Try this

jQuery.each($("div.banner"), function() {
  i++;
  $(this).addClass("test");
});

But if you just want to add the class you can also do it in the following one liner

alert($('div.banner').addClass("test").length);

本文标签： javascriptjQuery Each() function not workingStack Overflow