Потому что Вы итерируетесь по массиву, который изменяете в процессе итерации, изменение порядка следования элементов даст обратный результат, полюбуйтесь:
function bouncer(arr) {
arr.forEach(function(e){
if(e === false || e === '') {
arr.splice(arr.indexOf(e), 1 );
}
});
console.log( arr )
}
bouncer([7, "ate", false, "", 9]);
Если Вы хотите отфильтровать массив, на это есть специальный метод:
function bouncer(arr) {
arr=arr.filter(function(e){
return e !== false && e !== ''
});
console.log( arr )
}
bouncer([7, "ate", false, "", 9]);
или просто
function bouncer(arr) {
arr = arr.filter(e => e); // оставить только все не false значения (не NaN, "", 0, false)
console.log(arr)
}
bouncer([7, "ate", false, "", 9]);
Ну или воспользуйтесь циклами, как в другом ответе
filter()
вас не устроил? зы2: индекс элемента параметром коллбэка передается, его не надо искать