Как можно сократить код JS?

Question

[Роман Огарков]

Помогите сократить запись

$('body').on('click', function(e) {
		e.preventDefault();
		if ($(e.target).is('#device-btn') || $(e.target).is('.filter-icon') || $(e.target).is('#search-dev') || $(e.target).is('.search-li') || $(e.target).is('.search-container-c .input-row')) {
			$('#device-btn').closest('.filter-row').find('.search-container-c').addClass('db');
		} else {
			$('#device-btn').closest('.filter-row').find('.search-container-c').removeClass('db');
		};
	});

Comments

[Lynn «Кофеман»]

Можно ещё укоротить

$(e.target).is('#device-btn, .filter-icon, #search-dev, ...и т.д.')

Answer 1

[Котик Антон]

api.jquery.com/toggleClass/#toggleClass-className-state

$('body').on('click', function(e) {
    e.preventDefault();
    $('#device-btn').closest('.filter-row').find('.search-container-c').toggleClass('db', 
        $(e.target).is('#device-btn,.filter-icon,#search-dev,.search-li,.search-container-c .input-row')
    );
  });

Comments

[Роман Огарков]

Спасибо, про toggleClass был в курсе =)

Answer 2

[Денис Инешин]

Прежде всего вы должны заботиться о читабельности кода. Если сокращения ухудшают читабельность, то они не нужны. Для таких целей существуют uglifyjs.

Ваш код вполне себе читабелен. Другое дело, что тут на лицо ужасное проектирование)

Answer 3

[Alexander Litvinenko]

Итак уже ничего непонятно в

$(e.target).is('#device-btn') || $(e.target).is('.fi........

Удлините коментариями.