Как правильно удалить из массива определенные элементы по ключу?

Question

[slowkazak]

Добрый день! Вроде тривиальная задачка, но тем не менее она вогнала в некоторый ступор.
Есть массив

var arr= [{"ie":"false", "id:1"},{"ie":"true", "id:2"}];

Есть кусок кода который должен перебирать массив и если встречается ie=="false"
То удалять этот элемент из массива, если во всем массиве присутствуют все элементы с ie:"false" то, соответственно удалять их все

$.each(arr, function( key, value ) {
if (value.ie==='false') {
arr.splice(key.ie, 1);
console.log(arr.length);
}
});

Но, при переборе массива, для всех непоследних элементов с "ie":"false" возвращается вот такая вот ошибка
Uncaught TypeError: Cannot read property 'ie' of undefined
Собственно сам код https://jsfiddle.net/rfb7xLj1/
Может кто подскажет как победить?

Answer 1

[holfza]

У Вас в key индекс объекта в массиве.
А у индекса нет свойства ie
arr.splice(key, 1);

Comments

[slowkazak]

не, без разницы. Uncaught TypeError: Cannot read property 'ie' of undefined вот такой код, без излишеств https://jsfiddle.net/rfb7xLj1/

[holfza]

Создайте новый массив, и записывайте в него нужные значения. Вы удаляете из массива объект, и его длина соответственно уменьшается. А итераций у цикла будет столько, сколько было элементов в начале. И когда цикл доходит до undefined - вылетает ошибка, что у undefined нет свойства ie

[slowkazak]

действительно, splice же удаляет элемент с индексом а потом в конце цикл пытается обратиться к индексам которых уже не существует

Answer 2

[Виталий Инчин ☢]

var arr = [{"ie":"false", "id:1"},{"ie":"true", "id:2"}];

console.log(arr.filter(function(e){
 return e.ie != "false";
})); 

//filter вернет новый массив, НО если нужно делать манипуляции именно с текущим, можно пойти таким путем:
var i = -1;
while(++i<arr.length){
   if(arr[i].ie == "false") arr.splice(i--, 1);
}
console.log(arr);

Answer 3

[lyeskin]

Вы же читаете value.ie , так почему внутри if'а у вас вдруг key.ie ?

Comments

[slowkazak]

key.ie отдает индекс элемента в котором есть этот ie или я не несколько напутал?

[lyeskin]

slowkazak: во-первых, вы можете удалять так
delete arr[key];
во-вторых, у вас неверно описана переменная (попробуйте в консоль скопипастить)
в третьих, key - это только индекс(строка или число), в вашем случае это будет 0 на первой итерации и 1 на второй