Как при помощи метода forEach указать условие?

Question

[Иван Филипов]

Суть:
Простецкая задача на перебор массива(Посчитать сколько раз встретилось true).
Решил таким образом(на codewars зачли):

function countSheeps(arayOfSheep) {
    let count = 0;
    for (let i=0;i<arayOfSheep.length; i++) {
      if(arayOfSheep[i]===true) {
        count++;
      } 
    }
    return count;
  }

По идее можно перебрать быстрее и более короткой записью при помощи forEach, например:

function countSheeps(arrayOfSheep) {
  let count = 0;
  arayOfSheep.forEach((item) => item===true  count++1);
  return count;
}

Но второй код через forEach выдаёт ошибку.Не могу понять, что делаю не так...Может в условие метода тернарник записать?

Answer 1

[Алексей Ярков]

return arayOfSheep.filter((item) => item===true).length;

Или сокращённый вариант
return arayOfSheep.filter(Boolean).length;

Comments

[zkrvndm]

Если у стрелочной функции аргумент всего один круглые скобки для нее вроде же не требуются?

arayOfSheep.filter(i => Boolean(i)).length;

[Алексей Ярков]

Надим Закиров, не требуются

[zkrvndm]

Алексей Ярков, еще оказывается функция filter и так все приводит к boolean виду, поэтому можно еще короче:
arayOfSheep.filter(i => i).length;

[Алексей Ярков]

Надим Закиров не приводит она ничего. Коллбэк, который вы в filter передали, должен вернуть не false, чтобы элемент остался в массиве.

[zkrvndm]

Алексей Ярков, [ '', ' ', null, 0, '0' ].filter(i => i)
Приводится к boolean.

[Вадим]

Надим Закиров, Функция filter ничего не к чему не приводит.
JS приводит, в момент когда возвращённое значение сравнивает с false.
truthy и falsy values.

Answer 2

[otdameskapizm]

Ваш код не работает, потому что у вас там ошибка в этой строке

arayOfSheep.forEach((item) => item===true  count++1);

Если хотите тернарный оператор, то можете использовать вот так:

const countSheeps = function (arr) {
  let counter = 0;
  arr.forEach((sheep) => {
    counter += sheep === true ? 1 : 0;
  });
  return counter;
};

Answer 3

[Alexandroppolus]

const countSheeps = (array) => array.reduce((count, item) => item===true  ? count + 1 : count, 0);

Answer 4

[Rsa97]

Дык синтаксическая ошибка. Что должен означать этот код?
(item) => item===true count++1