The global storearr is being reused for all test cases without being reinitialized - best to avoid global variables - turn it into a local variable of the function - you can see what’s going wrong with console.log
See how there’s Math.max() ? Well, it takes a variadic number of arguments, so for example
Math.max(1, 2) // 2
Math.max(3, -5, 1) // 3
Are totally valid but you can’t pass an array to it. What you could do is:
// a) Use apply syntax, compatible with all browsers
Math.max.apply(null, [1, 2, 3]) // 3
// b) Use the ES6 spread operator
Math.max(...[1, 2, 3]) // 3
function largestOfFour(arr) {
// You can do this!
var a = 0, max = 0, b = [];
while( a < arr.length){
for(var i = 0; i < arr[a].length; i++){
if(arr[a][i] > max){
max = arr[a][i];
}
}
b[a] = max;
max = 0;
a++;
}
return b;
}
Hope this code does find max in each sub-array and put each maximum number in a new array which is returned at last.
This is indeed a very neat solution, but sadly ES6 is not available in the FCC interpreter (afaict) so you’d need to use a regular function and ‘apply’ to pass the tests.
function largestOfFour(arr) {
return arr.map(function(xs){
return Math.max.apply(Math, xs);
});
}