Is it bad practice to change my getter method like version 2 in my class.
Version 1:
public String getMyValue(){ return this.myValue }
No. You're doing two things here. Getting and setting.