Javascript массив чистой функции

Question

Простая функция для очистки массива, если он имеет нулевые или пустые значения, так что если мы имеем:

[ 'click1', 'click2', null, '', '', 'submitForm' ] 

... он вернется:

[ 'click1', 'click2', 'submitForm' ] 

Вот мой код:

function squeakyClean(arr) { 
    for (var i = 0; i < arr.length; i++) { 
    if (arr[i] == null || arr[i] == '') { 
     arr.splice(i); 
    }; 
    }; 
    return arr; 
} 

у меня есть цикл, чтобы проверить каждое значение в массиве и, если заявление, чтобы увидеть, если она равна нулю или и пустая строка, если так я используемый метод сращивания массива для удаления значения, а затем возвращает чистый массив вне цикла for.

Он работает, если вы вводите массив без пустых строк или нулевых значений, но если я вхожу в [ 1, , 2, 3, 0, -1, 1.1 ], он возвращает [1], которого он не должен делать. Что мне здесь не хватает?

PS: Я посмотрел, как другие люди решили это использовать без использования метода loop и splice, но меня интересует, как его решить, используя эти два.

Answer 1

Ваш код не зависит от использования метода .splice, где вы также должны указать количество элементов для этого индекса для удаления.

Пример: Array.splice (index, numberOfItemsFromIndex);

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

function squeakyClean(arr) { 
    for (var i = 0; i < arr.length; i++) { 
    if (arr[i] == null || arr[i] == '') { 
     arr.splice(i, 1); 
    }; 
    }; 
    return arr; 
} 

(Splice Documentation)

Answer 2

Рассмотрите возможность использования filter. Например, следующий будет отфильтровывать null, undefined, false, +0, -0, NaN и "":

arr.filter(Boolean) 

Answer 3

Ваш код имеет две проблемы:

• Вы должны пройти 2-й аргумент splice в чтобы рассказать, сколько предметов вы хотите удалить
• Удаления с splice reindex массив. Это означает, что вы пропустите элемент при удалении предыдущего. Вы можете решить это, итерации назад.

function squeakyClean(arr) { 
    for (var i = arr.length-1; i >= 0 ; --i) 
    if (arr[i] == null || arr[i] == '') 
     arr.splice(i, 1); 
    return arr; 
} 

Answer 4

без использования сращивания или другие функции, вы можете листать все пустые значения через другой массив, как это:

function swipeArray(arr) { 
 
    var newArr = new Array(); 
 
    for (var i = 0; i < arr.length; i++) { 
 
    if (arr[i]||arr[i]===0||arr[i]===false) 
 
     newArr.push(arr[i]); 
 
    } 
 
    return newArr; 
 
} 
 

 
var arr = new Array('click1', 'click2', null, '', '', 'submitForm',0,false); 
 

 
document.getElementById("out").textContent = JSON.stringify(swipeArray(arr))

<p id="out"></p>

Edit: если вы расширяете условие if, вы можете избежать очистки 0, false или других значений в массиве.

Answer 5

alert(
 
    [ 'click1', 'click2', null, '', '', 'submitForm' ].filter(function(x){ 
 
    return !!x; // or simple `return x;` 
 
    }) 
 
)