История об одном врестале, или что не так с forEach в js?

Question

[KononovD]

kononovD.github.io/studio gift/index.html

Добрый вечер, знатоки.
Если коротко, то я пытаюсь складывать "data-id" кнопок-товаров в объект в js(типо корзина) и потом их удалять, если пользователь удаляет товар из корзины, очевидно.
На этапе "добавлять товар в объект" - все хорошо

А вот когда я начинаю удалять один товар - удаляются все товары с таким же data-id а потом еще и в nan скатывается эта ячейка..

Нагляднее на гитхабе в консоли
kononovD.github.io/studio gift/index.html

console.log стоит после добавления товара в объект и после удаления из него.

И что делать я не знаю. Как быть?

добавление в корзину:
удаление из нее :

Comments

[Виталий Киренков]

Вы бы код показали... Если у вас только id, то зачем объект? Почему не просто массив?

[KononovD]

Виталий Киренков, скрины приложил

[ProjectSoft]

KononovD, удалять из массива надо через метод splice
delete - удаляет значение ключа, но не сам элемент массива.

[Виталий Киренков]

1. У вас ИД точно одного типа? Есть подозрение, что там сравнивается стринга и нумбер.
2. Зачем вы деаете инкремент и дикремент?

[KononovD]

ProjectSoft, логика была в том, что бы basket потом отправить в php и там взять из него ключ товара и получить доступ к этому же товару, только уже в json/phpMyAdmin
и тут мне не нужен ключ, у которого значение будет 0, поэтому delete

Или я написал не так, как придумал?

[KononovD]

Виталий Киренков,
1. да, конечно
2. что бы в объект увеличить количество конкретного товара и уменьшить, соответственно

[ProjectSoft]

KononovD, тогда вообще не понимаю зачем это делать на клиенте. Переложите на бек.
Добавил, увеличил, удалил - всё запросы к корзине и ответ в нужном вам формате.

[Виталий Киренков]

KononovD, я канеш возможно туплю, но в выражении basket[id]++ - вы наверно хотите увеличить кол-во товара, то есть изменить значение, а при такой записи вы пытаетесь увеличить ключ.

[Виталий Киренков]

А так, согласен с ProjectSoft, стоит унести это на бэк.

[KononovD]

ProjectSoft, сказывается недостаток опыта) на "беке" я могу, разве что mail и общение с бд написать

[KononovD]

Виталий Киренков, basket[id]++ , а не basket[id++] ))

[KononovD]

Виталий Киренков,
тадаам

[KononovD]

Виталий Киренков,
ProjectSoft,
lamer350, придумал следующее: навешивать событие на кнопку, при создании. и все заработало как я хотел)
всем спасибо)

[Модератор]

3.8. Использовать для демонстрации фрагментов кода только специальный тег < code> или специальные демонстрационные ресурсы (например, JsFiddle - https://jsfiddle.net/). Размещение фрагментов кода в виде изображений запрещено.

[KononovD]

Модератор, Понял, больше не буду

Answer 1

[lamer350]

Он складывается в NaN потому как вы полностью удаляете значение delete basket[id], а не обнуляете его. И при этом удаляете просто количество, а не заказ полностью из массива.
Более того у вас там еще и множатся обработчики при добавлении в корзину, например если добавить 4 товара одинаковых, то на первый товар в корзине 4 обработчика повиснет и если его удалить - то из массива для этого ID удаляются все количество, а 3 товара должны еще быть...
Ну и само собой попробуйте добавить 4 товара и удалять даже их с конца, увидите что у вас на каждом товара вешается по несколько событий на удаление при клике и удалив второй товар с конца - у вас уже будет -3...

Comments

[KononovD]

Спасибо, почему-то думал, что одинаковые обработчики заменяются