Jquery array - set and load values

Question

I am having 2 divs:

<div id="player" class="setPlayers">value1</div>
<div id="player2" class="setPlayers">value2</div>

All I want to do is to set the value1 and value2 of the divs to an array for reading later on; I tried out this code, but doesn' t work:

var array = new Array();

$.each($('.setPlayers'), function(key, object) {
    console.log(index + ':' + value);
    array.push(value);
    alert(value);
});

$.each(array, function(index, value) {
    console.log(index + ':' + value);
    console.log(index + ':' + $(this).val());
});

what' s wrong according to you ?,cheers

Well just as a code review level, where does the variable "value" get declared, in the first each ? — Mark Broadhurst
– Mark Broadhurst, Commented Aug 22, 2012 at 11:49

Mark Broadhurst · Accepted Answer · 2012-08-22 11:46:59Z

5

Using your code as a start I came up with:

$(function(){
  var array = [];

  $('.setPlayers').each(function(index) {
      console.log(index + ':' + $(this).text());
      array.push($(this).text());
      alert($(this).text());
  });

  $.each(array, function(index) {
      console.log(index + ':' + this);
      console.log(index + ':' + this);
  });
});

Few too many alerts and debug for my liking.

If you just want to fill the array then this will do:

$(function(){
  var array = $('.setPlayers').map(function() { return $(this).text()); }).get();
});

answered Aug 22, 2012 at 11:46

Mark Broadhurst

2,69523 silver badges47 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

dtjmsy Over a year ago

Cheers Gerbil, that, exactly what I need, yes indeed many alerts :)

Danil Speransky · Accepted Answer · 2012-08-22 11:41:27Z

3

Try this (see demo: http://jsfiddle.net/GcjgH/):

var array = $('.setPlayers').map(function() {
  return $(this).text();
}).get();

alert(array[0]);
alert(array[1]);

answered Aug 22, 2012 at 11:41

Danil Speransky

30.5k6 gold badges70 silver badges78 bronze badges

Comments

Ohad · Accepted Answer · 2012-08-22 11:39:47Z

0

The correct syntax would be $('.setPlayers').each(function(){ ... });

In the context of the question:

$('.setPlayers').each(function(){
     console.log($(this).attr("id")+':'+$(this).text());
     array.push($(this).text());
     alert($(this).text());
});

for (var i=0; i<array.len; i++){
    console.log(arr[i]);
}

etc.

answered Aug 22, 2012 at 11:39

Ohad

1,7191 gold badge16 silver badges20 bronze badges

Comments

sp00m · Accepted Answer · 2012-08-22 11:40:44Z

0

This should work:

var array = new Array();
$(".setPlayers").each(function() {
    array.push($(this).text());
});

answered Aug 22, 2012 at 11:40

sp00m

49k31 gold badges152 silver badges263 bronze badges

Collectives™ on Stack Overflow

Jquery array - set and load values

4 Answers 4

1 Comment

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

1 Comment

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related