연관관계 편의 메서드에서 mappedBy 필드에 대한 수정 로직 추가여부

학습하는 분들께 도움이 되고, 더 좋은 답변을 드릴 수 있도록 질문전에 다음을 꼭 확인해주세요.

1. 강의 내용과 관련된 질문인가요? 예
2. 인프런의 질문 게시판과 자주 하는 질문에 없는 내용인가요? 예
3. 질문 잘하기 메뉴얼을 읽어보셨나요? 예

https://www.inflearn.com/questions/25417/changeteam-%EA%B4%80%EB%A0%A8%ED%95%98%EC%97%AC-%EC%A7%88%EB%AC%B8-%EC%9E%88%EC%8A%B5%EB%8B%88%EB%8B%A4

연관관계 편의 메서드 관련 질문인 위 링크의 글과 연관이 있습니다.

질문을 먼저 작성해놓고 질문 끝에 다시 언급해놓겠습니다.

1) 어쩌면 필요없는 구현일 수 있는데, 이런 습관?을 들여도 괜찮은지

2) 아니면 강의처럼 DB에 반영되지 않고 거의 문제가 발생할 가능성이 없는 부분은 간단하게 구현하고 넘어갈지

mappedBy로 read-only상태인 조회 전용 필드인 members는 변경 상태가 DB에 반영되지 않지만,

인스턴스 입장에서는(객체지향적으로는?) 어플리케이션 런타임 시점에 변경을 해주어야한다고 생각했습니다.

위 링크의 영한님 말씀처럼 데이터 변경 후 이를 즉시 재활용하는 일은 극히 드물지만 말이죠.

• Member 엔티티

// 연관관계 편의 메서드
public void changeTeam(Team team) {
    // 기존에 팀이 있다면 탈퇴
    if (this.team != null) {      
        // 탈퇴를 위한 비즈니스 로직을 Team에 구현함      
        this.team.removeMember(this);   
    }
    this.team = team;
    team.getMembers().add(this);
}

그래서 위와 같이 if 분기를 두고 소속 팀이 있는 회원은 해당 팀에서 회원 자신을 지우는 로직을 추가했습니다.

this.team.removeMember(this)

위 로직은

this.team.getMembers().remove(this)

와 같지만

의미있는 메서드를 만들어보고 싶어서 아래처럼 removeMember 를 구현했습니다.

• Team 엔티티

// 비즈니스 로직  -> read-only 필드라서 DB에는 반영 안됨
public void removeMember(Member member) {
    this.members.remove(member);
}

null 확인 로직이 changeTeam() 내부에 있으므로 Member 엔티티 생성자는 다음과 같이 작성했습니다.

public Member(String username, int age, Team team) {
    this.username = username;
    this.age = age;
    changeTeam(team);
}

결론적으로 물어보고 싶은 질문은

1) 어쩌면 필요없는 구현일 수 있는데, 이런 습관?을 들여도 괜찮은지

2) 아니면 강의처럼 DB에 반영되지 않고 거의 문제가 발생할 가능성이 없는 부분은 아래처럼 간단하게 구현하고 넘어갈지

// 연관관계 편의 메서드
public void changeTeam(Team team) {
    this.team = team;
    team.getMembers().add(this);
}

이렇게 두 가지 질문이 있습니다.

늘 좋은 강의 해주셔서 감사합니다.