Question

I’m trying to set up an array through jQuery’s .each() function, but it seems like I’m not properly doing it?

I have attributes in the html such as:

<div class="cheers" data-fname = "fname" data-lname="lname">some ish..</div><!-- going through a while loop!-->

Then I have a jquery function that does something like this

var arrayMe = [];
$(".cheers").each(function(index){
    arrayMe[index] = $(".cheers").attr('data-fname')+","+$(".cheers").attr('data-lname');
});

Then, when I try to do various alerts:

alert(arrayMe); //this gives me the fname,lname
alert(arrayMe[0]); //this gives me the first fname,lname in the array
alert(arrayMe[0][1]); //this SUPPOSED to give me the first lname, but it gives me a letter...

Answer 1

You have to use arrayMe[1] instead of arrayMe[0][1].

You get a letter, because arrayMe[0] is a string, and arrayMe[0][1] retrieves the second character of the given string. It’s equivalent to arrayMe[0].charAt(1).

If you want to build a 2D array, use:

var arrayMe = [];
$(".cheers").each(function(index){
    var $this = $(this);
    arrayMe[index] = [$this.attr('data-fname'),
                      $this.attr('data-lname')];
});
alert(arrayMe);        // Array. Shows all pairs, eg: [['fname', 'lname'], ...]
alert(arrayMe[0]);     // Array. Shows first pair, eg: ['fname', 'lname']
alert(arrayMe[0][0]);  // String. fname            eg:  'fname'
alert(arrayMe[0][1]);  // String. lname            eg:           'lname'

(I have also fixed another issue in your code by replacing $('.cheers') with $(this))