[feat] #259 - Booking 엔티티 수정 및 환불/취소 요청 API 구현 #260

hyerinhwang-sailin · 2024-11-25T18:08:47Z

Related issue 🛠

closes [feat] Booking 엔티티 수정 및 환불/취소 요청 API 구현 #259

Work Description ✏️

Booking 엔티티에 필요 필드 추가
BookingStatus에 값 추가
입금완료 예매자 환불 요청 API 구현
미입금/무료공연 예매자 취소 요청 API 구현

Trouble Shooting ⚽️

Related ScreenShot 📷

유료공연 입금완료 예매자 환불 요청 API 구현

정상적으로 bookingStatus가 변경된 것을 확인했습니다.
환불요청의 경우 메이커가 환불처리를 해야 예매취소상태로 변경되기 때문에 schedule의 soldTicketCount는 변하지 않습니다.
미입금/무료공연 예매자 취소 요청 API 구현

정상적으로 bookingStatus가 변경된 것을 확인했습니다.
취소요청의 경우 즉각적으로 예매취소상태로 변경되기 때문에 취소 시 schedule의 soldTicketCount도 바뀌도록 구현했습니다.

Uncompleted Tasks 😅

To Reviewers 📢

예매 취소/환불 요청 모두 회원, 비회원에게 모두 허용되는 서비스이기 때문에 AUTH_WHITELIST에 url을 추가해두었습니다.
기존 booking 관련 서비스가 guest/member로 나뉘어있었으나 취소/환불 요청의 경우 회원 비회원 모두가 접근 가능한 서비스이므로 Service class를 새로 만드는 것이 적합하다고 판단해 BookingCancelService를 새로 만들었습니다.
class나 method 네이밍이 적합한가에 대한 확신이 부족해 더 나은 네이밍이 있다면 추천해주세요!
확인해보니 이전 pr과 커밋이 같이 합쳐진 듯합니다.. 이전 pr 머지 후 이 pr 머지하면 해결될 것 같습니다..!!

hoonyworld

고생하셨습니다 혜린님!!

몇가지 코멘트 남겼습니다~

hoonyworld · 2024-11-26T17:44:36Z

src/main/java/com/beat/domain/booking/application/BookingCancelService.java

+		booking.setRefundInfo(request.bankName(), request.accountNumber(), request.accountHolder());
+		booking.setBookingStatus(BookingStatus.REFUND_REQUESTED);


해당 부분 setter를 호출하지 않고 Booking 엔티티에서 refund라는 도메인 메서드에서 해당 비즈니스 로직을 적고. booking.refund() 형식으로 외부에서 호출하는 것이 나아보입니다!

setter 호출이 아닌 엔티티에 직접 작성한 메소드를 호출했습니다!

Booking.java

public void setBookingStatus(BookingStatus bookingStatus) { this.bookingStatus = bookingStatus; if (bookingStatus == BookingStatus.BOOKING_CANCELLED || bookingStatus == BookingStatus.BOOKING_DELETED) { this.cancellationDate = LocalDateTime.now(); } }

비즈니스 로직을 담고 있더라도 네이밍을 setter로 짓는것은 수정자로 오해를 줄 수 있을 것 같습니다.

setRefundInfo와 setBookingStatus 모두에 해당하는 말입니다!

booking.setRefundInfo(request.bankName(), request.accountNumber(), request.accountHolder()); booking.setBookingStatus(BookingStatus.REFUND_REQUESTED);

해당 메서드들을 refund()라는 도메인 비즈니스 메서드에서 처리를 하도록 하는게 어떨까요?
그리고 해당 메서드에 대한 예외 처는 도메인 비즈니스 메서드에서 해주면 될 것 같습니다.

네이밍 바꾸는 게 더 좋겠네요! 최신 pr에 반영하도록 하겠습니다~

hoonyworld · 2024-11-26T17:51:01Z

src/main/java/com/beat/domain/booking/application/BookingCancelService.java

+		return BookingRefundResponse.of(
+			booking.getId(),
+			booking.getBookingStatus(),
+			booking.getBankName(),
+			booking.getAccountNumber(),
+			booking.getAccountHolder()
+		);


이 부분은 메서드의 인자로 필요한 매개변수 대신 객체를 넘겨주고 해당 getter 들을 DTO에서 처리하는거에 대해서는 어떻게 생각하시나요?

return BookingRefundResponse.of(booking);

저는 개인적으로 dto는 데이터 전달에만 집중해야한다고 생각해 get을 dto에서 하는 것이 적합한 것 같지 않습니다..!! response는 클라이언트와 controller 사이의 dto이므로 response에 도메인쪽 import를 추가하지 않는 게 낫지 않을까요?

