sergey-telpuk / ppd Goto Github PK

Селекторы типа #listTest ul li a и #listAnswer ul li input довольно медленные, лучше пометить необходимые елементы классом и искать по нему. Еще лучше искать не по всему документу, а в их контейнере, а еще лучше найти их 1 раз и закэшировать.

Внутри

• getNumberAnswer
• getActiveTestTitleAnswer
• getActiveTestImgAnswer
• и пр

ты постоянно ищешь одни и те же элементы в DOM, лучше закэшировать их в переменных или использовать частичное применение

Переменная с именем div — это печально.. Старайся всегда давать семантичные названия.

Почему для echo ты используешь function declaration, а для getAnswer function expression? В данном случае разницы нету, но код выглядит не однообразно, это не хорошо. Почитай плиз эту статью.

Внутри getActiveTestIndex происходит странное, лучше завести на элементе-триггере теста какой-либо служебный атрибут, пускай будет data-test-id тогда тебе не придется каждый раз фигачить о всему сету тестов и, сравнивая строки, искать совпадения, ты сразу получишь индекс.