2주차 : 줌터파크 과제 제출합니다. #4
Conversation
minsoozz
left a comment
minsoozz
left a comment
There was a problem hiding this comment.
코드 보면서 많이 배웠어요~ 고생하셨습니다 👍
| @@ -0,0 +1,28 @@ | |||
| package com.week.zumgnmarket.entity; | |||
|
|
|||
| import lombok.*; | |||
There was a problem hiding this comment.
사람마다 견해가 다르지만 import 문에서 와일드카드는 지양하는 편입니다.
참고해서 링크 드릴게요
| @NoArgsConstructor(access = AccessLevel.PROTECTED) | ||
| @AllArgsConstructor | ||
| @Getter | ||
| @Builder |
There was a problem hiding this comment.
클래스에 @Builder 어노테이션을 추가하신 이유를 알 수 있을까요?
| public void decrease() { | ||
| final String thread = Thread.currentThread().getName(); | ||
| if((this.ticketCount - 1) < 0 ){ | ||
| //throw new RuntimeException("오늘자 티켓팅이 마감되었습니다 (재고 부족)"); - 테스트 위해 주석 |
There was a problem hiding this comment.
불필요한 코드와 주석은 과감하게 삭제하는 것도 좋을 것 같아요~
어차피 Git 에서 버전 관리를 해주고 있으니까요
| Musical musical = musicalTicketService.findMusicalById(ticketRequest.getMusicalId()); | ||
| MusicalTicket musicalTicket = musicalTicketService.findTicketByMusical(musical); | ||
| if (!musicalTicket.checkTicketingDate(ticketRequest.getTicketingDate())) { | ||
| throw new RuntimeException("지금은 티켓팅 기간이 아닙니다."); |
There was a problem hiding this comment.
RuntimeException 으로 예외를 처리하는 것보다는 티켓팅 기간이 아니라는 적절한 예외로 포장하는 것이 좋아보여요~
| Buyer buyer = buyerService.findBuyerById(ticketRequest.getBuyerId()); | ||
| Musical musical = musicalTicketService.findMusicalById(ticketRequest.getMusicalId()); | ||
| MusicalTicket musicalTicket = musicalTicketService.findTicketByMusical(musical); | ||
| if (!musicalTicket.checkTicketingDate(ticketRequest.getTicketingDate())) { |
There was a problem hiding this comment.
해당 조건문도 명확하게 메소드로 추상화 할 수 있을 것 같습니다.
|
|
||
| @Test | ||
| @Order(1) | ||
| void 락0_150명이_티켓팅을_실행() throws InterruptedException { |
There was a problem hiding this comment.
분산 락 개념이 정말 어렵던데 잘 구현해주셨네요 👍
| @Column(name = "ticketing_date") | ||
| private LocalDate ticketingDate; | ||
|
|
||
| public static MusicalTicket of(Musical musical, Long ticketCount, LocalDate ticketingDate) { |
There was a problem hiding this comment.
프로젝트를 보면 정적 팩토리 메소드로 객체를 생성하는 방법과, 빌더 패턴으로 객체를 생성하는 방법
이렇게 두가지를 사용하시는 것 같은데 이것을 구분하는 기준이 있으신가요?
| log.error("진행중인 사람 : {}, 메세지 : 티켓팅을 성공하셨습니다(남은 티켓 갯수 - {} 개) ", thread, this.ticketCount); | ||
| } | ||
|
|
||
| public boolean checkTicketingDate(LocalDate localDate) { |
There was a problem hiding this comment.
프레디케이트 메소드는 자바 메소드 규약에 맞게 is~ , has~ 로 통일하는 방법도 있습니다~!
| } | ||
| RLock lock = redissonClient.getLock(musical.getTitle() + ":" + ticketRequest.getTicketingDate()); | ||
| try { | ||
| if (!lock.tryLock(3, 5, TimeUnit.SECONDS)) { |
There was a problem hiding this comment.
락의 획득시도 시간, 락 점유 시간을 선택하신 기준이 있으신가요? 저도 동시성 이슈는 부족해서
영지님만의 기준이 있는지 궁굼해서 여쭤봅니다!
| return musicalJpaRepository.save(musical); | ||
| } | ||
|
|
||
| public MusicalTicket findTicketByMusical(Musical musical) { |
There was a problem hiding this comment.
해당 메소드는 조회 전용 쿼리로 보여서 readonly=true 옵션 또는 조회 전용 클래스를 만드는 것도 좋을 것 같습니다~!
2 participants
Facade 패턴을 적용하여 코드 작성을 하였습니다.
구매 로직에 대한 테스트 코드를 작성하였습니다.