🚀 2단계 - 요금 조회 #124

loop-study · 2022-02-20T09:38:30Z

안녕하세요 리뷰어님!
박현철입니다.
요금계산을 Enum으로 한번에 다 처리하도록 진행해봤습니다.
Enum 사용이 미숙해서 확인 부탁드리겠습니다.
감사합니다.

2022. 02. 17. README.md - 요금 조회 요구사항 정리 Resolves : loop-study See also : junwoochoi

2022. 02. 17. PathAcceptanceTest - 기존 테스트에 요금 검증 추가 Resolves : loop-study See also : junwoochoi

2022. 02. 17. PathDocumentation - 요금 검증 테스트 추가 PathAcceptanceTest - 요금 검증 금액 수정 Resolves : loop-study See also : junwoochoi

2022. 02. 20. SubwayMapTest - 다중 검증 테스트를 assertAll로 묶음 Resolves : loop-study See also : junwoochoi

2022. 02. 20. PathResponse - 요금 필드 추가. Resolves : loop-study See also : junwoochoi

2022. 02. 20. PathResponse - 9 -> 10으로 변경 Resolves : loop-study See also : junwoochoi

2022. 02. 20. PathFare - 금액 계산하는 enum 구현 Path - 기존 로직 수 PathResponse - 누락된 getter 추가 Resolves : loop-study See also : junwoochoi

2022. 02. 20. PathDocumentation - 불필요한 테스트 제거 Resolves : loop-study See also : junwoochoi

2022. 02. 20. PathFareTest - 거리별 요금 테스트 추가 Resolves : loop-study See also : junwoochoi

2022. 02. 20. PathSteps - 요금 필드 추가 Resolves : loop-study See also : junwoochoi

2022. 02. 21. PathFare - 네이밍 수정 Resolves : loop-study See also : junwoochoi

junwoochoi

안녕하세요! 현철님! :)
인수테스트 리팩토링 및 도메인 테스트 잘 생성해서 미션 진행해주신 부분 매우 좋았습니다 👍 😄
다만, 요구사항이랑 다르게 구현이 되어있는 부분이 있어, 코멘트 남겼으니 확인 부탁드립니다 🙏

src/main/java/nextstep/subway/applicaion/dto/PathResponse.java

src/main/java/nextstep/subway/domain/PathFare.java

src/test/java/nextstep/subway/acceptance/PathAcceptanceTest.java

src/test/java/nextstep/subway/unit/PathFareTest.java

junwoochoi · 2022-02-21T09:23:05Z

src/test/java/nextstep/subway/acceptance/PathAcceptanceTest.java

                ()-> assertThat(response.jsonPath().getInt("duration")).isEqualTo(5),
-                ()-> assertThat(response.jsonPath().getInt("distance")).isEqualTo(13)
+                ()-> assertThat(response.jsonPath().getInt("fare")).isEqualTo(1350)


`최단 시간 경로`로 요청을 준 경우에도, `최단 거리 경로`기준으로 금액이 산정되어야 하는데, 현재 구현하신 기능은 요청이 들어온 PathType의 요금을 계산해서 알려주고 있네요 😭 요 부분 수정 및 테스트 추가가 필요해 보입니다

허허헣.... 이게 이런 의미였군요....

src/test/java/nextstep/subway/documentation/PathSteps.java

2022. 02. 21. Path - 요금 계산용 거리 필드 추가 - 최단 시간의 거리와 구분하기 위한 생성자 추가 PathService - 최단 시간일 경우 최단 경로의 거리를 받기위한 조건문 추가 PathAcceptanceTest - 최단 시간이 아닌 최단 경로로 요금 구하는 테스트 추가 Resolves : loop-study See also : junwoochoi

loop-study · 2022-02-21T12:01:51Z

안녕하세요 리뷰어님!
박현철입니다.
최단 시간의 거리가 아닌 최단 거리로 구할 수 있도록 로직을 변경했습니다.
인수테스트 또한 잠실역을 추가하여 거리와 시간 괴리감으로 확인할 수 있도록 추가했습니다.

그리고 SubwayMap을 한번만 호출해서 해결할 방법이 없을까 고민을 해봤지만
PathService에서 조건문으로 한번 더 SubwayMap를 호출해서 최단 거리를 넣어준 것이 최선이였습니당.
괜찮은 방법이 있는지 확인 부탁드립니다. 🙈
감사합니다.

junwoochoi

