What is an appropriate loop for a function with multiple parameters

Question

The function nb_year should return n number of entire years needed to get a population greater or equal to p.The given parameters are p0(original population), percent(percentage of increase), aug (inhabitants coming or leaving each year), p (population to surpass) i understand what the math is to get the result i just don't know what type of loop i should write to keep increasing the numbers. here is my current code

function nbYear(p0, percent, aug, p) {
   var increaseInHab = p0 * (percent * .01);
   var currentInhab = increaseInHab + p0 + aug;

   for(var n = 0;currentInHab < p;n++) {
     console.log(n)
     n++
     console.log(currentInhab + increaseInHab);
     currentInhab + increaseInHab;
    }
   return n;
}

currentInhab + increaseInHab; adds two numbers and does nothing with the result. Hint, you need to store it back into the variable you want to update. — epascarello
– epascarello, Commented Jan 10, 2017 at 22:18
@Blauharley Are we reading the same question???? What is a "JSON-param" — epascarello
– epascarello, Commented Jan 10, 2017 at 22:19
Recursion is the process generally used when modeling populations. — Travis J
– Travis J, Commented Jan 10, 2017 at 22:19
"aug (inhabitants coming or leaving each year)" aug is expected to be position number, negative number or 0? "i just don't know what type of loop i should write to keep increasing the numbers." If aug can be a negative number, the numbers are not guaranteed to increase, yes? — guest271314
– guest271314, Commented Jan 10, 2017 at 22:19

Michał Pietraszko · Accepted Answer · 2017-01-10 22:27:22Z

2

Good loop, but implementation needs fixing.

function nbYear(p0, percent, aug, p) {
   var increaseInHab = p0 * (percent * .01);
   var currentInhab = increaseInHab + p0 + aug;

   for(var n = 0; currentInHab < p; n++) {
     currentInhab += increaseInHab;
   }

   return n;
}

answered Jan 10, 2017 at 22:27

Michał Pietraszko

6,3093 gold badges24 silver badges27 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Hunter Brennick · Accepted Answer · 2017-01-10 22:28:41Z

0

Without looking at the specific details of your code (I'm assuming the multiplication expressions are correct), below would be a quick fix for your for loop problems:

function nbYear(p0, percent, aug, p) {
    var increaseInHab = p0 * (percent * .01);
    var currentInhab = increaseInHab + p0 + aug;
    var n = 0;

    for (; currentInHab < p; n++) {
       currentInhab + increaseInHab;
    }

    return n;
}

Also, you were incrementing n twice.

edited Jan 10, 2017 at 22:28

answered Jan 10, 2017 at 22:21

Hunter Brennick

886 bronze badges

5 Comments

Hunter Brennick Over a year ago

I'm confused. I did not use let

Paul Over a year ago

You assumed that var works the way let does, but it doesn't. Your code was identical to the OP's before the edit and now it just has one less n++ and the logging is removed, but moving the declaration of n has no effect..

Hunter Brennick Over a year ago

I don't see let being used in my original answer, edited answer, nor the author's original question.

Paul Over a year ago

Exactly, var is used in both the OP's post and your answer, not let, so it doesn't matter where in the function n is declared. It would only matter if let was used, but it isn't.

Hunter Brennick Over a year ago

I understand what you are trying to communicate now. I did not before.

Collectives™ on Stack Overflow

What is an appropriate loop for a function with multiple parameters

2 Answers 2

Comments

5 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

5 Comments

Your Answer

Sign up or log in

Post as a guest

Related