JavaScript: Return largest number

Question

Can someone tell me what is wrong with this code? I am trying to return the largest numbers from four separate arrays.

function largestOfFour(arr) {
  var longList = [];
  for (var i in arr){
    var longest = 0;
    for (var x=0; x<i.length; x++){
      if (i[x] > longest){
        longest=i[x];
      }
      longList.push(longest);
    }
  }
  return longList;
}

largestOfFour([[4, 5, 1, 3], [13, 27, 18, 26], [32, 35, 37, 39], [1000, 1001, 857, 1]]);

EDIT: Lye Fish provided the answer below. Here is the new code:

function largestOfFour(arr) {
  return arr.map(Function.apply.bind(Math.max, Math));
}

largestOfFour([[4, 5, 1, 3], [13, 27, 18, 26], [32, 35, 37, 39], [1000, 1001, 857, 1]]);

Protip: Never use for(var k in ...) for an array. It does not do what you think it does. Btw, usually people would use the variable name k or key to indicate that it gives you the key instead of the value as in some other languages like Python. — Derek 朕會功夫
– Derek 朕會功夫, Commented Sep 9, 2015 at 1:58

Amadan · Accepted Answer · 2015-09-09 02:15:48Z

3

for(var i in arr) gives indices, so i[x] ends up being 0[0] instead of being arr[0][0].

Use for (var index=0; index<arr.length; index++) { i = array[index]; ... }, or (unless you are supporting the ancients) Array.each, for iterating through array items (or better yet, Array.reduce):

var arrayMax = function(arr) {
  return arr.reduce(function(a, e) { return Math.max(a, e); });
}
arrayMax(arr.map(arrayMax));
// ==> 1001

Or, better yet:

var arrayMax = function(arr) {
  return Math.max.apply(Math, arr);
}
arrayMax(arr.map(arrayMax));
// ==> 1001

EDIT: It might be I read the question too quickly; if you want the maximum of each sublist individually, Lye Fish provided it in the comments.

edited Sep 9, 2015 at 2:15

answered Sep 9, 2015 at 1:53

Amadan

200k23 gold badges252 silver badges321 bronze badges

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

3 Comments

t3dodson Over a year ago

I didn't feel like answering but using Math.max.apply was what I thought when I saw this problem. Stylistically do you prefer to explicitly pass Math as the this reference? Math.max.apply(null, arr); yields the same result.

Amadan Over a year ago

@t3dodson: Shouldn't matter, but Math.max.apply(Math, [1, 2]) is the actual equivalent of Math.max(1, 2); while there should be no surprises, puts me into good habits, I suppose.

Bryon Miller Over a year ago

Thank you for your help. Yes, Lye Fish had the perfect answer, but I also appreciate your effort to help.

webdeb · Accepted Answer · 2015-09-09 02:32:40Z

0

// sort function
var op = function(a, b) { return a < b};

// the input
arr2d = [[8,7,1,78,899],[8,7,1,78,198],[8,7,1,78,598],[8,7,1,78,398]]; 
// output
var result = [];
// process
arr2d.forEach(function(arr) {
   result.push(arr.sort(op)[0]); // sort and return the highest of the numbers
});

answered Sep 9, 2015 at 2:32

webdeb

13.3k5 gold badges32 silver badges44 bronze badges

Collectives™ on Stack Overflow

JavaScript: Return largest number

2 Answers 2

3 Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

3 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related