Как бы вы оптимизировали этот конструктор навигационных меню на основе jQuery? - PullRequest
0 голосов
/ 13 октября 2019

Я пытаюсь создать меню навигации на основе определенных типов контента на странице. Например, у меня есть страница уроков игры на гитаре, где у меня есть различные блоки упражнений.

На странице с заголовком появляется следующее упражнение:

<h3 id="This-is-an-Exercise" class="exercise" topic="25447">This is an Exercise</h3>

Темы уже отображаются на боковой панели навигациии я добавляю упражнения через это:

jQuery(document).ready(function($) {
    // get all exercises on page
    $(':header[class="exercise"]').each(function() {
        // add row to navigation menu under respective topic that contains the exercise
        var topicid = $(this).attr('topic');
        // now get topic div on left navigation sidebar        
        var topic = $('#' + topicid);
        var anchorid = $(this).attr('id');
        var namenoslashes = anchorid.replace(/-/g, " ");
        var html = "somehtml";
        // add the exercise
        topic.append(html);
    })

})

Я читал, что jquery часто очень неэффективен, если вы не выполняете свой выбор в хорошо продуманном виде. Просто я не знаю, что считается хорошо продуманным.

Есть ли в приведенном выше коде что-то, что явно нуждается в лучшей оптимизации?

Спасибо! Brian

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...