The code is simplier reading.
Another good practice, in my opinion, is calling the getXXX() in toString() too (instead then this.XXX), beacuse the getXXX() can have an important logic.
I think that use the name of attribute without this is'nt a good idea for the maintenance of the application.