Предпочитайте const и в крайнем случае используйте let, но никогда не используйте var
Преобразуйте все магические значения в константы
Придерживайтесь правил именования идентификаторов
Хорошо подумайте над именами идентификаторов, чтобы при прочтении кода всем было понятно их содержимое
Не делайте слишком длинных функций
Не делайте большую стопку if-выражений, вместо этого используйте массивы и объекты
Если вы пишете не на js, то переведите код на свой язык, насколько это возможно
Каждый шаг делайте отдельным коммитом, важно не переписать с нуля, а именно пошагово улучшать. На второй консультации я показал пример того, как нужно оптимизировать код
Когда закончите - сделайте пул-реквест в мой репозиторий и заполните форму, в которой укажите ссылку на свой форк с готовым заданием: https://forms.gle/qmbwSNCSbtDf6g4X7