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

Question

[reinmaker90]

Есть массив:

tasks: [
                {
                    title: 'Make todo list',
                    completed: true
                },
                {
                    title: 'Go skydiving',
                    completed: false
                }
            ],

есть кнопка clear completed, которая вызывает функцию

clearCompleted() {
				
				this.tasks.forEach((item,index)=>{
					if(item.completed){
						this.tasks.splice(index,1)
					}
				})
            }

Вроде бы при этом условии на каждой итерации интерпретатор должен проверить блок if и если true удалить текущий элемент по индексу из массива, но почему-то он то один, то два объекта с свойство completed:true оставляет на доске и только при повторном клике удаляет их. Где ошибка?

Comments

[Денис Юрьев]

дилетантское предположение, что
при удалении массива индексы последующих элементов смещаются и если два элемента подряд с item.completed=true, то второй перепрыгнет из-за сдвига этого

Answer 1

[Дмитрий Гололобов]

Можно сделать вот так. Такой метод целиком заменит весь массив, оставив в нем только элементы со свойством completed !== true

clearCompleted() {
    this.tasks = this.tasks.filter(t => !t.completed)
}

Comments

[reinmaker90]

спс, рили так!

Answer 2

[Александр Черемхин]

о почему-то он то один, то два объекта с свойство completed:true оставляет на доске и только при повторном клике удаляет их. Где ошибка?

Вы удаляете элементы массива который перебираете.