Trouble with variable scope in javascript

Question

I'm new to javascript and am having trouble with a counting program and I believe it could be trouble with variable scope.

var count = 0; {
    function gimmeRandom() {
        var rand = Math.floor(Math.random() * 10) + 1;
        count++;
    }

    function countToRandom() {
        for (count = 1; count <= rand; count++) {
            console.log(count);
        }
    }

    console.log("Counting to a random number");
    gimmeRandom();
    countToRandom();
    console.log("Counting to another random number");
    gimmeRandom();
    countToRandom();
    console.log("There has been " + count + " random numbers used");
}

There is an opening bracket at line 2, sure this is not a typo? — Charis Moutafidis
– Charis Moutafidis, Commented Mar 26, 2019 at 19:39
What goes wrong? What do you expect, and how do the results differ? edit and yes the code as posted is syntactically incorrect, so it's hard to answer the actual question. — Pointy
– Pointy, Commented Mar 26, 2019 at 19:39

Jonas Wilms · Accepted Answer · 2019-03-26 19:40:51Z

1

You declare var rand inside gimmeRandom, you can't access it in countToRandom. You probably want a global variable, just as you did with count.

answered Mar 26, 2019 at 19:40

Jonas Wilms

139k20 gold badges164 silver badges164 bronze badges

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

Comments

basic · Accepted Answer · 2019-03-26 19:41:34Z

Declaring var inside of a function scopes that variable to the function. You have two choices.

Since you are calling gimmeRandom first, drop the var keyword and it will be automatically global.

var count = 0;     

{                                               
function gimmeRandom()                                              
{                                               
  rand = Math.floor(Math.random()*10)+1;                                                
  count++;                                              
}                                               

function countToRandom()                                                
{                                               
  for (count = 1; count <= rand; count++)                                               
  {                                             
     console.log(count);                                                
  }                                             
}                                               

console.log("Counting to a random number");                                             
gimmeRandom();                                              
countToRandom();                                                
console.log("Counting to another random number");                                               
gimmeRandom();                                              
countToRandom();                                                
console.log("There has been "+count+" random numbers used");                                                
}

Define var rand at the top of the page to make it global.

var count = 0;     
var rand = 0;

{                                               
function gimmeRandom()                                              
{                                               
  rand = Math.floor(Math.random()*10)+1;                                                
  count++;                                              
}                                               

function countToRandom()                                                
{                                               
  for (count = 1; count <= rand; count++)                                               
  {                                             
     console.log(count);                                                
  }                                             
}                                               

console.log("Counting to a random number");                                             
gimmeRandom();                                              
countToRandom();                                                
console.log("Counting to another random number");                                               
gimmeRandom();                                              
countToRandom();                                                
console.log("There has been "+count+" random numbers used");                                                
}

This is basically what I needed but the the count variable isn't working properly. I think it's to do with the last line of code but I'm not too sure
@jonnyc132 what is count supposed to do? Right now it is doing exactly what you are telling it to do...
ideally I'd want it to show how many random numbers there have been, or how many times gimmeRandom has been called

chevybow · Accepted Answer · 2019-03-26 19:42:38Z

0

You define rand to be bounded in this function:

    function gimmeRandom() {
         var rand = Math.floor(Math.random() * 10) + 1;
         count++;
    }

And then try to use it in another function:

    function countToRandom() {
        for (count = 1; count <= rand; count++) {
            console.log(count);
        }
    }

As seen in this question:

What is the scope of variables in JavaScript?

Your variable rand you define is given a local scope to the gimmeRandom() function and therefore can't be used outside that function.

To use it across functions you'd probably wan to make the variable have global scope

answered Mar 26, 2019 at 19:42

chevybow

12.3k7 gold badges29 silver badges42 bronze badges

Collectives™ on Stack Overflow

Trouble with variable scope in javascript

3 Answers 3

Comments

3 Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

3 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related