Changing variables in if statements

Hey I'm trying to work out how to make a JLabel move around the screen with the wasd keys.
To do this I'm doing something like this:

int x = 0;
int y = 0;
if (input == 'w') {
y -= 5;
}
else if (input == 'a') {
x -= 5;
}
label.setLocation(x, y);

Only problem is, since I'm changing the value of the x and y variables inside an if statement, its not actually changing the value of the variable outside.
How do I make it do that?

Sorry for the dumb question, bit of a newbie :(think):

Quote:

---

Originally Posted by Guy

Hey I'm trying to work out how to make a JLabel move around the screen with the wasd keys.
To do this I'm doing something like this:

int x = 0;
int y = 0;
if (input == 'w') {
y -= 5;
}
else if (input == 'a') {
x -= 5;
}
label.setLocation(x, y);

Only problem is, since I'm changing the value of the x and y variables inside an if statement, its not actually changing the value of the variable outside.
How do I make it do that?

---

But you defined those variables outside of those if-statements so those are the variables that are changed by those if-statements. What made you think otherwise?

kind regards,

Jso

Hey. I'm sorry I phrased my question very badly and just realized the answer was very obvious. The part of code I posted was actually inside a method and each time I called the method it set the x and y variables to 0.
Sorry for wasting your time, but thanks for the quick reply!

Quote:

---

Originally Posted by Guy

Hey. I'm sorry I phrased my question very badly and just realized the answer was very obvious. The part of code I posted was actually inside a method and each time I called the method it set the x and y variables to 0.
Sorry for wasting your time, but thanks for the quick reply!

---

No need to say sorry; if you defined those variables in a method (they're local to that method then) they're initialized over and over again when that method gets executed. Better define them as member variables (outside of any method) so they can keep their value assigned to them in that method (that's what you want here).

kind regards,

Jos