Как удалить все четные элементы массива?

Question

[Ilya]

Подскажите что делаю не так? Нужно удалить все четные элементы массива, но при удалении индексы сдвигаются, как быть?

var arr = ['а','б','в','г', 'д'];

for (var i = 0; i <= arr.length; i++) {
   if (i % 2 == 1) {
          arr.splice(i,1);
      }
 }

Comments

[Сергей Соколов]

двигаться справа налево : )

Answer 1

[Stalker_RED]

var result = arr.filter((e,i)=>!(i%2))
https://jsfiddle.net/h0tj0mmL/

Comments

[frankie_peddler]

Доброго времени суток,
Подскажите пожалуйста, что происходит в стрелочной функции?
Перебираются индексы элементов и если остаток от деления не равен нулю(явно не указано, но так понимаю, что 0 в js это false), так?
Тогда почему, если явно объявлять в методе .filter индекс элемента массива с помощью .indexOf, как тут например,
arr.filter(item => arr.indexOf(item)%2 == 0);
итоговый массив будет сформирован неверно

Answer 2

[Сергей Соколов]

двигаться справа налево:

var i, arr = ['а','б','в','г','д'];

for( i = arr.length - 1; i >= 0; i--) if( i % 2 === 1) arr.splice(i,1);
// а,в,д

Comments

[EVGENIY ARAPOV]

Здравствуйте,а почему вы перебор делаете с последнего элемента?

[Сергей Соколов]

EVGENIY ARAPOV, идея была такая: чтобы сдвиг индексов и уменьшение длины не влияли.

Конечно, решение далеко не идеальное.

Answer 3

[Islam Ibakaev]

если императивно, то так

function isEven(num) {
 	return num % 2 === 0;
 }
 
function dropEvenElems(list) {
	var result = [], 
			len = list.length, i;
	
	for (i = 0; i < len; i += 1) {
		isEven(i) && result.push(list[i]);
	}
	
	return result;
}

var arr = ['а','б','в','г', 'д'];

dropEvenElems(arr);

Answer 4

[nhzch]

В чем проблема непонятно? Сдвиг сплайса всегда на единицу:
m-массив, n-число элементов

for(i=0;i<n/2;i++) m.splice(i,1);

Answer 5

[sweet_coder]

arr = arr.filter(item => arr.indexOf(item) % 2 === 0);

Comments

[Сергей Соколов]

плохой вариант, т.к. ошибётся c indexof() если в массиве будут повторяющиеся значения:

var arr = 'a,b,b,a,a,b,b'.split(',');
arr = arr.filter(item => arr.indexOf(item) % 2 === 0); 
// a,a,a

[Сергей Соколов]

Надо использовать второй параметр, передаваемый в callback – это индекс элемента в исходном массиве:

var arr = 'a,b,b,a,a,b,b'.split(',');
arr = arr.filter( (e,i) => i % 2 === 0); 
//a,b,a,b

[sweet_coder]

Сергей Соколов: Да спасибо, к моему стыду не знал что в этой функции ещё и индекс доступен.