Почему при решении задачи я получаю undefined?

Question

[Dmasgj]

Дана задача :
Дан массив целых чисел.

Верните новый массив , где первый элемент — это количество положительных чисел, а второй элемент — сумма отрицательных чисел. 0 не является ни положительным, ни отрицательным.

Если input представляет собой пустой массив или имеет значение null, верните пустой массив.

Вот моё решение :

let arr = [1, 2, 3, 4, 5, 6, 7, 8, 9, 10, -11, -12, -13, -14, -15];
  
  function countPositivesSumNegatives(input) {
    // your code here
    let newArr = [];
    let positiveCount = 0;
    let summOfNegativeCount = 0;

    input.forEach(el => {
        if (input === [] || null) {
            return [];
        }
        if (el > 0) {
            positiveCount++;
        } 
        else {
            summOfNegativeCount += el;
        }
        
        return newArr += positiveCount, summOfNegativeCount;
        
  });
  }
  console.log(countPositivesSumNegatives(arr));

По итогу получаю значение undefined, помогите понять почему получается именно это значение?

Comments

[WbICHA]

Чтобы что-то получить из функции, надо что-то вернуть из функции.

[Dmasgj]

WbICHA, аааа, увидел ошибку, спасибо)

Answer 1

[Виталий Моржов]

Здравствуйте!
Немного переписал ваш код.

function countPositivesSumNegatives(input) {
  // your code here
  let positiveCount = 0;
  let summOfNegativeCount = 0;

  if (input.length === 0) {
    return [];
  }

  input.forEach((el) => {
    if (el > 0) {
      positiveCount += 1;
    }
    if (el < 0) {
      summOfNegativeCount -= -el;
    }
  });

  return [positiveCount, summOfNegativeCount];
}

1. Undefined Вы получаете потому что из функции ничего не возвращаете.
2. Терминальное условие пишите в самом начале функции, а не в цикле.

Comments

[Dmasgj]

Спасибо)

[Виталий Моржов]

Dmasgj, есть более элегантный способ решить задачу, но он менее производительный. Сложность алгоритма будет O(n2).

const countPositivesSumNegatives = (input) => {
  if (input.length === 0) {
    return [];
  }
  const positive = input.filter((num) => num > 0).length;
  const sumOfnegative = input.filter((num) => num < 0).reduce((acc, num) => acc + num, 0);
  return [positive, sumOfnegative];
};

[WbICHA]

Виталий Моржов,

Сложность алгоритма будет O(n2).

Это в каком месте?

[Виталий Моржов]

WbICHA, два раза по массиву фильтром

[WbICHA]

Виталий Моржов,

const positive = input.filter((num) => num > 0).length;

Открой для себя some.

const sumOfnegative = input.filter((num) => num < 0).reduce((acc, num) => acc + num, 0);

Если уж заговорили о производительности, то это можно было сделать с одним редьюсом.

if (input.length === 0) {
    return [];
  }

А вот это вообще непонятно что тут забыло, оно вернёт 2 андефайнда при пустом массиве.

[WbICHA]

Виталий Моржов,

два раза по массиву фильтром

Прочти что такое алгоритмическая сложность и как она считается.
И да, по твоей логике n3 сложность должна быть, там же ещё и редьюс!