질문있습니다.

public class KnightTest {
    public static void main(String[] args) {
        // 객체 생성
        Knight knight = new Knight("돈키호테",30);
        
        System.out.println("[객체 생성]");
        System.out.printf("Knight { name: %s, hp: %d }\n",knight.getName(), knight.getHp());
        knight.setHp(30);
        System.out.println("[체력 증가 +30]");
        System.out.printf("Knight { name: %s, hp: %d }",knight.getName(), knight.getHp());


    }
}

class Knight {
    // 필드
    private String name;
    private int hp;
    // 생성자
    public Knight(String name, int hp){
        this.name=name;
        this.hp = hp;
    }
    // 게터
    public String getName(){
        return this.name;
    }
    public int getHp(){
        return this.hp;
    }
    //// 세터
    //public void setName(String s){
    //  name = s;
    //}
    public void setHp(int n){
        hp += n;
    }
}



///이런식의 코드도 괜찮은걸까요?? 뭔가 toString()메서드를 만들어서 return을 해서 그값을 출력하는것보다 
이게 더 와닿아서 게터세터에만 의존해서 출력을 하였는데. 어떤 방법이 좋은건지 궁금해서 그리고 왜 더 좋은지가 궁금해서
질문드립니다