0

Почему false не удалилась, но "" — удалилась?

function bouncer(arr) {
  arr.forEach(function(e) {
    if(e ==  false || e == '') {
      arr.splice(arr.indexOf(e), 1);
    }
  });
  console.log(arr);
}

bouncer([7, "ate", "", false, 9]);

То, что получилось:

[7, "ate", false, 9]

1
  • 1
    вам не кажется несколько сомнительным факт удаления элементов массива внутри цикла по его перебору? обычно если такое делают, то перебирают массив с конца, что бы удаление элемента не влияло на индексы последующих перебираемых элементов. зы: чем filter() вас не устроил? зы2: индекс элемента параметром коллбэка передается, его не надо искать
    – teran
    Commented 23 июн. 2019 в 10:29

3 ответа 3

2

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

function bouncer(arr) {
  for (let i = arr.length - 1; i > 0; --i) {    
      let e = arr[i];
      if(e ==  false || e == '') {          
          arr.splice(i, 1);
      }  
    }
    
    console.log( arr )
}
bouncer([7, "ate", "", false, 9]);

Или использовать какую-либо функцию по фильтрации элементов массива, например filter

2

Потому что Вы итерируетесь по массиву, который изменяете в процессе итерации, изменение порядка следования элементов даст обратный результат, полюбуйтесь:

function bouncer(arr) {
    arr.forEach(function(e){
        if(e ===  false || e === '') {
            arr.splice(arr.indexOf(e), 1 );
        }
    });
    console.log( arr )
}
bouncer([7, "ate",  false, "", 9]);

Если Вы хотите отфильтровать массив, на это есть специальный метод:

function bouncer(arr) {
    arr=arr.filter(function(e){
        return e !==  false && e !== '' 
    });
    console.log( arr )
}
bouncer([7, "ate",  false, "", 9]);

или просто

function bouncer(arr) {
    arr = arr.filter(e => e); // оставить только все не false значения (не NaN, "", 0, false)
    console.log(arr)
}
bouncer([7, "ate",  false, "", 9]);

Ну или воспользуйтесь циклами, как в другом ответе

1

Так происходит потому что Вы удаляете из массива который перебираете, в результате элементы массива смещаются! Можете копировать массив! и удалять из него:

function bouncer(arr) {
    var newArray = arr.slice();
    arr.forEach(function(e){
        if(e ==  false || e == '') {
            newArray.splice(newArray.indexOf(e), 1 );
        }
    });
    console.log( newArray )
}
bouncer([7, "ate", "", false, 9]);

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

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