Почему splice не удаляет?

Question

[syb2024]

если выводить в кончоль data3.splice(index,1) он показывает что выбран 4 индекс но не удаляет его

const data3 = [
    { name: "Андрей", age: 31 },
    { name: "Валерия", age: 31 },
    { name: "Егор", age: 25 },
    { name: "София", age: 29 },
    { name: "Антон", age: 40 },
  ];
    data3.forEach((item, index)=>{
        
        let maxAge = Math.max(...data3.map((n) => n.age));

        if(item.age == maxAge){
            data3.splice(index,1)
        }
        console.log(item)
       });

Comments

[Ivan Ustûžanin]

если выводить в кончоль data3.splice(index,1) он показывает что выбран 4 индекс но не удаляет его

где не удаляет?

и вообще, что нужно сделать? т.к. тут ИМХО filter() лучше подойдёт

[syb2024]

Ivan Ustûžanin, мне надо чтоб удалял age 40 , потом после удаления самый больше возраст будет 31 его тоже надо удалить, до тех пор пока не останется пустой массив

[Ivan Ustûžanin]

syb2024, так он его и удаляет, но в приведённом коде нет вывода массива после forEach()

а вообще, если нужно сделать массив пустым, то data3.splice(0, data3.length) сделает это на ура )
т.е. ещё раз: какая изначальная задача, т.к. это не задача, а какая-то странная и непонятная попытка её решить
но сразу скажу, что за один проход по массиву такой алгоритм не справится

[WbICHA]

Ivan Ustûžanin, есть метод по проще: arr.length = 0. Но да, это не задача.

[Wispik]

В чем задача-то. Как вариант, отсортируй объекты в массиве по возрасту и потом бери их с краю и удаляй

[iljaGolubev]

Потому что алгоритм у вас такой - не удалять всё ... а так-то "splice удаляет"
вот проверьте

const data3 = [
    { name: "Андрей", age: 31 },
    { name: "Валерия", age: 31 },
    { name: "Егор", age: 25 },
    { name: "София", age: 29 },
    { name: "Антон", age: 40 },
  ];
console.log(data3.length)
    data3.forEach((item, index)=>{
        
        let maxAge = Math.max(...data3.map((n) => n.age));

        if(item.age == maxAge){
          console.log(item)
            data3.splice(index,1)
        }
        
       });

console.log(data3.length)

[ProjectSoft]

syb2024,

мне надо чтоб удалял age 40 , потом после удаления самый больше возраст будет 31 его тоже надо удалить, до тех пор пока не останется пустой массив

forEach здесь плохо подходит, так как массив уменьшается.

const data3 = [
	{
		name: "Андрей",
		age: 31
	},
	{
		name: "Валерия",
		age: 31
	},
	{
		name: "Егор",
		age: 25
	},
	{
		name: "София",
		age: 29
	},
	{
		name: "Антон",
		age: 40
	},
];
while(data3.length) {
	data3.splice(data3.findIndex((n) => {
		return (n.age === Math.max(...data3.map((n) => n.age))) ? !0 : !1;
	}), 1);
}

А что сразу пустой массив не поставить? Честно говоря, то, что я предоставил - полная ересь. С задачей справляется, но для чего?