Почему ноль не добавился в массив?

Question

[alexnotonfire]

Задача - написать функцию, которая возьмет массив с числами и строками и оставит только числа.

Например, [1,'a','b',0,15] => [1,0,15]

Я написал функцию, но почему-то 0 не вернулся в новый массив. В чем дело?

function filter_list(l) {
  return l.filter(function(i){
    if (i !== String(i)) return i;
  })
}

filter_list([1,'a','b',0,15]); // 1, 15

Answer 1

[Данакт Фрост]

Предикат должен возвращать trueили false. Поскольку вы возвращаете сам элемент, он приводится к логическому типу, соответственно 0 приводится к false.

Лучше делайте так:

function filterList(list) {
  return list.filter(item => typeof item !== 'string')
}

Comments

[alexnotonfire]

Спасибо!

Answer 2

[Negwereth]

0 приводиться к false в данном случае, потому предикат и не возвращает его.

Чем вам typeof не угодил?

UPD:
Твоя задача правильно решается так:

function filterList (list) {
  return list.filter(item => !isNaN(+item));
}

console.log(filterList([1,'a','b',0,15] ))

Comments

[alexnotonfire]

А что именно заставляет 0 привестись к false?

[Negwereth]

alexnotonfire,
if (i !== String(i)) return i;

Вот эта строка. Ты говоришь - если элемент массива не равен себе как строке, верни этот элемент. Но предикат для .filter (а это и есть функция, которую ты подаешь на вход фильтра) должен вернуть или true или false, но не само значение. В предикате фильтра мы ставим условие, и если это условие выполняется, то элемент попадает в результирующий массив, и наоборот.

То есть результат выполнения предиката внутри фильтра проверяется только на true или false для каждого элемента. Именно это и заставляет 0 приводиться к false в данном случае (тут уже работают правила приведения типов).

[alexnotonfire]

Negwereth, благодарю. Надо было внимательней прочитать что нужно возвращать в коллбеке для метода filter.

Answer 3

[Павел Корнилов]

Если сильно хочется, то можно так:

console.log(filter_list([1,'a','b',0,15]).valueOf());

function filter_list(arr) {
	var out = [];
	arr.forEach(function(i) {
		if(!isNaN(+i)) out.push(+i);
	})
	return out;
}
}