Переключение классов, прохождение событий, лучшие практики - PullRequest
0 голосов
/ 31 октября 2018

Я рылся в объектах событий на консоли, пытаясь связать два элемента HTML вместе.

Я пытаюсь написать функцию (vanilla Javascript), которая будет переключать класс в другом элементе к тому, по которому щелкают.

Это путь, по которому я нашел доступ к содержанию, которое я хочу показать при клике.

function toggleActive() {
    let bodyClassList = event.path[2].children[1].classList;
    bodyClassList.add("active");
};

document.querySelector("#head").addEventListener("click", toggleActive);

HTML:

    <div id="wrap">
        <div id="head" class="headTab">
            <h3>clicked header tab</h3>
        </div>
        <div id="body" class="bodyContent">
            <ul>
                <li>li here</li>
                <li>li here</li>
                <li>li here</li>
            </ul>
        </div>
    </div>

Мне было интересно, будет ли это классифицироваться как грязный код? Или ненадежный код, если так, почему? Есть ли лучший, более широко используемый подход, с которым я не сталкивался?

Другие примеры, которые я видел, не используют этот подход, хотя он намного меньше кода - я подумал, что для этого должна быть причина.

Спасибо за любой ответ.

PS извините за дерьмовый заголовок, действительно не был уверен, какая область была более конкретной.

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