[1, 2, 3 단계 - 방탈출 예약 관리] 미아(이종미) 미션 제출합니다. #4

jongmee · 2024-04-16T08:56:47Z

안녕하세요 수달! Level 2 첫 미션 잘 부탁드립니다 🙇🏻‍♀️

웹 개발 경험

웹 어플리케이션의 Restful 서버를 만들고 배포해 본 경험이 2회 있습니다. (Thymeleaf를 활용한 프로젝트 경험은 없습니다.)
미션에서 사용할 Springboot, Java, JPA, H2를 사용해본 적 있습니다.
Springboot 공부 및 프로젝트를 시작한 지는 1년이 조금 넘었습니다.

Co-authored-by: jongmee <ra123221@gmail.com>

her0807

안녕하세요 미아!

리뷰어 수달입니다 ㅎㅎ
빠르게 구현해주셔서 놀랐어요~
커밋 시간을 보니 어제 2시부터 10시 까지 내리 했던데
열정이 대단합니다.

미아는 스프링에 대해서 구현 경험이 있다고 하셨는데요!
그래서 적절한 annotation 을 사용해서 spring 으로 view 를 적절히 반환하도록 요구사항이 충족된 것 같아요. 아주 잘하셨습니다.

그런데 레벨 1 때 저희가 연습했던 도메인 객체에 대해서 직접 요구사항을 도출해보고 검증하고, 단위테스트를 해보았던 것들이 코드에서 잘 보이지 않더라고요 ㅠㅠ

물론 요구사항에 적혀있지 않아서 그럴수도 있지만, 저희는 점진적으로 더 나음을 도모해야하니까요! 레벨 1 때 것들도 함께 챙겨나가볼까요?

README.md

src/main/java/roomescape/repository/ReservationRepository.java

src/test/java/roomescape/MissionStepTest.java

src/main/java/roomescape/domain/Reservation.java

src/main/java/roomescape/controller/AdminController.java

her0807 · 2024-04-17T11:40:43Z

src/main/java/roomescape/controller/AdminController.java

+import java.util.concurrent.atomic.AtomicLong;
+
+@Controller
+public class AdminController {


Controller 에서 repository 를 바로 의존하고 있는데요,

그러다보니 도메인에서 DTO 를 만드는 형태로 간결함에 초점을 두고 개발해주신 것 같아요!!

가독성 좋고 읽기 깔끔하지만, 도메인에 뷰의 변경에 따른 전파가 가기 때문에

service layer 를 두어 검증과 계층 별 의존성 분리 역할을 주면 어떨까요? 😁

기능 요구 사항을 만족하는 간결한 프로그램을 생각하며 구현하다 보니 전체적으로 도메인이 뷰의 변경에 영향을 받는 구조가 되었네요 🥲

Service layer를 추가하여 계층 분리를 하고, 검증(예약 id 검증)을 service layer에서 하도록 변경하였습니다. 우선 Presentation 계층인 controller만 dto를 사용하도록 하여 service에서는 dto를 생성하거나, dto에서 entity를 변환하는 로직 등은 존재하지 않도록 구현하였는데, 수달은 이에 대해 어떻게 생각하시나요?

저는 DTO의 역할이 아래와 같다고 생각합니다. (DTO를 controller에서만 사용한 이유)

뷰의 변경에 도메인이 영향을 받지 않고, 도메인 자체를 뷰에 노출시키지 않는다.

현재 DTO(ReservationResponse, ReservationSaveRequest)는 요청과 응답에서 데이터를 전달하고, 컨트롤러의 역할에서 필요하다.

이것저것 레퍼런스를 찾아보니 의견이 나뉘는 것 같은데(controller에서만 사용한다, controller와 service에서 모두 사용한다) 수달은 현재 현업에서 어떤 의견을 가지고 사용하고 계신지 궁금합니다!

좋은 고민이네요!

저는 dto는 controller에서 dto -> domain 으로 변환하여 의존성을 끊어서 사용하고 있어요.
이유는 미아가 말씀해주신 영역도 있고, service layer 가 여러 곳에서 재사용될 여지가 큰데, dto 의 형태가 일정하지 않으니
재사용성에도 좋지 않고, 뷰 변경에 대한 영향도 전파되기 때문이에요!

그러나 재사용성이 적고, 뷰와 도메인이 항상 핏하게 떨어질 경우 혹은 개발 초기 단계에 빠르게 치고 나가야하는 경우에서는
controller와 service 모두 혼용해서 사용하기도 하는 것 같아요.

transfer 한다는 것은 여러 고민들이 수반되어야하기 때문인데요. (어느곳에서 어떤 형태로 변환해줄것인지 등등 또 얼마나 잘게 나눌 것인가에 대한 고민도요!)

다만,service 에서 controller 로 응답값을 반환할 때는 domain -> reseponse dto 로 꼭 변환해주는 편이에요.
controller 는 model 과 view 사이에 중개자 역할을 하는 것인데 도메인을 반환하게되면
controller 영역에서 의도치 않게 도메인에 상태를 변경할 수도 있기 때문인데요! JPA 를 써보셨다고 하셔서 첨언드리면
JPA 의 장점 중 하나인 Dirty Checking 이 일어나서 의도치 않게 변경점이 DB 까지 이어지기도 하고요!

src/main/java/roomescape/controller/AdminController.java

…r 분리

jongmee · 2024-04-18T12:29:06Z

안녕하세요 수달! 리뷰에 코멘트 및 질문 남겨놓았습니다 :)

