Нет, они написали всё верно, собака порылась именно в функциональном подходе - чтобы пройти по массиву и при этом чтобы каждый проход(читай вызов функции) не имел состояния (которым у нас в цикле for является текущий индекс) - приходится строить цепочку вызовов функций, и изголяться чтобы прокинуть индекс по этой цепочке. Если грубо - .filter это не zero-cost абстракция, но многие этого не понимают. Вообще всегда в целом за иммутабельность и функциональщину всегда платишь памятью.