[feat]: 공연,날짜,회차에 대한 좌석 조회 API 추가 #42

ssssujini99 · 2023-12-27T08:00:19Z

closed [feat]: 사용자는 공연의 특정 날짜에 예매 가능한 회차와 좌석을 조회할 수 있다 #36

📄 무엇을 개발했나요?

공연 좌석 정보 조회 controller, application, repository 구현 및 각 레이어 별 테스트 구현 완료

🍸 무엇을 집중적으로 리뷰해야할까요?

네이밍, 컨벤션 확인
application 비즈니즈 로직 확인
테스트에서 중복되는 객체 생성에 대해 fixture로 빼고 리팩토링까지 함께 진행했습니다

github-actions · 2023-12-27T08:01:44Z

🦞 Test Coverage Report

Overall Project	NaN% `NaN%`	🟢

There is no coverage information present for the Files changed

github-actions · 2023-12-27T08:15:34Z

🦞 Test Coverage Report

Overall Project	NaN% `NaN%`	🟢

There is no coverage information present for the Files changed

EunChanNam

고생하셨습니다!! 👏
이번 리뷰는 좀 고봉밥이네요 ...ㅎㅎ
내일 스크럼하고 같이 리뷰 얘기나눠봅시당 ㅎㅎ

EunChanNam · 2023-12-27T16:23:59Z

api/src/main/java/dev/hooon/show/ShowSeatsApiController.java

+		@RequestParam("date") @DateTimeFormat(iso = DateTimeFormat.ISO.DATE) LocalDate date,
+		@RequestParam("round") int round


이런건 Request dto 를 만드는게 어떨까요??

EunChanNam · 2023-12-27T16:27:10Z

core/src/main/java/dev/hooon/show/application/ShowSeatsService.java

+		List<SeatsInfoDto> seatsInfoDtoList = seatRepository.findSeatInfoByShowIdAndDateAndRound(showId,
+			date, round);


파라미터를 한줄에 나열하거나 너무길면 한라인씩 분리해서 표현하는게 보기 좋을거같습니다 ..! ㅎㅎ

반영완료했습니다!

EunChanNam · 2023-12-27T16:29:31Z

core/src/main/java/dev/hooon/show/application/ShowSeatsService.java

+	private void throwIfShowDoesNotExist(Long showId) {
+		showRepository.findById(showId).orElseThrow(
+			() -> new NotFoundException(ShowErrorCode.SHOW_NOT_FOUND)
+		);
+	}


Seat 에 대한 조회인데 저는 이부분에서 Show 에대한 검증이 없어도 될거같아요 🤔
그리고 해야된다면 메소드 네이밍은 검증의 역할이 강한거같은데 validateShowExist 이렇게 가는게 어떨까요?

throwIfShowDoesNotExist 라고 했는데 (show가 존재하지 않으면 예외를 던진다) 라는 의미입니다!! 혹시 어떠신가요?

EunChanNam · 2023-12-27T16:31:59Z

core/src/main/java/dev/hooon/show/domain/entity/seat/Seat.java

+	@Override
+	public boolean equals(Object o) {
+		if (this == o)
+			return true;
+		if (o == null || getClass() != o.getClass())
+			return false;
+		Seat seat = (Seat)o;
+		return isSeat == seat.isSeat && price == seat.price && Objects.equals(show, seat.show)
+			&& seatGrade == seat.seatGrade && Objects.equals(positionInfo, seat.positionInfo)
+			&& Objects.equals(showDate, seat.showDate) && Objects.equals(showRound, seat.showRound)
+			&& seatStatus == seat.seatStatus;
+	}
+
+	@Override
+	public int hashCode() {
+		return Objects.hash(show, seatGrade, isSeat, positionInfo, price, showDate, showRound, seatStatus);
+	}


id 비교도 추가하는게 좋을거같네요!

EunChanNam · 2023-12-27T16:32:42Z

core/src/main/java/dev/hooon/show/domain/repository/SeatRepository.java

+		@Param("showId") Long showId,
+		@Param("date") LocalDate date,
+		@Param("round") int round


퓨어 인터페이스에는 Param 어노테이션 없어도 됩니다!

와 ㅋㅋㅋㅋ 저걸 왜넣었지? ㅋㅋ

