How can I modify this very nice function to find the closest number but never higher than the input?

function closest(arr, closestTo){

var closest = Math.max.apply(null, arr);

for(var i = 0; i < arr.length; i++){
    if(arr[i] >= closestTo && arr[i] < closest) closest = arr[i];
}

return closest;
}

console.log(closest(myArray, 1234));

Any help appreciated

How can I modify this very nice function to find the closest number but never higher than the input?

function closest(arr, closestTo){

var closest = Math.max.apply(null, arr);

for(var i = 0; i < arr.length; i++){
    if(arr[i] >= closestTo && arr[i] < closest) closest = arr[i];
}

return closest;
}

console.log(closest(myArray, 1234));

Any help appreciated

• javascript
• arrays

Share Improve this question Follow asked Oct 23, 2015 at 19:20 M. El-SetM. El-Set 76344 gold badges1111 silver badges1616 bronze badges

Add a ment  | 

2 Answers 2

Sorted by: Reset to default 5

You could remove the part that checks for greater. Another method would be: remove higher values, and get max from remaining ones:

function closest(arr,val){
    return Math.max.apply(null, arr.filter(function(v){return v <= val}))
}

console.log(closest([1,22,121223],24)) // prints 22

remove the greater than parator:

if(arr[i] == closestTo && arr[i] < closest) closest = arr[i];