Какие замечания можете привести по представленным функциям

[Антон Крамолов]

function hasClass(node, cls) {
    return node.className != '' ? (' ' + node.className + ' ').indexOf(' ' + cls + ' ') > -1 : false;
}

function addClass(node, cls) {
    if (!hasClass(node, cls)) {
        node.className =  node.className.trim() + ' ' + cls;
        return true;
    }
    return false;
}

function removeClass(node, cls) {
    if (hasClass(node, cls)) {
        node.className = ((' ' + node.className + ' ').replace(' ' + cls + ' ', ' ')).trim();
        return true;
    }
    return false;
}

[Антон Крамолов]

Например первое можно переписать так:

function hasClass(node, cls) {
    return !node.className ? false : (' ' + node.className + ' ').indexOf(' ' + cls + ' ') > -1;
}

[cmygeHm]

Для hasClass
если класс у элемента всего один => "superClass", то hasClass не найдет в нем вхождение ' ' + cls + ' '. Нужно проверять из скольки слов состоит класс. Даже если он состоит из трех - надо учесть, что он может быть слева или справа. Yeah?

[cmygeHm]

function hasClass(node, cls) {
    return !node.className ? false : (' ' + node.className + ' ').indexOf(' ' + cls + ' ') > -1;
}

Да, клево подметил

[cmygeHm]

Блин! Втупил, ты ведь для этого и прибавил по пробелу слева и справа... Извини.

[cmygeHm]

Для addClass не вижу смысла тримить node.className.trim(), потому что во первых ты чаще всего будешь присылать без пробелов, во вторых если пробелов будет два и более, класс все равно добавится, и будет нормально работать.

[Антон Крамолов]

наверное

[Антон Крамолов]

честно не разу даже не проверял

[cmygeHm]

честно не разу даже не проверял

Я проверил.

[Антон Крамолов]

document.querySelector && document.querySelectorAll
какую обертку для них лучше всего написать?