I have read that \" a variable is shadowed if there is another variable with the same name that is closer in scope\". I found this Point class with a constructor as an example:<
0
is the default value for int
type variables, and as the warning says, you're shadowing the variables in the constructor so the object variable is never assigned to.
public Point(int x, int y) {
x = x; //assign value of local x to local x
y = y; //assign value of local y to local y
}
You need to use this
keyword to refer to the object variables x
and y
instead of the local variables:
public Point(int x, int y) {
this.x = x; //assign value of local x to object variable x
this.y = y; //assign value of local y to object variable y
}
I thought that "x = x" in the constructor would be like "23 = 23".
Within the constructor, the meaning of the simple name x
is always just the parameter. So the assignment x = x
in the constructor takes the value of the x
parameter and assigning it to the x
parameter as well. The instance variable is never touched. (It's not clear what you mean by 23 = 23;
, so I can't tell whether or not that's accurate.) Basically, this is a no-op and some IDEs will give you a warning about it.
To force it to copy to the instance variable, you want:
this.x = x;
(And likewise for y
, of course.)