Почему не срабатывает return false?

Question

[vetsmen]

const checkArrayForInteger = data => {
	function isInteger(num) {
		return (num ^ 0) === num;
	}

	data.forEach(item => {
		if(!isInteger(item)) {
			return false;
		}
	});
	return true;
}

Когда в функцию отдаю массив [1, 2, 3, 4, 'asfas'] все равно отвечает true. Почему?

Comments

[Kovalsky]

Как-то так это для меня выглядит:

const checkArrayForInteger = data => {
  data.forEach( someFunction );
  return true;
}

Потому что true возвращается всегда)

А задача вроде простая:

[1, 2, 3, 4, 'asfas'].every(item => item instanceof Number)

Или так:

[1, 2, 3, 4, 'asfas'].every( Number.isFinite )

[kulaeff]

Метод forEach просто вызывает коллбэк для каждого элемента массива, ему пофигу, что вы там возвращаете. Делайте, как говорит Никита Полевой.

Answer 1

[Astrohas]

{
    if(!isInteger(item)) {
      return false;
    }
  }

это тоже отдельная функция, и ваш return false возвращает результат ей

Comments

[vetsmen]

А как сделать, чтобы результат возвращался главной функции?

[Anton Mashletov]

vetsmen, не использовать forEach, а обычный for

Answer 2

[levchak0910]

Если самостоятельно:

const checkArrayForInteger = data => {
    if(data.length < 1) return false;

    let result = true;
    data.forEach(num => {
        if((num ^ 0) !== num) {
            result = false;
            return false;
        }
    });
    return result;
};

Или используйте функции из lodash

const checkArrayForInteger = data => data.length > 1 ? _.every(data, num => _.isInteger(num)) : false;