Стоит ли делать такую проверку?

Question

[Nwton]

Интересно узнать, какой вариант оптимальный в плане нагрузки на браузер:
1) Есть div.
2) В ходе работы функции, ему может быть присвоен класс, а может и не быть.
3) В конце работы функции идет очистка div от ненужных классов.

Стоит ли перед очисткой проверять есть ли у div определенный класс и удалять его только, если он есть? Или лучше сразу сделать classList.remove без проверки?

if(dom.classList.contains(class))dom.classList.remove(class);
или
dom.classList.remove(class);

Answer 1

[Aves]

Если действие только в удалении класса, то такая проверка бессмысленна — remove сам проверяет наличие класса, удаляет в случае наличия и ничего не делает в случае отсутствия. Дополнительная проверка является лишней нагрузкой, хотя и очень незначительной, вряд ли заметно отразится на производительности.

Answer 2

[xmoonlight]

Проверку лучше делать всегда, т.к. иначе - код будет неуправляемым (try-catch-exception).

Comments

[malbaron]

А если делать всегда - код станет нечитаемым.

[gleendo]

xmoonlight remove() не кидает исключения. Если класс есть, то удалит, если класса нет, то ничего не сделает. Так что проверять все подряд наоборот сделает код плохо читаемым, следовательно неуправляемым. (try-catch-exception) - стоило бы написать (try - catch - finally)

[xmoonlight]

evg_: да. согласен полностью! (я просто изначально думал, что кидает исключение).