혜린님 말씀대로 "dto는 데이터 전달에만 집중해야한다!" 라는 말에 동의합니다.
"dto의 매개변수를 객체로 줄것인가 or 필요한 필드만 줄것인가"에 있어서는 혜린님이 말씀해주신 관점이 더 와닿는 같습니다 ㅎㅎ

제가 말씀드렸던 이유는 getter zone이 Service 계층에서 발생하는게 객체지향적이지 못한 코드라고 생각이 들었습니다. 도메인 객체의 내부 상태를 서비스 계층에서 과도하게 노출하는 게 아닌가? 라는 생각이 들기도 합니다.

그렇다면 getter 없이 DTO가 데이터 전달에만 집중해서 사용하는 방법에는 어떤게 있을 지 찾아봤는데, MapStruct나 커스텀 매퍼 클래스를 사용하는 방법이 있더라고요!
이 방법 더 찾아보고 추가 코멘트 남겨보도록 하겠습니다.

저도 최근 이 부분에 대해서 공부하는 중입니다!
https://github.com/Nexters/gaetteok-backend
엔티티와 도메인 분리 및 mapper 사용한 팀의 코드를 참고해서
SOPT-all/35-COLLABORATION-SERVER-SAFETYREPORT#8
최근 이런 방식으로 다른 프로젝트의 코드를 구성해봤었습니다~
https://github.com/AND-SOPT-SERVER/yeongju.cho/tree/seminar2-daehwan
이 코드의

일기 작성

일기 상세 조회

전체 일기 목록 조회
api 구현에서 controller 부터 아래로 가는 흐름, 도메인간 분리, 예외 처리 부분도 보시면 좋은 참고가 될 것 같습니다 :)

공부해서 추후 리팩토링 하면 좋을 것 같네요 😊

hoonyworld · 2024-11-26T17:53:41Z

src/main/java/com/beat/domain/booking/application/BookingCancelService.java

+		Booking booking = bookingRepository.findById(request.bookingId())
+			.orElseThrow(() -> new NotFoundException(BookingErrorCode.NO_BOOKING_FOUND));
+
+		booking.setBookingStatus(BookingStatus.BOOKING_CANCELLED);


setter를 호출하는 대신 booking.cancelBooking() 이라는 메서드를 만들고 해당 메서드 내부에서 BookingStatus를 BOOKING_CANCELLED로 바꾸어주면 좀더 캡슐화된 코드를 작성할 수 있을 것 같아요

상단 답변 코멘트와 마찬가지로 setter 호출이 아닌 엔티티에 직접 작성한 메소드를 호출해서 사용했습니다!

hoonyworld · 2024-11-26T17:54:58Z

src/main/java/com/beat/domain/booking/application/BookingCancelService.java

