Compare Numbers in Javascript array to get largest number

Question

I seem to have the right syntax but i keep getting a NaN value, i believe it has something to do with the propagation of the array, please could you check the code and see where i am going wrong.

var _array = new Array();

var _first = 1;
var _second = 4;

$('#con > div').each(function(index){

    var _data = $(this).css('left').split('px')[0];
    var _class = $(this).attr('class').split(' ')[0];

    _array[_first] = [_data];
    //_array[_second] = _class;

    _first++;
    _second++;

    if(index == 2){
        _first = 1;
        _second = 4;

        for(var i = 0; i < _array.length; i++)
        console.log(_array[i]);

        var large = Math.max.apply(Math, _array);
        console.log(large);

    }
});

Thanks

http://jsfiddle.net/LTMbr/1/

Note that your variable _array is not an array of numbers, it is an array of arrays because you have square brackets around _data when you say _array[_first] = [_data]; (unless that was just a typo in your question). — nnnnnn
– nnnnnn, Commented Oct 4, 2011 at 14:01

itsmatt · Accepted Answer · 2011-10-04 13:38:42Z

2

So, I modified it a bit to see - http://jsfiddle.net/LTMbr/4/ and your _array[0] is undefined. So that's why your call to Math.max is returning NaN.

answered Oct 4, 2011 at 13:38

itsmatt

31.5k11 gold badges103 silver badges165 bronze badges

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

Comments

meouw · Accepted Answer · 2011-10-04 13:41:11Z

2

The problem is that you are populating your array starting at index 1. If any of the arguments to Math.max cannot be converted to a number, NaN is returned
This is the case because _array[0] is undefined

answered Oct 4, 2011 at 13:41

meouw

42.1k11 gold badges55 silver badges69 bronze badges

Collectives™ on Stack Overflow

Compare Numbers in Javascript array to get largest number

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related