추가로, 리팩토링 및 추가 검증을 구현하면서 최대한 단위 테스트를 진행하려고 하였습니다!

her0807

안녕하세요 미아! ㅎㅎ

워밍업 제대로 된 것 같은데요? 🎉
질문주신 내용과 간단한 코멘트들 남겼습니다 :)
해당 코멘트들은 다음 단계 진행하면서 함께 반영해주시면 좋을 것 같아요👍🏻

her0807 · 2024-04-19T00:28:52Z

src/main/java/roomescape/controller/ReservationController.java

+
+import java.util.List;
+
+@RestController


her0807 · 2024-04-19T00:31:25Z

src/main/java/roomescape/controller/ReservationController.java

+
+    @PostMapping
+    public ResponseEntity<ReservationResponse> createReservation(@RequestBody ReservationSaveRequest request) {
+        var reservation = request.toReservation();


dto, -> domain 변환할 때 toModel 이라는 메서드명을 자주 사용하는 것 같아요 🙂

https://stackoverflow.com/questions/63679542/generic-mapper-function-in-mapstruct

her0807 · 2024-04-19T00:31:44Z

src/main/java/roomescape/controller/ReservationController.java

+import java.util.List;
+
+@RestController
+@RequestMapping("/reservations")


공통된 url 를 묶어두셨군요 👍🏻 잘하셨습니다

src/main/java/roomescape/domain/Reservation.java

her0807 · 2024-04-19T00:33:56Z

src/main/java/roomescape/service/ReservationService.java

+        if (existingSameUser) {
+            throw new IllegalArgumentException("동일한 시간에 같은 사용자가 예약할 수 없습니다.");
+        }
+        if (reservationsInSameDateTime.size() >= MAX_RESERVATION_PER_TIME) {
+            throw new IllegalArgumentException("해당 시간대에 예약이 모두 찼습니다. (최대 4팀)");
+        }


검증 로직을 따로 추출해볼까요?

private 메소드로 추출하였습니다 :)

her0807 · 2024-04-19T00:34:08Z

src/main/java/roomescape/service/ReservationService.java

