Function addClass() Функция будто игнорирует значения true or false

Question

[Alexander]

function hasClass(obj, name) {
  obj = ge(obj);
  return obj && (new RegExp('(\\s|^)' + name + '(\\s|$)')).test(obj.className);
}
function addClass(obj, name) {
	var obj = ge(obj);
  if (!hasClass(obj, name)) {
    obj.className = (obj.className ? obj.className + ' ' : '') + name;
  }
}

Проблема в том, что addClass() будто игнорирует логическое значение функции hasClass и добавляет классы подряд. В чем может быть проблема?

Comments

[Mikhail Osher]

Я тоже брал утилиты из VK, только при код у меня работал. Очень странно.

Answer 1

[Илья Милошевич]

Может быть переписать вот так:

function hasClass(selector, name) {
  var obj = ge(selector);

  return obj && (new RegExp('(\\s|^)' + name + '(\\s|$)')).test(obj.className);
}

function addClass(selector, name) {
  var obj = ge(selector);

  if ( !hasClass(selector, name) ) {
    obj.className = (obj.className ? obj.className + ' ' : '') + name;
  }
}

Answer 2

[personaljs]

как бы у тебя работает// Первый раз показал что нет класа, далее ты добавил класс, он показал что есть.

Почему не работает у вас: вы получили в опциях obj и переопределяете его, а после чего проверяете наличие класса у изменившегося значения obj

Answer 3

[Pozadi]

Вопрос в том что делает ge(), вы пишите obj = ge(obj), после чего передаете новый obj в hasClass(). А в hasClass() этот obj опять проходит через ge(). Думаю проблема в этом.

Еще вы забыли var перед obj во второй строке, и name перед добавлением в регексп нужно заэскейпить.

Функция для эскейпа (stackoverflow.com/questions/3446170/escape-string-...):

function escapeRegExp(str) {
  return str.replace(/[\-\[\]\/\{\}\(\)\*\+\?\.\\\^\$\|]/g, "\\$&");
}

Answer 4

[Николай Васильчук]

Может лучше использовать classList и polyfill к нему?