Каким образом можно отрефакторить такой код?

Question

[game802]

Добрый день, уважаемые знатоки. Каким образом можно отрефакторить этот код так, чтобы он выглядел более лаконичнее?

$('.side input[type="checkbox"]').on('click', function() {
  if ($('.side input[type="checkbox"]').is(':checked') ) {
    $('.side .auto_recharge-text').addClass('active');
  } else {
    $('.side .auto_recharge-text').removeClass('active');
  }
});

$('.side .auto_recharge-text').on('click', function() {
  if ($('.side input[type="checkbox"]').is(':checked') ) {
    $('.side input[type="checkbox"]').prop("checked", false);
    $('.side .auto_recharge-text').removeClass('active');
  } else {
    $('.side input[type="checkbox"]').prop("checked", true);
    $('.side .auto_recharge-text').addClass('active');
  }
});

Большое спасибо за помощь!

Comments

[Модератор]

game802, за порчу вопросов будет больно.

Answer 1

[Stalker_RED]

$('.side input[type="checkbox"]').on('click', function() {
  $('.side .auto_recharge-text').toggleClass('active', $(this).is(':checked'));
});

$('.side .auto_recharge-text').on('click', function() {
  var isChecked = $('.side input[type="checkbox"]').is(':checked');
  $(this).toggleClass('active', !isChecked);
  $('.side input[type="checkbox"]').prop("checked", !isChecked);
});

Answer 2

[string001]

Я сам новичок, но мне кажется слишком много однотипных обращений
.side input[type="checkbox"]

Comments

[game802]

безусловно.. это сильно снижает читаемость кода!