Что делать не работает удаление с массива?

Question

[grisha228]

Всем привет! Вчера задавал вопрос по поводу удаление из массива.
Есть следующий массив

[{id: "8", name: "Тест 8", price: "80000"}, {id: "9", name: "Тест  9", price: "90000"}, {id: "2", name: "Тест 2", price: "20000"}]

Так же есть функция удаления, она принимает значение с атрибутом кнопки data-id и отдает в функцию в виде id55, в консоль выводится верное значение, но при этом этот код удаляет только первый объект из массива.

Код удаления из массива:

console.log(id55);
var main = carts.findIndex(entry => (entry.id !== id55));
carts.splice(carts.indexOf(main), 1);
console.log(carts);

Comments

[drawnofmymind]

попробуй carts.filter((item)=> item !== main)

Answer 1

[Pavel Shvedov]

в main уже лежит index, который вернулся из findIndex, зачем его еще раз отдавать в indexOf, отсюда и неожидаемое поведение

Comments

[grisha228]

Убрал, но проблема не решилась, но теперь удаляет последний объект

[Pavel Shvedov]

grisha228, как убрал, покажи

[grisha228]

Pavel Shvedov,

var main = carts.findIndex(entry => (entry.id !== id55));
carts.splice(main, 1);

[Pavel Shvedov]

grisha228, не смущает, что условие поиска entry.id !== id55 ищет элемент с айдишником не равным заданному в id55, подозреваю, что нужно обратное, найти именно с таким айдишником, те ===?

[grisha228]

Pavel Shvedov, carts.findIndex(entry => (entry.id === 9)) отдает значение -1, поэтому удаляется элемент с конца

[grisha228]

Pavel Shvedov, Если carts.findIndex(entry => (entry.id !== 9)) отдает значение 1, поэтому удаляет элемент с начала

[Pavel Shvedov]

grisha228, потому, что в массиве айдишники лежат в виде строк ("9"), === строгое сравнение, без приведения типов, тоесть 9 и "9" не будут равны, поэтому не находит ни одного элемента, и отдает -1, как указано в документации в таком случае

[grisha228]

Pavel Shvedov, Спасибо огромное!

[Pavel Shvedov]

grisha228, entry.id !== 9 также не приводит типы, первый же элемент массива удовлетворяет условию, только возвращать оно должно не 1 а 0