Как удалить элементы массива внутри цикла JS?

Question

[Александра]

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

var invItems = [
    {
        nameItem: 'Предмет_1'
    },
  {
        nameItem: 'Тест'
  },
    {
        nameItem: 'Предмет_2'
    }
];

var arr = [0,2];

for (var i = 0; i < arr.length; i++) {
  var index = arr[i];
  invItems.splice(index, 1);
}
console.log(invItems);

Comments

[i1yas]

Так не получится удалить, у вас индексация invItems меняется после удаления первого элемента.
Это вообще плохая идея так делать.

[Иван Дмитриев]

А откуда вы берете массив индексов? Сами придумываете ? Попробуйте объект вместо массива. На первый взгляд это решит вашу проблему , но лишит вас особенностей массива . Опишите более точно область применения. Ну или используйте костыль , считая сколько элементов вы удалили и минусуя это значение от ключа

[Александра]

Иван Дмитриев, ну странные же у вас вопросы) нет конечно , я не придумываю массив, у меня большая задача, я вырвала из большого контекста маленький кусок, чтобы выложить сюда, или мне весь проект надо было сюда выложить? Куратор тега уже ответил на мой вопрос коротко и по сущетсву

[Иван Дмитриев]

Александра, Блин, на то он и куратор =) Слишком глубоко попытался вникнуть вопрос , а решения как всегда просты и на поверхности =)

[Александра]

Иван Дмитриев, Пашенька, хорошо принято) объясняю - массив с индексами не статичный, данные туда попадают в зависимости от определенных условий, элементов может быть два или сколько угодно. Функция принимает строку, которая потом идет на проверку, есть ли такая строка в свойстве объекта, если есть возвращаем индекс объекта в массиве, и из этих индексов формируется тот самый массив, который был принят за статичный. Потом по этим индексам проверяется еще одно свойство, отвечающее за количество предметов, если количество ноль нужно удалить этот объект из массива

Answer 1

[0xD34F]

идет смещение индексов, но как это обойти, уже не знаю

Можно перебирать массив индексов с конца:

for (let i = arr.length; i--;) {
  invItems.splice(arr[i], 1);
}

Или развернуть его:

for (const n of [...arr].reverse()) {
  invItems.splice(n, 1);
}

Или исправлять индексы - вычитать из них количество уже удалённых элементов, это будет индекс индекса:

for (const [ i, n ] of arr.entries()) {
  invItems.splice(n - i, 1);
}

Или, есть вариант ограничить применение метода splice единственным вызовом - сначала собираем подходящие элементы, а затем полностью заменяем ими содержимое массива с данными (применимо и в том случае, если массив индексов не отсортирован):

invItems.splice(
  0,
  invItems.length,
  ...invItems.filter((n, i) => !arr.includes(i))
);

Или, долой splice, будем использовать два счётчика - один для массива данных, другой для массива индексов. Второй одновременно будет счётчиком удаляемых элементов. Перебираем массив данных, копируем элементы влево на столько позиций, сколько в данный момент элементов должно быть удалено, увеличиваем счётчик удалённых элементов если текущий индекс массива с данными равен текущему элементу массива индексов, после обхода массива с данными удаляем посчитанное количество элементов с конца:

invItems.length -=
invItems.reduce((acc, n, i, a) => (
  a[i - acc] = n,
  acc + (i === arr[acc])
), 0);

Comments

[Александра]

спасибо))

Answer 2

[Пашенька]

var invItems = [
    {
        nameItem: 'Предмет_1'
    },
  {
        nameItem: 'Тест'
  },
    {
        nameItem: 'Предмет_2'
    }
];

var arr = [0,2],
    arrLength = invItems.length;

for (var i = 0; i < arrLength; i++) {
   for (var j = 0; i < arr.length; j++) {
       if (i === j) {
           invItems.splice(i, 1);
           break;
       }
   }
  
}
console.log(invItems);