refactor: 발급 티켓 도메인 메서드 정리 및 API 추가 #180

sanbonai06 · 2023-01-23T08:42:51Z

개요

close 🔨[refactor] 발급 티켓 도메인 정리, 주석 처리 및 입장 처리 관련 메서드 추가 #167

작업사항

발급 티켓 도메인 내부 메서드들을 조금 정리했습니다.
정리하면서 필요해보이는 상태변환 메서드들을 추가했습니다.
또 상태변환 메서드를 추가하니 입장 처리해주는 API가 필요해서 추가해주었습니다.
정상 응답

PATCH /v1/admin/issuedTickets/{issuedTicketId}

{
  "success": true,
  "status": 200,
  "data": {
    "issuedTicketId": 16,
    "issuedTicketNo": "T1000016",
    "uuid": "f8197615-ec94-472b-87b5-ba5b03b69db7",
    "ticketName": "test",
    "ticketPrice": "4000원",
    "createdAt": "2023-01-23 03:24:58",
    "issuedTicketStatus": "입장 완료",
    "optionPrice": "0원"
  },
  "timeStamp": "2023-01-23T17:40:11.571074"
}

티켓이 이미 입장 처리 되었을 때

{
  "success": false,
  "status": 400,
  "code": "IssuedTicket_400_5",
  "reason": "이미 입장 처리된 티켓입니다.",
  "timeStamp": "2023-01-23T17:40:50.49026",
  "path": "http://localhost:8080/api/v1/admin/issuedTickets/16"
}

추가로 해당 부분에도 락을 걸고 들어가야 할지 고민을 해봤습니다. 하지만 QR 찍는 행위가 동시에 일어 날 일은 없다고 생각해서 따로 처리하지 않았습니다.

변경로직

찬진님이 말한 티켓재고관련 Redisson Lock에서 새로운 트랜잭션이 필요하기 때문에 needSameTransaction 옵션을 디폴트값인 false로 변경하였습니다.

    @RedissonLock(LockName = "티켓재고관리", paramClassType = TicketItem.class, identifier = "id")

ImNM

굿입니다요~

ImNM · 2023-01-23T08:44:36Z

...main/src/main/java/band/gosrock/domain/domains/issuedTicket/adaptor/IssuedTicketAdaptor.java

    public void delete(IssuedTicket issuedTicket) {
-        issuedTicket.cancelIssuedTicket();
+        issuedTicket.cancel();
    }


cancel 로 통일시키는게 좋아보입니당!

ImNM · 2023-01-23T08:46:43Z

DuDoong-Domain/src/main/java/band/gosrock/domain/domains/issuedTicket/domain/IssuedTicket.java

+    티켓이 입장 미완료 상태가 아니면 입장 할 수 없음
+     */
+    public void entrance() {
+        if (this.issuedTicketStatus == IssuedTicketStatus.CANCELED) {


백기선 님 강좌듣는데 이넘값을 == 으로 비교하면 더 좋다 그러더라구요
this.issuedTicketStatus.equals(IssuedTicketStatus.CANCELED)
위방식보다
민준님 방식이 euqals 칠때 null 포인터 exception 까지 막을수 있으니