Updating variable problem

Question

i am writing some code that updates a variable but the facing variable never seems to change, i want it to be updated at the end of every call from the main class
it seems to keep the value of Zero
any help would be great Eoin
public class RobotMover
Navigator robot;
private final int NORTH = 0;
private final int SOUTH = 180;
private final int WEST  = 270;
private final int EAST  = 90;
int facing = 0;
int clockWise  = 90;
int antiClockWise  = -90;
public RobotMover(Navigator newRobot)
robot = newRobot;
public void north()
     //if facing north : forward ()
          if(facing == NORTH)
               LCD.drawString("north", 0, 3);
               robot.travel(25);
               int facing = NORTH;
     //if facing south : rotate180(); forward ()
          else if(facing ==SOUTH)
               LCD.drawString("south to north", 0, 3);
               robot.rotate(180);
               robot.travel(25);
               int facing = NORTH;
     //if facing west  : rotate90Anticlockwise(); forward()
          else if(facing ==WEST)
               LCD.drawString("west to north", 0, 3);
               robot.rotate(clockWise);
               robot.travel(25);
               int facing = NORTH;
     //if facing east  : rotate90Clockwise(); forward()
          else if(facing == EAST)
               LCD.drawString("east to north", 0, 3);
               robot.rotate(antiClockWise);
               robot.travel(25);
               int facing = NORTH;
} the rest of the methods are similar setting facing in the given directions. however when called from main doesn't update from 0;
Edited by: Chuckie on Aug 9, 2009 2:26 PM

Answer 1

stevejluke wrote:
In each of your if statements you are creating a NEW locally scoped (to the IF statement) variable named facing, and assigning the direction to that.
Get rid of the int before each of the facing assignments:
          if(facing == NORTH)
               LCD.drawString("north", 0, 3);
               robot.travel(25);
               facing = NORTH;
After you get rid of the local declarations, please also notice that you are always setting facing to NORTH