[Vlasenko Fedor]

В нижнем варианте функция как-то громоздко выглядит

document.querySelector('.accordeon').addEventListener('click', function (event) {
    var element = event.target;
    if (element.classList.contains('btn-menu')) {
        event.preventDefault();
        element.classList.toggle('btn-menu--active');
        element.nextElementSibling.classList.toggle('sub-menu--show');
    }
});

Уменьшить код можно еще если класс active задавать li? переделав разметку css
Менять класс только одному єлементу