Array return

Question

I have to do a function with the name saldoDeMesesComLucro where a array of a period of months, in the function I need to find out which of those months gave a valor > 0 and make the sum of this month, after that the return has to be a array which will show the months in which it tested positive with their balances next door.

What I did didn’t work out.

function saldoDeMesesComLucro(5) {
  var soma= 0;
  var mes = 0;
  for (var i=0; i<umPeriodo.length; i++) {
    if(umPeriodo[i] > 0) {
    mes = umPeriodo[i];
    soma = soma+ mes;}
  }
  return soma;
}

Answer 1

The code is very wrong and not close to what it asks even having basic errors in the code, so consider doing simpler exercises first while not mastering something so complex. And if the statement is the same it is very vague and fits interpretation, so if you use the code fixed on some online trial site and keep giving error is probably a problem of how the statement was passed.

For example only those who wrote this know what is "it has to be an array that will show the months when q has tested positive with their balances on the side" since arrays don’t show anything, they serve to store values, and it gets harder to know what it means next door. Nor is it easy to understand what the exact relation is, the text is very ambiguous. I gave my interpretation. If this is not enough the question is not clear and we have no way to solve the problem with what was posted.

If you have to generate one array new to return at the end of the function then have to create it starting empty, do not have to create several other arrays for this.

So what you should do is create a new element in that array every time the condition is set in execution, the condition is right but before it was adding a value instead of creating that new element.

The statement asks to assemble the months that are above 0 with the balance value. It is not clear what is this balance. I understand that it is the accumulation of the previous months. Then you have to add up the previous month used and the current value that is greater than 0. That’s why I controlled the position of the array new I am to know which is the previous position (variable j).

I make an exception to the first element of array because there is no way to catch the previous helmet when there is no element.

function saldoDeMesesComLucro(umPeriodo) {
    var somas = [];
    for (var i = 0, j = 0; i < umPeriodo.length; i++) {
        if (umPeriodo[i] > 0) {
            somas.push(umPeriodo[i] + (j == 0 ? 0 : somas[j - 1]));
            j++;
        }
    }
    return somas;
}
console.log(saldoDeMesesComLucro([0, 1, 2, 0, 3, 4, 0, 5, 0, 0, 6]));

I put in the Github for future reference.

Reinforcement to catch better defined problems even for help that is really useful.

Answer 2

Good afternoon! Follow the solution below, as requested.

function saldoDeMesesComLucro(umPeriodo) {
  var arrayMesesLucro = [];
  var filtro = umPeriodo.filter(positivo);
  for (let mes=0; mes< umPeriodo.length; mes++) {
   arrayMesesLucro[mes] = filtro[mes];
   }
  return filtro;
}

function positivo(valor) { 
  return valor > 0;
}

Any doubt I’m available!