Global variable conflicts with algorithm.

Challenge Factorialize a Number has an issue. User Agent is: Mozilla/5.0 (Windows NT 5.1; rv:47.0) Gecko/20100101 Firefox/47.0. Please describe how to reproduce this issue, and include links to screenshots if possible.

My code:

var count=1;
function factorialize(num) {
  for(var i=1; i<=num;i++){

    count= count * i;

  }
  return count;

}

factorialize(0);

[Locheed]

Just spend some time figuring out why my test fails and it was this problem with a global variable.

Just a thought, how about adding a note to descriptions, that “Using global variables in this test will cause the test to fail.” I think it would help in this kinda situation, but doesn’t give too many tips for an exercise.

[raisedadead]

@krishnakumar360 : Thanks for reporting this.

Your code is correct but using the var count=1 in the global scope is causing the test cases to fail. Moving it to the function’s scope should let you pass the challenge.

function factorialize(num) {
  var count = 1;
  for(var i=1; i<=num; i++) {
   count = count*i;
...

The reason being that we run multiple test cases in on the code, using a global var like this persists the last value and hence the test cases fail.

Nevertheless this is valid code and we should be addressing this.