Что лучше: проверять наличие класса у набора элементов и удалять класс, если есть или сразу удалять?

Question

[Денис Букреев]

В один момент на странице по клику происходит очень много замен классов (пара десятков элементов).

И вот какой момент. У некоторых элементов надо еще удалять старые классы.
Какая запись будет правильнее с точки зрения оптимизации работы скрипта?

$elements.each(function () {
  if ( $(this).hasClass('active') ) {
    $(this).removeClass('active')
  }
}

Или

$elements.removeClass('active');

Answer 1

[Alexander Pushkarev]

Вторая будет лучше.

Answer 2

[Nwton]

$elements.removeClass('active');
Потому что перед удалением, все равно идет проверка на наличие класса. Выходит, что в вашем первом варианте вы проверяете наличие класса 2 раза.

Answer 3

[Islam Ibakaev]

возможно так будет быстрее

var $activeElements = $('.element.active');
$activeElements.removeClass('active');

Comments

[Денис Букреев]

нет, идет повторная выборка, и так постоянно придется делать ее - жирная нагрузка