Search:

I would also be consistent in variable naming. If the variable name is 'city', the setter signature should be:

public void setCity( String city )

and the getter signature should be:

public...

The purpose of a 'setter' is to provide an interface to the outside world to set something, usually a variable in the class, to the parameter passed to the method. You currently set most variables...