Почитать про принципы KISS , Dry . И про строгое/не строгое сравнение в JS.
Конкретно для данного кода - этого хватит, я думаю.
А вообще, для рефакторинга было бы неплохо понимать сложность базовых алгоритмов -
ссылка ,
ещё ссылка
Ну, и хорошо знать язык. Например, не засорять глобальную область видимости var-ами в 2019 году :) Честно говоря, кажется можно вспомнить про метод indexOf и сократить эту функцию в разы