인프런 커뮤니티 질문&답변

안녕하세요 토비님! 좋은 강의 만들어주셔서 잘 듣고 있습니다.

작은 질문이 하나 있는데요. checkDuplicateEmail 메서드와 동시 요청에 관한 질문이 있습니다.

섹션 5. 28강 회원 애플리케이션 서비스 테스트(2) 20:47초 경에 email 중복 체크를 위하여 checkDuplicateEmail 메서드 코드를 작성하였고, 같은 email로 가입하려고 하면 DuplicateEmailException이 발생하는 것을 확인하였습니다.

그런데 여기서 다른 가정을 해보고 그런 상황에서 토비님이라면 어떻게 하셨을지 궁금합니다.

만약 email이 아닌 id로 회원가입을 하는 상황을 가정한다면,

email과 다르게 id는 여러 사람이 같은 id로 회원가입하려고 시도할 수 있습니다.

따라서 동시에 2개의 요청이 들어오게 된다면 checkDuplicateEmail는 둘 다 통과하고 DataIntegrityViolationException이 발생하게 될 것입니다. (DB에서 유니크 제한을 걸었기 때문에)

규칙인 id는 중복되지 않는다는 지켜지겠지만,

사용자가 보게 될 예외는 우리가 의도했던 DuplicateEmailException가 아니게 되겠죠.

여기서 궁금한 점은 이런 상황까지 고려하여 코드를 작성하여야 하는 것인지

아니면 그냥 넘어갈 것인지 궁금합니다.

저라면,

드물게 발생할 것이라고 예상을 했다면 로깅만 잘 해놓고, 해당 예외가 많이 발생했거나 관련 cs문의가 많이 들어온다면 추가로 코드를 작성할 것 같습니다.

처음부터 만약 많이 발생할 것이라고 예상했다면 try-catch를 통해 예외를 변경해줬을 것 같습니다.

아마 다음과 같은 코드가 될 것 같습니다.

@Override
public Member register(MemberRegisterRequest registerRequest) {
    try {
        checkDuplicateEmail(registerRequest);

        Member member = Member.register(registerRequest, passwordEncoder);

        memberRepository.save(member);

        emailSender.send(member.getEmail(), "등록을 완료해주세요", "아래 링크를 클릭해서 등록을 완료해주세요");

        return member;
    } catch (DataIntegrityViolationException e) {
        if (/*e를 통해 유니크 키 예외를 확인했다면*/) {
            throw new DuplicateEmailException();
        }
        throw e;
    }
}

서비스의 코드가 현재보다는 보기 지저분해진다고 생각했습니다. (아니면 이 방법이 아닌 다른 방법이 있을까요?)

토비님은 어떻게 생각하시는지 궁금합니다.