+        return reservationRepository.save(reservation);
+    }
+
+    public void deleteReservation(Long id) {


her0807 · 2024-04-19T00:34:28Z

src/main/resources/application.properties

+server.servlet.encoding.charset=UTF-8
+server.servlet.encoding.enabled=true
+server.servlet.encoding.force=true


오 해당 속성들은 어떤 이유로 추가하셨나요?

처음에 MockMvc로 컨트롤러 슬라이싱 테스트를 할 때 json의 String을 검증했는데 한글 인코딩 때문에 제대로 검증이 되지 않아 해당 설정을 추가했습니다! 하지만 아래처럼 jsonPath로 변경하니(단순 문자열 체크를 하지 않으니) 인코딩 문제가 발생하지 않아 더 이상 해당 설정은 필요 없습니다..!

.andExpect(jsonPath("$[0].name").value(USER_MIA))

삭제해야 했는데 빠뜨렸네요 🥲 삭제하였습니다!

her0807 · 2024-04-19T00:35:50Z

src/test/java/roomescape/domain/ReservationTest.java

+    @DisplayName("이미 초기화된 예약 ID를 초기화(수정)할 경우 예외가 발생한다.")
+    void initializeId() {
+        // given
+        Reservation reservation = new Reservation(USER_MIA, MIA_RESERVATION_DATE, MIA_RESERVATION_TIME);


테스트를 위해 반복 생성되는 객체들은 Test Fixture 로 추출해도 좋을 것 같아요!

그렇네요! Reservation 객체도 TestFixture로 추출하였습니다.

her0807 · 2024-04-19T00:36:19Z

src/test/java/roomescape/service/ReservationServiceTest.java

+    }
+
+    @Test
+    @DisplayName("동일한 시간대에 최대 4팀이 예약할 수 있다. 초과되면 예외가 발생한다.")


귀여운 요구사항을 적용시켰군요 고마워요 🤩

ay-eonii and others added 18 commits April 16, 2024 14:19

docs(README): 기능 요구 사항 추가

09afbfc

Co-authored-by: jongmee <ra123221@gmail.com>

feat(RoomEscapeController): 어드민 메인페이지 응답

279ebf3

Co-authored-by: jongmee <ra123221@gmail.com>

feat(RoomEscapeController): 어드민 조회페이지 응답

12095bd

Co-authored-by: jongmee <ra123221@gmail.com>

feat(RoomEscapeController): 예약 목록 조회 API 현

1799a5e

Co-authored-by: jongmee <ra123221@gmail.com>

feat(ReservationDto): Reservation dto 생성

089d78a

Co-authored-by: jongmee <ra123221@gmail.com>

feat(RoomEscapeController): 예약 목록 페이지 동적으로 조회

8b50980

Co-authored-by: jongmee <ra123221@gmail.com>

feat(RoomEscapeController): 예약 생성 API 구현

222679c

Co-authored-by: jongmee <ra123221@gmail.com>

refactor(ReservationResponse): ReservationResponse DTO 이름 변경

52b2421

Co-authored-by: jongmee <ra123221@gmail.com>

feat(RoomEscapeController): 예약 삭제 API 구현

c26c91e

Co-authored-by: jongmee <ra123221@gmail.com>

feat(RoomEscapeController): Thymeleaf 제거

5ffd652

Co-authored-by: jongmee <ra123221@gmail.com>

refactor(Reservation): getter 제거

22c8278

Co-authored-by: jongmee <ra123221@gmail.com>

feat(RoomEscapeController): 멤버변수 final 추가

2231ad2

Co-authored-by: jongmee <ra123221@gmail.com>

docs(README): 기능 요구 사항 완료 체크

4dad07c

Co-authored-by: jongmee <ra123221@gmail.com>

refactor(Reservation): 불변 객체로 변경

5202a2c

Co-authored-by: jongmee <ra123221@gmail.com>

refactor(RoomEscapeController): 변수명 변경

04242f6

Co-authored-by: jongmee <ra123221@gmail.com>

refactor(ReservationRepository): 레포지토리 분리

168ee69

Co-authored-by: jongmee <ra123221@gmail.com>

refactor(AdminController): controller 이름 변경, 변수 타입 변경

247bf2c

refactor(AdminController): 메소드명 변경

f81680d

her0807 requested changes Apr 17, 2024

View reviewed changes

her0807 reviewed Apr 17, 2024

View reviewed changes

src/main/java/roomescape/controller/AdminController.java Outdated Show resolved Hide resolved

jongmee added 10 commits April 17, 2024 23:43

refactor(ReservationRepository): 데이터를 Map에 저장

f063abf

refactor(ReservationRepository): 존재하지 않는 데이터 예외 처리 controller 역할로 변경

b332120

refactor(ReservationRepository): ConcurrentHashMap으로 자료구조 변경

9853201

refactor(AdminController): Reservation index를 Repository에서 관리

aac76cb

refactor(ReservationService): Service layer 생성

96c1097

refactor(Reservation): 사용하지 않는 메소드 삭제

6a70b9d

refactor(Reservation): domain에서 dto 의존성 제거

38ac066

feat(Reservation): Reservation 필드 검증

ab45376

feat(Reservation): Reservation 필드 검증 추가

e8e2b30

fix(Reservation): 불필요한 검증 삭제

b393ca1

jongmee added 12 commits April 18, 2024 13:38

fix(ReservationController): response 반환 controller와 view 반환 controlle…

5614c64

…r 분리

test(ReservationControllerTest): ReservationController 슬라이싱 테스트

caff1ae

test(AdminControllerTest): AdminController 슬라이싱 테스트

529e08d

docs(README): 기능 요구 사항 구체화

3a6e052

test(TestFixture): 테스트픽스쳐 생성

d36343e

feat(Reservation): 예약 시간 단위 검증 추가

349f600

test(ReservationServiceTest): 예약 삭제시 예외 발생 로직 테스트

facd41a

feat(ReservationRepository): 동일 시간에 최대 4팀까지 예약 가능하도록 검증 추가

4129861

test(ReservationTest): 예약자 이름 테스트 범위 추가

ef59632

docs(README): 추가된 비즈니스 기능 기록

4240335

refactor(ReservationService): 동일한 시간대의 예약 제한 검증을 서비스 계층에서 진행

af3a432

feat(ReservationService): 동일한 시간대의 같은 사용자가 예약 불가 검증 추가

e0a772f

jongmee added 2 commits April 18, 2024 22:04

test(MissionStepTest): 테스트메소드명 변경 및 다이나믹 테스트 도입

58bf398

refactor(ReservationController): 중복되는 uri 분리

ec6abfb

her0807 approved these changes Apr 19, 2024

View reviewed changes

her0807 merged commit 3857d35 into woowacourse:jongmee Apr 19, 2024

jongmee mentioned this pull request Apr 21, 2024

[4 - 9단계 방탈출 예약 관리] 미아(이종미) 미션 제출합니다. #95

Merged


		import java.util.List;

		@RestController

[1, 2, 3 단계 - 방탈출 예약 관리] 미아(이종미) 미션 제출합니다. #4

[1, 2, 3 단계 - 방탈출 예약 관리] 미아(이종미) 미션 제출합니다. #4

Uh oh!

Conversation

jongmee commented Apr 16, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

웹 개발 경험

Uh oh!

her0807 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

jongmee commented Apr 18, 2024

Uh oh!

her0807 left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

jongmee commented Apr 16, 2024 •

edited

Loading