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

회원테이블(Member)과 티켓 테이블(Ticket)이 있으며

티켓을 예매한 회원의 티켓 수정 삭제는 티켓테이블(JoinColumn)에서만 가능하고

회원테이블에선 티켓 조회(mapped)만 가능하도록 한다고 하고

이렇게 회원의 유니크한 username(아이디)로 티켓을 조회하는 메서드가 있다고 가정하겠습니다

(첫번째, 두번째 둘다 회원의 티켓 조회하는 메서드)

  @Transactional(readOnly = true)
    public Ticket findMyTicket(String username) {
        //회원 찾기
        Member member = memberRepository.findByUsername(username)
                                        .orElseThrow(() -> new UsernameNotFoundException(username + "에 해당되는 유저를 찾을수 없습니다"));

        //티켓 찾기
        Ticket ticket = member.getTicket();
        if (ticket == null) { //찾은 티켓 null 체크
            throw new NoFindMyTicketException(member.getName() + "님의 티켓을 찾을수 없습니다");
        }
        return ticket;
    }


  @Transactional(readOnly = true)
    public Ticket findMyTicket(String username) {
        //회원 찾기
        Member member = memberRepository.findByUsername(username)
                                        .orElseThrow(() -> new UsernameNotFoundException(username + "에 해당되는 유저를 찾을수 없습니다"));
        
        //티켓 찾기
        Ticket ticket = ticketRepository.findByMember_Id(member.getId())
                                       .orElseThrow(() -> new NoFindMyTicketException(member.getName() + "님의 티켓을 찾을수 없습니다"));
        return ticket;

    }
}

첫번째 경우는 테이블간 맵핑으로 인해 ticketRepository에 메서드를 만들지 않고 바로 조회할수 있지만 if문으로 예외처리를 해줬습니다

-->Member에서 바로 Ticket을 조회해서 사용하는 장점이 있었지만 NPE 처리해주는 코드를 if문으로 작성해야했습니다

두번째 경우는 ticketRepository에 따로 메서드를 만들어줬지만 Optional로 예외처리를 바로 해주었습니다

-=>ticketRepository에 메서드를 작성해야했지만 Optional로 인해 NPE방지 코드를 orElseThrow으로 바로 작성할수 있었습니다(Optional이 NPE방지에 좋다고만 알고 있는데 사용할때나 하지않을때나 둘다 코드를 작성하긴 해야해서 성능상으로 더 좋은건지 가독성이 좋은건지 잘 모르겠습니다..)

위 두 경우중 어떤게 더 좋은 코드라고 볼수있는건가요 선생님