+		return BookingCancelResponse.of(
+			booking.getId(),
+			booking.getBookingStatus()


이 부분은 매개변수 개수가 2개 밖에 없어서 매개변수를 넘겨도 될 것 같긴하나, 만약 위에서 booking 객체를 넘기는 방식으로 바꾼다면 통일성을 위해 해당 부분도 바꾸는게 좋아보이는데 혜린님 생각은 어떠신가요?

hoonyworld · 2024-11-26T17:55:28Z

src/main/java/com/beat/domain/booking/application/GuestBookingService.java

+			null,
+			null,
+			null,


null 처리하신 이유가 궁금합니다.

환불계좌 정보는 환불요청했을 시에만 예매자가 등록하게 되기 때문에 처음 booking을 생성할 때는 null로 들어가게 됩니다.

hyerinhwang-sailin added 19 commits November 25, 2024 13:38

[#257] feat(Performance): 엔티티 수정

e56a150

[#257] feat(PerformanceErrorCode): 유의사항, 소개 글자수 1500자로 수정

d5d9445

[#257] chore(ScheduleRequest): 코드 포맷터 적용

42f9920

[#257] feat(PerformanceDetailResponse): 공연 상세 정보 dto 수정

c3bb5a5

[#257] feat(PerformanceService): 공연 상세 정보 service 수정

5826f62

[#257] feat: 공연 수정 dto 수정

630bcee

[#257] feat(PerformanceModifyService): 공연 수정 service 수정

c12fc4a

[#257] feat: 공연 생성 dto 수정

ba35326

[#257] feat(PerformanceManagementService): 공연 생성 service 수정

0a20268

[#257] chore: longtitude 오타 수정

fd90186

[#259] feat(Booking): Booking에 필드 추가 및 setRefundInfo 메소드 추가

cae8909

[#259] feat(BookingStatus): BookingStatus 값 추가

3bf6f39

[#259] feat(BookingSuccessCode): BookingSuccessCode 값 추가

fffafcb

[#259] feat: 예매취소요청 dto 추가

3a8d5be

[#259] feat: 예매환불요청 dto 추가

7569aac

[#259] feat(BookingCancelService): 예매 취소, 환불요청 service 추가

b7ea086

[#259] feat: 예매 취소, 환불요청 controller 추가

b108620

[#259] feat(SecurityConfig): 예매 취소, 환불요청 url AUTH_WHITELIST에 추가

45f722a

[#259] feat: booking 엔티티 변경사항 반영

689e8d5

hyerinhwang-sailin added feat 🎯 feat 혜린🍀 labels Nov 25, 2024

hyerinhwang-sailin requested a review from hoonyworld November 25, 2024 18:08

hyerinhwang-sailin self-assigned this Nov 25, 2024

hyerinhwang-sailin linked an issue Nov 25, 2024 that may be closed by this pull request

[feat] Booking 엔티티 수정 및 환불/취소 요청 API 구현 #259

Closed

3 tasks

hyerinhwang-sailin temporarily deployed to dev November 25, 2024 18:08 — with GitHub Actions Inactive

hyerinhwang-sailin temporarily deployed to dev November 26, 2024 08:02 — with GitHub Actions Inactive

hyerinhwang-sailin temporarily deployed to dev November 26, 2024 08:17 — with GitHub Actions Inactive

hyerinhwang-sailin temporarily deployed to dev November 26, 2024 08:23 — with GitHub Actions Inactive

hyerinhwang-sailin temporarily deployed to dev November 26, 2024 08:33 — with GitHub Actions Inactive

hyerinhwang-sailin temporarily deployed to dev November 26, 2024 08:40 — with GitHub Actions Inactive

hyerinhwang-sailin temporarily deployed to dev November 26, 2024 08:47 — with GitHub Actions Inactive

hoonyworld reviewed Nov 26, 2024

View reviewed changes

hoonyworld previously approved these changes Nov 28, 2024

View reviewed changes

Merge branch 'develop' into feat/#259

6b18618

hyerinhwang-sailin dismissed hoonyworld’s stale review via 6b18618 November 28, 2024 09:49

hyerinhwang-sailin had a problem deploying to dev November 28, 2024 09:49 — with GitHub Actions Failure

[#261] chore(modifyPerformance): conflict 해결

8578b82

hyerinhwang-sailin temporarily deployed to dev November 28, 2024 09:58 — with GitHub Actions Inactive

hoonyworld approved these changes Nov 28, 2024

View reviewed changes

hyerinhwang-sailin merged commit 9aae2b3 into develop Nov 28, 2024
1 check passed

hoonyworld deleted the feat/#259 branch January 13, 2025 13:09

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[feat] #259 - Booking 엔티티 수정 및 환불/취소 요청 API 구현 #260

[feat] #259 - Booking 엔티티 수정 및 환불/취소 요청 API 구현 #260

hyerinhwang-sailin commented Nov 25, 2024 •

edited

Loading

hoonyworld left a comment

hoonyworld Nov 26, 2024

hyerinhwang-sailin Nov 27, 2024 •

edited

Loading

hoonyworld Nov 28, 2024

hoonyworld Nov 28, 2024

hyerinhwang-sailin Nov 28, 2024

hoonyworld Nov 26, 2024

hyerinhwang-sailin Nov 27, 2024

hoonyworld Nov 28, 2024

hyerinhwang-sailin Nov 28, 2024 •

edited

Loading

hoonyworld Nov 28, 2024

hoonyworld Nov 26, 2024

hyerinhwang-sailin Nov 27, 2024

hoonyworld Nov 26, 2024

hoonyworld Nov 26, 2024

hyerinhwang-sailin Nov 27, 2024

		booking.setRefundInfo(request.bankName(), request.accountNumber(), request.accountHolder());
		booking.setBookingStatus(BookingStatus.REFUND_REQUESTED);

[feat] #259 - Booking 엔티티 수정 및 환불/취소 요청 API 구현 #260

[feat] #259 - Booking 엔티티 수정 및 환불/취소 요청 API 구현 #260

Conversation

hyerinhwang-sailin commented Nov 25, 2024 • edited Loading

Related issue 🛠

Work Description ✏️

Trouble Shooting ⚽️

Related ScreenShot 📷

Uncompleted Tasks 😅

To Reviewers 📢

hoonyworld left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

hyerinhwang-sailin Nov 27, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

hyerinhwang-sailin Nov 28, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

hyerinhwang-sailin commented Nov 25, 2024 •

edited

Loading

hyerinhwang-sailin Nov 27, 2024 •

edited

Loading

hyerinhwang-sailin Nov 28, 2024 •

edited

Loading