EunChanNam · 2023-12-27T16:44:15Z

core/src/test/java/dev/hooon/common/fixture/TestFixture.java

Fixture 클래스를 도메인별로 가져가는건 어떨까요?
일단 저는 도메인으로 나눠서 작성했고 Fixture 클래스 많아지면 관리가 힘들지 않을까 생각합니다....!

아 그리고..! testFixture 모듈로 이전하는건 별로일까요? api 테스트할때 사용할 일이 있을수도 있을거같아서 건의드려요!

저는 더 커지면 그때 리팩토링하며 나누려고 했었습니다..!!

넵 해당 클래스는 testFixture모듈에 있으면 더 좋을것같네요 ㅎㅎ 패키지 위치 옮겨놓았습니당 ㅎㅎ!!

EunChanNam · 2023-12-27T16:47:49Z

core/src/test/java/dev/hooon/show/application/ShowSeatsServiceTest.java

+		given(showRepository.findById(showId)).willReturn(Optional.of(show));
+
+		given(seatRepository.findSeatInfoByShowIdAndDateAndRound(showId, date, round))
+			.willReturn(seatsInfoDtoList);
+		given(seatRepository.findSeatsByShowIdAndDateAndRoundAndGrade(showId, date, round, VIP))
+			.willReturn(vipSeatsDetailDtoList);
+		given(seatRepository.findSeatsByShowIdAndDateAndRoundAndGrade(showId, date, round, S))
+			.willReturn(sSeatsDetailDtoList);
+		given(seatRepository.findSeatsByShowIdAndDateAndRoundAndGrade(showId, date, round, A))
+			.willReturn(aSeatsDetailDtoList);


고생하신 흔적이....보이네요 👏

EunChanNam · 2023-12-27T16:49:32Z

core/src/test/java/dev/hooon/show/infrastructure/adaptor/ShowRepositoryAdaptorTest.java

-		Place place = new Place(
-			"블루스퀘어 신한카드홀",
-			"1544-1591",
-			"서울특별시 용산구 이태원로 294 블루스퀘어(한남동)",
-			"http://www.bluesquare.kr/index.asp"
-		);
+		Place place = TestFixture.getPlace();
 		Place savedPlace = placeRepository.save(place);
-		ShowPeriod showPeriod = new ShowPeriod(LocalDate.of(2023, 10, 10), LocalDate.of(2023, 10, 12));
-		ShowTime showTime = new ShowTime(150, 15);
-		Show show = new Show
-			("레미제라블",
-				MUSICAL,
-				showPeriod,
-				showTime,
-				"만 8세 이상",
-				300,
-				savedPlace
-			);
+
+		Show show = TestFixture.getShow(savedPlace);


깔끔해졌네요! 👍

EunChanNam · 2023-12-27T16:51:02Z

api/src/test/java/dev/hooon/show/ShowSeatsApiControllerTest.java

