4

Есть массив и ф-ция, нужно удалять элементы с массива, я понимаю что delete удаляет элемент но после него образовывается «дырка», как его полностью удалить?

var myArr = [1, 2, 3, 6, 7, 13, 349, 3483, 5432];
var doubledOdds = onlyDoubledOdds(myArr);
console.log(doubledOdds.length);
console.log(doubledOdds);
if (doubledOdds === myArr) {
  console.log('WTF?!');
} else {
  console.log('Ok');
}

function onlyDoubledOdds(sourceArray) {
  var result = sourceArray;
  for (var i = 0; i < result.length; i++) {
    if (i % 2 == 0) {
      delete result[i];
    } else {
      result[i] *= 2;
    }
  }
  return result;
}

а вообще оно выводит

9
[ undefined, 4, undefined, 12, undefined, 26, undefined, 6966, undefined ]
WTF?!

А нужно:
6
[ 2, 6, 14, 26, 698, 6966 ]
Ok

8
  • delete оператор удаляет из памяти, а не из массива. из массива удаляют процедуры, относящиеся к классу массива (array), например pop
    – nick_n_a
    11 июл 2016 в 9:09
  • а что должна делать приведенная функция?
    – Grundy
    11 июл 2016 в 9:11
  • Про splice знаю, не могу понять как правильно записать... if (i%2 == 0) { result.splice[i]; }
    – Webitek
    11 июл 2016 в 9:20
  • @Webitek, а по какому принципу построен массив А нужно: 6 [ 2, 6, 14, 26, 698, 6966 ]?
    – Grundy
    11 июл 2016 в 9:24
  • 1
    а почему он должен выбирать число 13? есть описание словами как из исходного массива получить результат?
    – Grundy
    11 июл 2016 в 10:33

3 ответа 3

15

Для удаления можно использовать специальные методы, такие как

  • shift - удалит из начала
  • pop - с конца
  • splice - из середины

C помощью splice - как раз можно удалять элементы, вставлять элементы, заменять элементы – по очереди и одновременно.

Синтаксис таков:
arr.splice(start[, countDelete[, elemInsert1, ..., elemInsertN]])

т.е. удалить countDelete элементов, начиная с номера start, а затем вставить elemInsert1, ..., elemInsertN на их место. Если countDelete не указано, то отрежется все, начиная со start и до конца. Возвращает массив из удалённых элементов.

Пример с удалением выглядит так:

var arr = ["Привет", "тебе", "Миша"];
arr.splice(1, 1); // начиная с позиции 1, удалить 1 элемент
alert( arr ); //  осталось ["Привет", "Миша"]

а замещением:

var arr = ["Привет", "тебе", "мальчик", "Миша"];
// удалить 3 первых элемента и добавить другие вместо них
arr.splice(0, 3, "Здароф!")
alert( arr ) // теперь ["Здароф!", "Миша"]
0
5

При применении оператора delete не изменяется свойство length массива, поэтому при использовании длины в цикле - разницы не будет.

Функция onlyDoubledOdds делает что-то непонятное, в зависимости от индекса элемента, причем меняет исходный массив, поэтому при сравнении возвращенное значение закономерно равно переданному, а не "WTF"

Если нужно отфильтровать массив - можно воспользоваться функцией filter,
если нужно получить массив на основе существующего, но с другими элементами - можно воспользоваться функцией map,
если хочется за один проход, можно воспользоваться функцией reduce

Например:

var myArr = [1, 2, 3, 6, 7, 13, 349, 3483, 5432];
var doubledOddsMapFilter = onlyDoubledOddsMapFilter(myArr);
console.log('doubledOddsMapFilter.length', doubledOddsMapFilter.length);
console.log('doubledOddsMapFilter', doubledOddsMapFilter);
if (doubledOddsMapFilter === myArr) {
  console.log('WTF?!');
} else {
  console.log('Ok');
}

var doubledOddsReduce = onlyDoubledOddsReduce(myArr);
console.log('doubledOddsReduce.length', doubledOddsReduce.length);
console.log('doubledOddsReduce', doubledOddsReduce);
if (doubledOddsReduce === myArr) {
  console.log('WTF?!');
} else {
  console.log('Ok');
}

function onlyDoubledOddsMapFilter(sourceArray) {
  return sourceArray.filter((el, i) => i % 2 != 0).map(el => el * 2);
}

function onlyDoubledOddsReduce(sourceArray) {
  return sourceArray.reduce((acc, el, i) => {
    if (i % 2 != 0) acc.push(el * 2);
    return acc;
  }, []);
}

UPDATE: так как начальное решение в вопросе было с ошибкой, то примеры выше ориентирующиеся на пример в вопросе так же ошибочны, проверяться должен сам элемент а не индекс.

var myArr = [1, 2, 3, 6, 7, 13, 349, 3483, 5432];
var doubledOddsMapFilter = onlyDoubledOddsMapFilter(myArr);
console.log('doubledOddsMapFilter.length', doubledOddsMapFilter.length);
console.log('doubledOddsMapFilter', doubledOddsMapFilter);
if (doubledOddsMapFilter === myArr) {
  console.log('WTF?!');
} else {
  console.log('Ok');
}

var doubledOddsReduce = onlyDoubledOddsReduce(myArr);
console.log('doubledOddsReduce.length', doubledOddsReduce.length);
console.log('doubledOddsReduce', doubledOddsReduce);
if (doubledOddsReduce === myArr) {
  console.log('WTF?!');
} else {
  console.log('Ok');
}

function onlyDoubledOddsMapFilter(sourceArray) {
  return sourceArray.filter(el => el % 2 != 0).map(el => el * 2);
}

function onlyDoubledOddsReduce(sourceArray) {
  return sourceArray.reduce((acc, el) => {
    if (el % 2 != 0) acc.push(el * 2);
    return acc;
  }, []);
}

3
  • Спасибо, не совсем все понял...буду сидеть разбираться
    – Webitek
    11 июл 2016 в 9:29
  • @Webitek, где конкретно не понятно? в примере с filter-map - сначала отбираются элементы у которых индекс нечетный(с помощью функции filter), затем по полученному массиву строится новый у которого элементы удвоены по сравнению с исходным массивом
    – Grundy
    11 июл 2016 в 9:31
  • @Webitek, обновил ответ
    – Grundy
    11 июл 2016 в 11:38
0
var myArr = [1, 2, 3, 6, 7, 13, 349, 3483, 5432];
myArr = myArr.map(el => el % 2 !== 0 ? el * 2 : el);
myArr = myArr.filter(el => el % 2 === 0 ? true : false);
2
  • пожалуйста, постарайтесь оставлять чуть более развёрнутые ответы. дополнить ответ можно, нажав править 1 июл 2020 в 8:43
  • после первого map в массиве будут только четные числа, поэтому если убрать filter - ничего не изменится.
    – Grundy
    1 июл 2020 в 9:11

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.