>>521135

Комментарии по коду:

> function addClass(el, addClass) {
Что-то тут все переусложнено. Функция addClass пишется так:

if (!hasClass(..)) {
element.className += ' ' + class;
}

> el.hasAttribute('class')
Проще использовать свойство element.className

>  if (oldClassName.indexOf(addClass) > -1) return;
неправильная проверка. Она найдет класс abc в атрибуте "abcdef abcghj", а его там нет

> var newClassName = oldClassName.slice(0, indexOfClass) +
>            oldClassName.slice(indexOfClass + removeClass.length)
Какой-то перейсложеннный код, тут проще использовать регулярки либо разбить строку регуляркой на отдельные классы.

>     for (var i=0; i < items.length; i++) {
>        items[i].addEventListener('click', function(){
События всплывают. Не надо ставить 100500 обработчиков на каждый пункт списка, достаточно одного на сам спсиок: https://learn.javascript.ru/event-bubbling

>  items[i].style['background-color'] = '';
Лучше писать style.backgroundColor, а еще лучше не писать стили в яваскрипте, а просто добавлять класс.

>   this.style['background-color'] = '#55b';
По моему то что this указывает на кликнутый элемент, не гарантируется.