Is it bad practice to change my getter method like version 2 in my class.
Version 1:
public String getMyValue(){ return this.myValue }
A setter could modify as part of validation, but a getter should return the value and let the validation be done by the caller. If you do validate, then how should be documented.