+			jsonPath("$.seatsInfo").isArray(),
+			jsonPath("$.seatsInfo[0].grade").isString(),
+			jsonPath("$.seatsInfo[0].leftSeats").isNumber(),
+			jsonPath("$.seatsInfo[0].price").isNumber(),
+			jsonPath("$.seatsInfo[0].seats[0].id").isNumber(),
+			jsonPath("$.seatsInfo[0].seats[0].date").isString(),
+			jsonPath("$.seatsInfo[0].seats[0].isBookingAvailable").isString(),
+			jsonPath("$.seatsInfo[0].seats[0].seat").isBoolean(),
+			jsonPath("$.seatsInfo[0].seats[0].positionInfo_sector").isString(),
+			jsonPath("$.seatsInfo[0].seats[0].positionInfo_row").isString(),
+			jsonPath("$.seatsInfo[0].seats[0].positionInfo_col").isNumber(),
+
+			jsonPath("$.seatsInfo").isArray(),
+			jsonPath("$.seatsInfo[1].grade").isString(),
+			jsonPath("$.seatsInfo[1].leftSeats").isNumber(),
+			jsonPath("$.seatsInfo[1].price").isNumber(),
+			jsonPath("$.seatsInfo[1].seats[0].id").isNumber(),
+			jsonPath("$.seatsInfo[1].seats[0].date").isString(),
+			jsonPath("$.seatsInfo[1].seats[0].isBookingAvailable").isString(),
+			jsonPath("$.seatsInfo[1].seats[0].seat").isBoolean(),
+			jsonPath("$.seatsInfo[1].seats[0].positionInfo_sector").isString(),
+			jsonPath("$.seatsInfo[1].seats[0].positionInfo_row").isString(),
+			jsonPath("$.seatsInfo[1].seats[0].positionInfo_col").isNumber(),
+
+			jsonPath("$.seatsInfo").isArray(),
+			jsonPath("$.seatsInfo[2].grade").isString(),
+			jsonPath("$.seatsInfo[2].leftSeats").isNumber(),
+			jsonPath("$.seatsInfo[2].price").isNumber(),
+			jsonPath("$.seatsInfo[2].seats[0].id").isNumber(),
+			jsonPath("$.seatsInfo[2].seats[0].date").isString(),
+			jsonPath("$.seatsInfo[2].seats[0].isBookingAvailable").isString(),
+			jsonPath("$.seatsInfo[2].seats[0].seat").isBoolean(),
+			jsonPath("$.seatsInfo[2].seats[0].positionInfo_sector").isString(),
+			jsonPath("$.seatsInfo[2].seats[0].positionInfo_row").isString(),
+			jsonPath("$.seatsInfo[2].seats[0].positionInfo_col").isNumber()


와우 꼼꼼하시네요 ㅋㅋㅋ 👍

감사합니당 😆

EunChanNam · 2023-12-27T16:53:24Z

