scrollTop >= 4500
автоматически удовлетворяет условию scrollTop >= 4000
и всегда срабатывает первая ветка. Попробуйте поменять условия местами, возможно, дело в этом.$('li').removeClass('active')
вообще лучше вынести из условий, чтобы упростить код.