Thread: Scope Question

Hi,

I'm having a problem with variable scope. I'm trying to modify a global integer within a function, and it doesn't seem to be working ... I've simplified the code to show what I mean:

Code:

static int variable;

void function1(void)
{
  cout << variable;
}

void function2(void)
{
  variable++;
}

int main(void)
{
  function2();
  function1();

  return(0);
}

For some reason the variable isn't getting modified in function 2, because the output in function 1 is the same. Any thoughts? Thanks for the input!!

what do u mean its not gettin modified isnt variable start off at zero? it gets incramented to 1.

i dont see the problem mabye im just not gettin it

Hi tunerfreak,

Well, the variable starts out as zero. When I call function 2, it should be incremented to one, but when I call function 1 after function 2, it is still zero, and I'm not sure why.

it comes out 1 when i run it ????

Strange. Well here's the real program; it's an OpenGL program:

Code:

static int xrot;

void reshape(int w, int h)
{
  ...
  glRotated(5.0, xrot, yrot, zrot);
  ...
}

...

void keyboard(unsigned char key, int x, int y)
{
  ...
  switch(key)
  {
    case('x'):
    xrot = 1;
    break;
  }
  ...
}

For some reason, xrot remains zero, and it's been bugging me for hours!

openGL sorry dont know that one

Ah, it's all good. Thanks though!!

switch(key)
{
case('x'):
xrot = 1;
break;

Either you declare a local xrot variable so "xrot" there is not the global xrot, 'key' is never 'x', or you reset the value of the xrot variable after the assignment.

You shouldn't be using a global variable anyway...

Hi joni,

Unfortunately, the program requires the global variable. When I replace xrot with 1 in the reshape function, it works fine, so I know the problem is the global xrot not being set to 1 inside the function.

I've tried placing the rotate function inside the switch statement, and it worked, so I know the key can equal 'x'. There's no other declaration of xrot, nor is there any other assignment of xrot within the program. I'm at a total loss. I'm thinking of just placing all of the commands inside the switch statement just to get it done.

Thanks for the help joni and tunerfreak, hopefully I'll figure it out!

AH! I fixed it with the line:

Code:

glutPostRedisplay();

after my code.

Thanks everyone!