Может кто-нибудь помочь с решением задачи на JavaScript?

Question

[Veqtor]

Всем привет. Я пытаюсь освоить JavaScript самостоятельно через учебник code.mu/ru/javascript/book/prime. Имеется там вот такая задача: "Сделайте функцию delElem, которая параметрами будет принимать значение и массив и удалять из массива все элементы с таким значением." Я её решаю так:

function delElem (num, arr) {
	for (let elem of arr) {
		if (num == elem) {
			arr.splice(elem, 1);
		}
	}
	return(arr);
}

но почему-то это не верно. Подсмотрел решение этой задачи в интернете, и по сути там также решают но через цикл for, а не for-of как я. Объясните кто-нибудь почему моё решение не работает.

Answer 1

[Rsa97]

Во-первых, for ... of перебирает значения элементов массива, а не индексы, поэтому splice использовать нельзя.
Во-вторых, модифицировать массив, по которому идёт перебор, вообще не лучшая идея. Если делать такое, то делать надо умеючи и аккуратно.
В-третьих, для такой цели есть метод массива filter.

Comments

[Veqtor]

Ага. Понял. Спасибо. Тогда может ещё подскажите, а методом delete через for-of решаема эта задачка или нет?

[Rsa97]

Veqtor, Нет. Для for ... of подойдёт только создание нового массива и вставка в него нужных элементов исходного.

[WapSter]

Veqtor,

а методом delete

может стоит почитать, что делает "метод" delete (это оператор) с массивом

[Veqtor]

WapSter, спасибо. Я делал оператором delete (не методом) через цикл for, но получался разряжённый массив. Попробую filter. Спасибо ещё раз.

[Veqtor]

Rsa97, спасибо. Вы все предельно понятно объяснили. Вам бы преподавать. Реально я догнал.

Answer 2

[WapSter]

const arr = [1, 2, 3, 4, 1, 6, 7];

const delEl = (val, arr) => {
  let index = arr.indexOf(val);
  if(~index) {
    arr.splice(index, 1);
    return delEl(val, arr);
  }
  return arr;
}

console.log(delEl(1, arr)); //[2,3,4,6,7]

Гораздо проще применить метод filter