Функция должна проверять, какие числа четные, объясните почему не работает?

Question

[Karli]

let arr = [1, 2, 3, 4, 5, 6, 7, 8, 9, 10];

function isTrue(arr) {
  for (let elem of arr) {
    if (elem % 2 === 1) {
      return false;
    }
  }

  return true;
}

func(arr);

Comments

[Ирина]

нужно на равенству 0 проверять (значит делится без остатка), а у вас elem % 2 === 1

[Михаил Р.]

Ирина,

function isTrue(arr)

func(arr);

И еще бы к нужной функции обратиться, тогда ваще норм будет!

[Сергей delphinpro]

Потому что первый же return прерывает выполнение функции.
По сути ваша функция проверяет являются ли все числа в массиве четными.
Если в массиве встретится хотя бы одно нечетное число, то она вернет false.

[Сергей delphinpro]

Ирина, тогда функция будет проверять являются ли все числа в массиве нечетными

[Сергей delphinpro]

Karli,

Функция должна проверять, какие числа четные

ваш вопрос подразумевает проверку каждого числа. А функция возвращает булево значение (да/нет). Как можно ответить на несколько проверок одним утверждением "да" или "нет"?
Вы бы поподробнее объяснили что вам нужно и в каком виде требуется результат.

Answer 1

[otdameskapizm]

Ваша функция не проверяет, какие числа четные - она отвечает на вопрос, все ли числа массива четные
Если нужно вывести четные числа, то попробуйте хотя бы так:

const isEven = (arr) => {
  for (const num of arr) {
    if (num % 2 === 0) {
      console.log(num);
    }
  }
};

Comments

[Karli]

не так сформулировала вопрос, она и должна проверять, то почему не работает?

[otdameskapizm]

Так Вы вызываете не ту функцию. Вам надо вызывать isTrue, а у Вас в коде вызов func(arr)

[Karli]

otdameskapizm, выдает ошибку

[otdameskapizm]

Поставьте проверку условия, не (elem % 2 === 1), а (elem % 2 !== 0). В Вашем случае происходит проверка наличия числа в массиве, которое без остатка делится на 1. А нужно проверять, чтобы числа без остатка делились на 0. Если после этой проверки выскочит false, то значит, что у Вас в массиве есть число, которое не делиться на 0 без остатка - нечетное.

function isTrue(arr) {
  for (let elem of arr) {
    if (elem % 2 !== 0) {
      return false;
    }
  }
  return true;
}

[WbICHA]

otdameskapizm,

Поставьте проверку условия, не (elem % 2 === 1), а (elem % 2 !== 0). В Вашем случае происходит проверка наличия числа в массиве, которое без остатка делится на 1. А нужно проверять, чтобы числа без остатка делились на 0. Если после этой проверки выскочит false, то значит, что у Вас в массиве есть число, которое не делиться на 0 без остатка - нечетное.

Что за ересь ты тут написал?

Answer 2

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

Нужно чётко формулировать желания. На английском )

У массивов есть встроенные методы, которыми удобно решать задачи.

Хочется узнать, все ли элементы массива чётные? «Все» — "every"
arr.every( item => (item & 1) === 0 )

Узнать, какие элементы массива чётные? Что в результате получить.. Только четные элементы – значит, отфильтровать искомое, "filter": arr.filter( item => (item & 1) === 0 )

Функциям хорошо бы давать понятные названия. В примерах выше одна и та же функция определяет четность очередного item'а. Можно её сделать отдельной функцией: const isEven = num => (num & 1) === 0;тут проверяется последний бит числа, если он 0, значит, число четное. Годится только для небольших 32-битных целых.

Примеры станут лаконичнее:

arr.every(isEven) // false
arr.filter(isEven) // [2, 4, 6, 8, 10]