hufscheer / spectator-server

7 stars 0 forks source link

[FEAT] 리그팀 수정 기능 구현 #165

Closed Jin409 closed 3 months ago

Jin409 commented 3 months ago

🌍 이슈 번호

🍀 확인해야 할 부분

쿼리 관련

public void validateLeagueTeamPlayer(LeagueTeamPlayer leagueTeamPlayer) {
        if (!this.leagueTeamPlayers.contains(leagueTeamPlayer)) {
            throw new IllegalStateException("해당 리그팀에 속하지 않은 선수입니다.");
        }
    }

처음에는 위와 같이 리그팀에 속한 선수인지 확인하는 과정에서 leagueTeamPlayer 가 영속화 되지 않아서 오류가 발생 했었습니다. 그래서 fetchType 을 eager 로 바꾸는 대신에 다음과 같이 직접 쿼리로 fetch join 해오도록 했는데 괜찮은 방법일지에 대한 고민이 있었습니답.

@Query(
            "SELECT lt FROM LeagueTeam lt "
                    + "LEFT JOIN FETCH lt.leagueTeamPlayers "
                    + "WHERE lt.id = :id"
    )
    LeagueTeam findById(Long id);

이미지 url 변경 관련

이미지 삭제 관련

이미지 삭제 기능이 간단해서 따로 PR 을 분리하지 않고 이 PR 에서 모두 구현했습니답