Как добавить сумму в массив?

Question

[Cheizer]

Друзья, есть такая непростая корзина для LP, просто набивает товары на лету и отправляет их список на почту, там единственно сложность с массивами и общей стоимостью. Так вот, вроде дело простое, но вот засада, не могу добавить число 300 к итоговой сумме (цена доставки например). Там засада в том что есть IF

if (include(Arrays, thisID)) {
.......
 } else {
    Arrays.push(thisID);
}

Вроде просто, сделал var delivery = 300;
Но вот не могу понять где ее приплюсовать, везде пробую и беда в том что считает правильно если один товар, с другим товаром еще накидывает 300. Не сильно силен просто в JS но разбираюсь немного, но с массивами не умею сильно работать, подскажите пожалуйста, для знатоков плевое дело уверен.

Как отловить финальную сумму и приплюсовать к ней 300?

Вот наглядно поднял https://codepen.io/Cheizer/pen/RdMBaP

Answer 1

[Анатолий]

Либо переписывать алгоритм, либо писать костыль, либо править вёрстку:

<span>300</span>&nbsp;руб.

Comments

[Cheizer]

Переписывать алгоритм сложно для меня, я хочу его дополнить что то типа
var delivery = 300;
и в нужном месте + delivery;

[Анатолий]

Роман Кузнецов, вот костыль:

var Arrays = new Array();	
var b = true; //вводим переменную

$('#wrap').on('click', 'li', function() {
  var baseQuantity = 10;
  var delivery = 300;
  var thisID = $(this).attr('id');
  var itemname  = $(this).find('div .name').html();
  var itemprice = +$(this).find('div .price').html();

...

  if ( b ) { // и единоразово прибавляем 300
    var prev_charges = +$('.cart-total span').html();
    prev_charges = prev_charges + delivery;
    $('.cart-total span').html(prev_charges);
    b = false;
  }
});

[Cheizer]

Анатолий, спасибо, сейчас буду внедрять, с одним товаром все ок, но вот когда второй добавляешь, не ок.

[Анатолий]

Роман Кузнецов, с моим решением не ок?

[Cheizer]

Анатолий, нет нет, я думаю это я косячу, сейчас разбираюсь :)

[Анатолий]

Роман Кузнецов, ещё не забудьте обнулить, при нажатии на кнопку "Очистить"

$('#clear').click(function() {
  b = true; //
  $(".cart-info").html("");
  $('.cart-total span').html("0");
  $('#total-hidden-charges').val(0);
  Arrays.splice(0, Arrays.length)
  return false;
});

[Cheizer]

Анатолий, понял суть, я в IF первый раз вставил ваш код, а надо после условий вообще. Понял!

[Cheizer]

Анатолий, БЛАГОДАРЮ! Дальше думаю разберусь, немного разбираюсь :)

Answer 2

[Alex]

Как мне кажется у вас слишком переусложнен алгоритм.
В массиве храните объекты в виде

[
{id: 'prod1', count: 1, price: 100, name: 'Prod 1'},
{id: 'prod2', count: 1, price: 200, name: 'Prod 2'},
{id: 'prod3', count: 1, price: 300, name: 'Prod 3'},
]

При выводе просто списка корзины просто выведите все элементы из массива по имени (Цела * количество)
При добавлении нового товара, проверьте есть ли такой и массиве по ИД. Если есть — просто увеличте количество. Если нет — добавьте.

При таком подходе, для добавления любой надбавочной стоимости, вам будет достаточно в самом начеле объявить массив с ней

var Arrays = [
{id:'order', count: 1, price: 300, name: 'Order Price'}
]

И всё

Comments

[Cheizer]

Ой это еще упрощен для вас, на самом деле сложнее, там есть и опт и розница, где * на 10 штук происходит, переделывать сложнее. Так что переделывать не вариант, я не могу понять как мне финальную сумму отловить и приплюсовать 300

[Alex]

Роман Кузнецов, На этапе когда выводите итоговую сумму. Вот когда проплюсовали всё перед выводом в ДОМ.

[Cheizer]

Александр Козак, если это сделать в одном из условий IF то 300 добавляется каждый раз при добавлении товара :)

[Alex]

Роман Кузнецов, Потому что должно быть так

var total = 0
if (include(Arrays, thisID)) {
.......
total ++
 } else {
    Arrays.push(thisID);
    total ++
}

$('span').html(total)

Вы своим алгоритмом и загнали себя в тупик

[Cheizer]

Александр Козак, да :)) он докручивался постепенно, но что поделать теперь, все устраивает, но вот только новая фишка. Надо просто как то решить, а как пока ума не приложу.

[Alex]

Роман Кузнецов,
Проведите рефакторинг и перепишите. Ибо через месяц придётся докрутить ещё фишечек и уже никто не поможет

[Alex]

Роман Кузнецов,

[Cheizer]

Александр Козак, это точно