Is it bad practice to change my getter method like version 2 in my class.
Version 1:
public String getMyValue(){ return this.myValue }
absolutely yes, it's a bad pratice.
Imagine you communicate accross network with a third party (remoting, COM, ...), this will increase the round-trip and then hit application performance.