core/src/main/java/dev/hooon/show/domain/repository/SeatRepository.java


 public interface SeatRepository {

 	void saveAll(Iterable<Seat> seats);

 	List<SeatDateRoundDto> findSeatDateRoundInfoByShowId(Long showId);
+
+	List<SeatsInfoDto> findSeatInfoByShowIdAndDateAndRound(


DTO 로 필요한 데이터만 조회하는거 좋은거같아요 👍

github-actions · 2023-12-29T03:51:41Z

🦞 Test Coverage Report

Overall Project	NaN% `NaN%`	🟢

There is no coverage information present for the Files changed

EunChanNam

리뷰반영까지 고생하셨습니다~
방금 얘기 나눈것만 반영해주시고 머지하면 될거같아요~ 👏

#36/show-seats-api

github-actions · 2023-12-29T04:55:05Z

🦞 Test Coverage Report

Overall Project	NaN% `NaN%`	🟢

There is no coverage information present for the Files changed

* [test]: ShowSeatsApiController test 추가 * [feat]: 공연 좌석 조회 controller 추가 * [feat]: 공연 좌석 조회 Application 추가 * [refactor]: 좌석 도메인 리팩토링 * [test]: TestFixture 추가 및 기존 코드 리팩토링 * [test]: 공연 좌석 조회 Application 테스트 추가 * [feat]: 공연 좌석 조회 - SeatRepository, SeatRepositoryAdaptor 추가 * [test]: 공연 좌석 조회 - jpa query 테스트, adaptor 테스트 추가 * [docs]: 인수테스트용 sql 더미데이터 파일 추가 * [refactor]: 코드 리팩토링 * [refactor]: 코드 라인 줄맞춤 * [refactor]: EqualsAndHashCode 수정 * [refactor]: 쿼리모델, 응답모델 분리 * [refactor]: 줄맞춤 리팩토링 * [refactor]: sql 코드 라인 줄맞춤 * [refactor]: testFixture 클래스 위치 변경 * [refactor]: 2차 리팩토링

ssssujini99 added 9 commits December 27, 2023 16:31

[test]: ShowSeatsApiController test 추가

a4b5c4a

[feat]: 공연 좌석 조회 controller 추가

3ca04db

[feat]: 공연 좌석 조회 Application 추가

2d462d1

[refactor]: 좌석 도메인 리팩토링

d67ce94

[test]: TestFixture 추가 및 기존 코드 리팩토링

5c71ae8

[test]: 공연 좌석 조회 Application 테스트 추가

d00aa3b

[feat]: 공연 좌석 조회 - SeatRepository, SeatRepositoryAdaptor 추가

d04118c

[test]: 공연 좌석 조회 - jpa query 테스트, adaptor 테스트 추가

7aec39f

[docs]: 인수테스트용 sql 더미데이터 파일 추가

53d365b

ssssujini99 added the ✏️ Feat 기능 개발 label Dec 27, 2023

ssssujini99 added this to the 2차 스프린트 milestone Dec 27, 2023

ssssujini99 self-assigned this Dec 27, 2023

ssssujini99 linked an issue Dec 27, 2023 that may be closed by this pull request

[feat]: 사용자는 공연의 특정 날짜에 예매 가능한 회차와 좌석을 조회할 수 있다 #36

Closed

[refactor]: 코드 리팩토링

7906643

EunChanNam reviewed Dec 27, 2023

View reviewed changes

ssssujini99 and others added 7 commits December 29, 2023 11:49

[refactor]: 코드 라인 줄맞춤

a65dd13

[refactor]: EqualsAndHashCode 수정

af5bbe9

[refactor]: 쿼리모델, 응답모델 분리

00ad503

[refactor]: 줄맞춤 리팩토링

b0172c6

[refactor]: sql 코드 라인 줄맞춤

ab37917

[refactor]: testFixture 클래스 위치 변경

9fddc46

Merge branch 'dev' into feat/#36/show-seats-api

1715ae5

EunChanNam approved these changes Dec 29, 2023

View reviewed changes

ssssujini99 added 2 commits December 29, 2023 13:50

[refactor]: 2차 리팩토링

c4164d4

Merge remote-tracking branch 'origin/feat/#36/show-seats-api' into feat/

cd64c3e

#36/show-seats-api

ssssujini99 merged commit de8ee0d into dev Dec 29, 2023
2 checks passed

EunChanNam deleted the feat/#36/show-seats-api branch December 29, 2023 05:50

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[feat]: 공연,날짜,회차에 대한 좌석 조회 API 추가 #42

[feat]: 공연,날짜,회차에 대한 좌석 조회 API 추가 #42

ssssujini99 commented Dec 27, 2023

github-actions bot commented Dec 27, 2023

github-actions bot commented Dec 27, 2023

EunChanNam left a comment

EunChanNam Dec 27, 2023

EunChanNam Dec 27, 2023

ssssujini99 Dec 29, 2023

EunChanNam Dec 27, 2023

ssssujini99 Dec 29, 2023 •

edited

Loading

EunChanNam Dec 27, 2023

EunChanNam Dec 27, 2023

ssssujini99 Dec 29, 2023

EunChanNam Dec 27, 2023

EunChanNam Dec 27, 2023

ssssujini99 Dec 29, 2023

ssssujini99 Dec 29, 2023

EunChanNam Dec 27, 2023

EunChanNam Dec 27, 2023

EunChanNam Dec 27, 2023

ssssujini99 Dec 29, 2023

EunChanNam Dec 27, 2023

github-actions bot commented Dec 29, 2023

EunChanNam left a comment

github-actions bot commented Dec 29, 2023

		@RequestParam("date") @DateTimeFormat(iso = DateTimeFormat.ISO.DATE) LocalDate date,
		@RequestParam("round") int round

		List<SeatsInfoDto> seatsInfoDtoList = seatRepository.findSeatInfoByShowIdAndDateAndRound(showId,
		date, round);

[feat]: 공연,날짜,회차에 대한 좌석 조회 API 추가 #42

[feat]: 공연,날짜,회차에 대한 좌석 조회 API 추가 #42

Conversation

ssssujini99 commented Dec 27, 2023

📄 무엇을 개발했나요?

🍸 무엇을 집중적으로 리뷰해야할까요?

github-actions bot commented Dec 27, 2023

🦞 Test Coverage Report

github-actions bot commented Dec 27, 2023

🦞 Test Coverage Report

EunChanNam left a 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

ssssujini99 Dec 29, 2023 • 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

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

github-actions bot commented Dec 29, 2023

🦞 Test Coverage Report

EunChanNam left a comment

Choose a reason for hiding this comment

github-actions bot commented Dec 29, 2023

🦞 Test Coverage Report

ssssujini99 Dec 29, 2023 •

edited

Loading