Корректировка JS кода ( сложение, вычитание )?

Question

[LyciferZla]

Как выглядит чекбокс

<input type="checkbox" value='2990' name='Price' id="PriceActive" onclick='Check()'>

собственно сам JS код

function Check() {
	var Price = document.getElementsByName('Price');
	var ParsSumMessage = $('.message').val();

	for (var i = 0; i < Price.length; i++) {
	 	if (Price[i].type === 'checkbox' && Price[i].checked) {
	     	rezultat = Price[i].value;
	     	ParsSum = Number(ParsSumMessage) + Number(rezultat);
  			$('.message').attr('value', ParsSum);
	 	} 
	}
}

Код исправно функционирует но нужно сделать так что бы при переводе чекбокс в режим off значения не прибавлялись с атрибута value а отнимались. Помогите пожалуйста, весь день голову уже ломаю(

Comments

[Кира]

дописать еще одну проверку if (!Price[i].checked) {...} и в ней написать вычитание вместо сложения?

Answer 1

[Сергей Соколов]

Раз используется jQuery почему бы сразу не отбирать нужные чекбоксы. И считать сумму их цен. При этом предыдущее значение цены никому уже не нужно, никаких вычитаний.

function Check() {
  var price = 0;
  var PriceBoxes = $('[name=Price][type=checkbox]:checked');
  if (PriceBoxes.length) price = $.makeArray(PriceBoxes)
    .map(el => +el.value)
    .reduce((p,c) => p+c);
    
  $('.message').val(price);
}

Comments

[LyciferZla]

Не хочет работать(

[Сергей Соколов]

LyciferZla, вы бы разобрались, что именно и почему не работает..
Обновил ответ.

[LyciferZla]

Помогите сделать дополнение к этому же коду который вы опубликовали, с левой стороны так же есть активные элементы которые выводят сумму, а с правой стороны нужно плюсовать эти чекбоксы к имеющейся сумме с левой стороны

[LyciferZla]

LyciferZla, Решение найдено, было добавлено дополнительное скрытое поле куда выводилась сумма с левой стороны потом это приплюсовывалось к результату справа

Answer 2

[hzzzzl]

как-то непонятно,
то есть если чекбокс checked, то результат = результат + 2990, а если !checked, то результат = результат - 2990 (почему так? должно же быть по логике результат = результат без изменений тогда)

или же хватит каждый раз при чеке обнулять результат

function Check() {
  var Price = document.getElementsByName('Price');
  var ParsSumMessage = $('.message').val();
  rezultat = 0;

  // а теперь перебираем все чекбоксы и считаем заново
  for (var i = 0; i < Price.length; i++) {
   	if (Price[i].type === 'checkbox' && Price[i].checked) {
       	rezultat = Price[i].value;
       	ParsSum = Number(ParsSumMessage) + Number(rezultat);
  			$('.message').attr('value', ParsSum);
   	} 
  }
}

Comments

[LyciferZla]

Вот сама логика

[Сергей Соколов]

LyciferZla, offtop это что за устройство такое офигенное?

[hzzzzl]

Сергей Соколов, офигенное по ходу устройство да ^^

[hzzzzl]

LyciferZla, предлагаю все же каждый раз при клике чекбокса всё пересчитывать как-то так

var prices = document.querySelectorAll('input[name=Price]');
var totalPrice = 0

for(let i = 0; i < prices.length; i++) {
  if(prices[i].checked) {
    totalPrice = +totalPrice + +prices[i].value	// так переводим строку в число
  }
}

document.querySelector('#div-total-price').textContent = totalPrice

[LyciferZla]

hzzzzl, при снятие галочки с чекбокса происходит вычитание суммы товара из total суммы