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