🚀 2단계 - 요금 조회 #482

seonje1 · 2024-03-13T17:52:46Z

No description provided.

bactoria

선제님 안녕하세요! 😀
이번 단계 잘 구현 해주셨는데요.

이번 단계의 요구사항에 누락된 부분이 있어,
코멘트 참고하시어 반영해 주세요! 💪 💪

bactoria · 2024-03-16T03:47:40Z

src/main/java/nextstep/subway/path/PathFinder.java

@@ -37,11 +37,11 @@ public PathResponse getPath(List<Sections> sectionsList, Station sourceStation,
                            .map(StationResponse::ofEntity)
                            .collect(Collectors.toList()),
                    distance,
-                    duration
+                    duration,
+                    FareCalculator.calculate(distance)


지하철 요금은 최단거리 기준으로 부과하는 것이 정책인데요.
현재 출발역/도착역이 같더라도 조회하는 경로 타입에 따라 요금이 달라지는 것 같아 수정이 필요 해 보여요.

bactoria · 2024-03-16T03:51:05Z

src/main/java/nextstep/subway/path/PathResponse.java

        this.stations = stations;
        this.distance = distance;
        this.duration = duration;
+        this.fare = fare;


현재 경로를 구할 때 응답으로 요금을 함께 계산하여 반환하고 있네요.

요금을 계산하는 역할은 경로를 찾는 PathFinder 에 두기에는 과한 책임을 부과한 것 으로 보여져요.

거리를 이용하여 요금을 계산하는 클래스를 만들어 리팩토링 해보면 어떨까요? 😊

bactoria · 2024-03-16T03:54:01Z

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

+    /**
+     * Given 지하철 구간을 등록하고
+     * When 경로를 조회하면
+     * Then 출발역과 도착역까지의 요금을 조회한다
+     */
+    @DisplayName("출발역과 도착역까지의 요금 조회")


인수테스트를 잘 작성 해 주셨네요 👍
큐컴버를 이용한 인수테스트가 요구사항에 존재하는데요!
큐컴버를 이용해서 만들어보는 건 어떨까요?

bactoria · 2024-03-16T03:56:58Z

src/main/java/nextstep/subway/path/FareCalculator.java

+public class FareCalculator {
+
+    private static final int BASE_FARE = 1250;
+    private static final int OVER_50KM_FARE = 2150;


SURCHARGE 가 변경된다면 OVER_50KM_FARE 도 변경이 필요하겠네요.
이는 유지보수에 취약할 수 있어보이지만, 단위테스트를 만들어주셔서 지금도 괜찮을 것 같습니다!

(개인적으로는 단위테스트를 더욱 꼼꼼하게 만들기 위해 @ParameterizedTest 를 이용하여 더욱 세세하게 테스트 케이스를 추가했을 것 같아요)

seonje1 added 3 commits March 14, 2024 02:04

피드백 반영

49d7532

요금 조회 인수테스트 작성

76c5968

요금 조회 기능 구현 밑 단위테스트 작성

1b2a21c

bactoria requested changes Mar 16, 2024

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

🚀 2단계 - 요금 조회 #482

🚀 2단계 - 요금 조회 #482

seonje1 commented Mar 13, 2024

bactoria left a comment

bactoria Mar 16, 2024

bactoria Mar 16, 2024

bactoria Mar 16, 2024

bactoria Mar 16, 2024

🚀 2단계 - 요금 조회 #482

Are you sure you want to change the base?

🚀 2단계 - 요금 조회 #482

Conversation

seonje1 commented Mar 13, 2024

bactoria left a comment

Choose a reason for hiding this comment

bactoria Mar 16, 2024

Choose a reason for hiding this comment

bactoria Mar 16, 2024

Choose a reason for hiding this comment

bactoria Mar 16, 2024

Choose a reason for hiding this comment

bactoria Mar 16, 2024

Choose a reason for hiding this comment