Что не так с кодом?

Question

[Alex_87]

Добрый день! Задание следующее:
Реализуйте функцию getSameParity, которая принимает на вход массив чисел и возвращает новый, состоящий из элементов, у которых такая же чётность, как и у первого элемента входного массива.
Примеры
getSameParity([]); // []
getSameParity([1, 2, 3]); // [1, 3]
getSameParity([1, 2, 8]); // [1]
getSameParity([2, 2, 8]); // [2, 2, 8]

console.log(getSameParity([1, 2, 3])); // [1, 3]

function getSameParity(arr){
  const newArr = [];
  let s;
  if(arr[1] % 0 != 0){
    s = '!=';
  }else{
    s = '==';
  }

  for(let i = 0; i < arr.length; i += 1){
    if(arr[i] % 0 +`${s}`+ 0){
       newArr.push(arr[i]);
    }
  }
  return newArr;
}

У меня весь массив возвращается. Почему?

Comments

[Alex_87]

Дмитрий, Я поменял вопрос. Прочитайте пожалуйста!

[Aetae]

Вы просто складываете строки и числа. Код из строки никаким чудесным образом не выполнится(потому что он не код, а просто текст).
Забудьте об идее "самогенерации" кода на ходу(хоть это и возможно, но это совершенно не то, о чём должен думать начинающий).
Просто пишите код сразу, пусть даже получится(на ваш взгляд) многословнее.

[Дмитрий]

Alex_87, Прочитал, отписался в ответах.

Answer 1

[Дмитрий]

Потому что эта строчка не имеет ни малейшего смысла
arr[i] % 0 +`${s}`+ 0
Во первых вы пытаетесь получить остаток от деления на нуль вместо двойки, во вторых вы составляете строку, а строка это всегда true, Поэтому все возвращается.
Я понимаю что вы хотите сделать, но это код, вы не можете составлять код на лету, правильный подход

function getSameParity(arr){
  const newArr = [];
  let s = arr[0] % 2;

  for(let i = 0; i < arr.length; i += 1){
    if(arr[i] % 2 === s){
       newArr.push(arr[i]);
    }
  }
  return newArr;
}

console.log(getSameParity([3, 6, 7, 8])) // [3, 7]
console.log(getSameParity([4, 6, 7, 8])) // [4, 6, 8]

либо короче

const getSameParity = arr => arr.filter(v => v % 2 === arr[0] % 2)

Comments

[Alex_87]

Спасибо, что написали мне! Но ваш код возвращает всегда новый массив из чётных значении, а в задании указанно, что в зависимости от того, какое первое значение входящего массива(чётное/нечётное) от этого и решаем, что добавлять в новый массив.

[Дмитрий]

Alex_87, Это не так, в моем коде в первом примере была маленькая ошибка, перекочевавшая из вашего кода, должно быть let s = arr[0] % 2;

вместо

let s = arr[1] % 2;

Код рабочий

[Владимир Солнцев]

Alex_87, ну проверьте четность первого числа, и в зависимости от него возвращайте числа

Answer 2

[psytonik]

тут проблема в том что если числа в массиве будут отрицательные то не сильно это правильный ответ
[1, 2, -3] должен вернуть [1, -3]
[-3, 2, 1] должен вернуть [-3, 1]

Comments

[RikkiTikiTavi]

если обернуть в Math.abs, вот так let s = Math.abs(arr[0] % 2); то будет отрицательные тоже выводить