[josm-dev] Automatic tag correction

Frederik Ramm frederik at remote.org
Mon Jul 7 10:04:25 BST 2008


Hi,

>>> You mean I should use public fields? Well, the access methods *is* the
>>> correct way :-) but on the other hand it's such a simple "record class"
>>> so public fields are probably ok here.

>> No, actually not. I do think we should clean up josm code sooner or later. 
>> I really encourage to use the proper java way and completely omit public 
>> fields ;-)

> Ah, ok.

Please do not flood JOSM with getters and setters. If anyone wants to 
completely clean up JOSM they are free to do it - I agree that an 
architectural review would make sense - but I would not like a mix of 
the "traditional JOSM way" and the "textbook Java way" in the code.

The textbook Java way is to use getters and setters unless this is not 
possible for some reason - even if you cannot imagine any situation 
where they might be needed. The traditional JOSM way is to use public 
members unless getters and setters are required right now.

Bye
Frederik




More information about the josm-dev mailing list