안녕하세요! 요구사항 다시 수정해주신 부분 잘 확인했습니다!
추가로 의견 코멘트 드렸는데, 고민해보시면 좋을 것 같습니다.
다음 미션 이어서 진행해주세요! 🚀

junwoochoi · 2022-02-21T13:21:37Z

src/main/java/nextstep/subway/applicaion/PathService.java

+        if (PathType.DURATION == type) {
+            // 최단 시간의 거리가 아닌 최단 경로의 거리 구하기
+            Path shortestDistancePath = new SubwayMap(lines, PathType.DISTANCE).findPath(upStation, downStation);
+            path = path.changeShortestDistancePath(shortestDistancePath);


🧐 path에 필드 변수로 shortestDistance를 넣는 것보다는 PathResponse.of(path, shortestPath.extractFare())와 같은 식으로 PathResponse에 따로 넣는 것도 좋을 것 같아요. Path를 사용하다보면 Path도메인이 매번 shortestDistance와 함께 사용되지는 않을 수도 있을 것 같다는 개인적인 의견입니다!

Path를 변경하지 않고 편하게 사용이 가능하네요 👍

src/test/java/nextstep/subway/acceptance/PathAcceptanceTest.java

loop-study added 11 commits February 20, 2022 14:36

docs : step2 요구사항 정리

5f2e48d

2022. 02. 17. README.md - 요금 조회 요구사항 정리 Resolves : loop-study See also : junwoochoi

test : 실패하는 요금 인수테스트

c7a20f4

2022. 02. 17. PathAcceptanceTest - 기존 테스트에 요금 검증 추가 Resolves : loop-study See also : junwoochoi

test : 요금 테스트 추가

eb7c9c5

2022. 02. 17. PathDocumentation - 요금 검증 테스트 추가 PathAcceptanceTest - 요금 검증 금액 수정 Resolves : loop-study See also : junwoochoi

refactor : 요금 테스트 추가

aba33eb

2022. 02. 20. SubwayMapTest - 다중 검증 테스트를 assertAll로 묶음 Resolves : loop-study See also : junwoochoi

feat : 요금 필드 추가

9dedecd

2022. 02. 20. PathResponse - 요금 필드 추가. Resolves : loop-study See also : junwoochoi

refactor : 잘못된 요금 기준 수정.

48434eb

2022. 02. 20. PathResponse - 9 -> 10으로 변경 Resolves : loop-study See also : junwoochoi

feat : 요금 계산 구현

54eacf0

2022. 02. 20. PathFare - 금액 계산하는 enum 구현 Path - 기존 로직 수 PathResponse - 누락된 getter 추가 Resolves : loop-study See also : junwoochoi

test : 문서 테스트 수정

fe9590f

2022. 02. 20. PathDocumentation - 불필요한 테스트 제거 Resolves : loop-study See also : junwoochoi

test : 요금 단위 테스트 추가

3014135

2022. 02. 20. PathFareTest - 거리별 요금 테스트 추가 Resolves : loop-study See also : junwoochoi

refactor : 누락된 문서 정보 추가

6c504db

2022. 02. 20. PathSteps - 요금 필드 추가 Resolves : loop-study See also : junwoochoi

refactor : 네이밍 수정

47081e8

2022. 02. 21. PathFare - 네이밍 수정 Resolves : loop-study See also : junwoochoi

junwoochoi requested changes Feb 21, 2022

View reviewed changes

junwoochoi approved these changes Feb 21, 2022

View reviewed changes

junwoochoi merged commit 171d3df into next-step:loop-study Feb 21, 2022

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

🚀 2단계 - 요금 조회 #124

🚀 2단계 - 요금 조회 #124

loop-study commented Feb 20, 2022

junwoochoi left a comment

junwoochoi Feb 21, 2022

loop-study Feb 21, 2022

loop-study commented Feb 21, 2022 •

edited

junwoochoi left a comment

junwoochoi Feb 21, 2022

loop-study Feb 22, 2022

🚀 2단계 - 요금 조회 #124

🚀 2단계 - 요금 조회 #124

Conversation

loop-study commented Feb 20, 2022

junwoochoi left a comment

Choose a reason for hiding this comment

junwoochoi Feb 21, 2022

Choose a reason for hiding this comment

loop-study Feb 21, 2022

Choose a reason for hiding this comment

loop-study commented Feb 21, 2022 • edited

junwoochoi left a comment

Choose a reason for hiding this comment

junwoochoi Feb 21, 2022

Choose a reason for hiding this comment

loop-study Feb 22, 2022

Choose a reason for hiding this comment

loop-study commented Feb 21, 2022 •

edited