사용하는 곳

• 처음 유저가 4명이 모여 게임 시작을 해 역할 선택으로 넘어가기 직전까지

  • 4명이 모여서 게임을 했던 팀이 있는지 찾는다
  • 있다면 그 팀명을, 없다면 공백을 넘긴다 → 같은 창에서 처리할 수 있게
  • 모험 시작을 눌렀을 때 팀명 유효성 검사를 거친 후 게임을 시작할 수 있게 한다

• 서브쿼리의 중요성

  • 유저 네 명이 모두 함께했을 때의 ID를 찾는 것이지만 유저 팀은 각 유저마다 저장이 되기 때문에 찾는 법을 고민함 → 컨설턴트님이 도와주셨음
  • 나의 해결법

  SELECT team_id FROM user_team
  WHERE user_id IN (1, 2, 3, 4)
  GROUP BY team_id
  HAVING count(team_id) = 4
  

  • 컨설턴트님의 해결법

  SELECT team_seq FROM (
  SELECT team_seq, COUNT(team_seq) cnt FROM tbl_team_user WHERE user_seq IN (1, 2, 3, 4) GROUP BY team_seq
  ) X
  WHERE cnt = 4
  

• 고려했던 점

  • 초기 모험 시작을 눌렀을 때(controller에 있는 함수 중 1)

    • 네 명의 조합이 있는지 위 사항을 QueryDSL을 사용해서 확인
    • 있다면 그 팀명을 String으로 출력

    // team/repository/UserTeamRepositoryImpl.java
    // 모든 멤버가 들어 있는
        @Override
        public List<Integer> checkTeam(List<Integer> userId) {
            List<Integer> ans = jpaQueryFactory
                    .select(userTeam.team.teamId)
                    .from(userTeam)
                    .where(userTeam.user.userId.in(userId))
                    .groupBy(userTeam.team.teamId)
                    .having(userTeam.team.teamId.count().goe(4))
                    .fetch();
    
            return ans;
        }
    

    // team/service/TeamService.java
    public Team findTeam(List<Integer> users){
            List<Integer> teamId = userTeamRepositoryCustom.checkTeam(users);
            if(teamId.size() == 0) return null;
    
            Optional<Team> team = teamRepository.findByTeamId(teamId.get(0));
            return team
                    .orElseThrow(() -> new RuntimeException("findTeam 에러"));
        }
    

  • 이름을 정해서 시작했을 때(컨트롤러의 함수 중 2)

    • 팀명이 존재한다면 → 위의 팀 조합 확인을 다시 함

      • 멤버 조합이 있는 팀의 팀명과 현재 팀명이 동일하다면 찾은 팀을 return
      • 아니라면 중복된 팀명이라는 message 전달

      // team/service/TeamService.java
      // 팀명 중복 체크
          public Team checkTeamName(RegistTeamDto registTeamDto) {
              if(teamRepository.existsByTeamName(registTeamDto.getTeamName())){
                  Team team = findTeam(registTeamDto.getUsers());
                  if(team.getTeamName().equals(registTeamDto.getTeamName())){
                      return team;
                  }else throw new RuntimeException("중복된 팀명입니다");
              }else{
                  return teamSave(registTeamDto);
              }
          }
      

    • 팀명이 존재하지 않는다면 teamSave라는 메서드 접근

      • 멤버 조합이 있는 팀이 있는지 findTeam을 함
      • 있다면 그 조합에서 이름이 바뀌었으므로 update
      • 없다면 아예 새로운 team을 만들어서 저장

      // team/service/TeamService.java
      // 팀 만들기
          @Transactional
          public Team teamSave(RegistTeamDto registTeamDto){
              Team team = findTeam(registTeamDto.getUsers());
      
              if(team == null) team = new Team();
              team.setTeamName(registTeamDto.getTeamName());
      
              teamRepository.save(team);
              team = teamRepository.findByTeamName(team.getTeamName()).orElseThrow(() -> new RuntimeException());
              userTeamSave(registTeamDto, team);
              return team;
          }
      

    • 위의 단계 이후 team의 일련번호를 받아와 userTeamSave 메서드 접근

      • 저장되었던 user 일련번호들을 불러옴
      • for문을 이용하여 각각의 user를 저장

      // team/service/TeamService.java
      // 유저 팀 만들기
          @Transactional
          public void userTeamSave(RegistTeamDto registTeamDto, Team team){
              List<Integer> userList = registTeamDto.getUsers();
              for(Integer userId : userList){
                  User user = userRepository.findById(userId).orElseThrow(() -> new RuntimeException());
      
                  UserTeam userTeam = UserTeam.builder().user(user).team(team).build();
                  userTeamRepository.save(userTeam);
              }
          }
      

  • 위 함수의 문제점

    • 하나가 안 될 때 모두 무효시키기 위해 Transcational을 걸어 놨으나 유저가 한 명만 존재해도 팀이 만들어져 버림,,, ㅠㅠ
    • 유저의 유효성을 먼저 체크한 후 save를 네 번 진행하는 게 아니라 list에 저장해 두었다가 saveAll 한번에 진행하는 것은 어떨까? → 이후 리팩토링으로 확인해 보아야 함

• 한모씨의 mapper를 훔쳐서 써 보고 싶어서 이해하려고 해 보았으나 어떤 식으로 구현을 해야 할지 모르겠어서 미뤘음