🚀 3단계 - 요금 정책 추가 #488

moonstal1506 · 2024-04-03T12:07:58Z

안녕하세요 준오님!
저번 피드백 자세하게 설명해주셔서 감사했습니다🙏
피드백 적용해서 코드 수정했습니다!
넘 늦었지만.. 끝까지 안하면 후회할 것 같아서 PR 올려봅니다..ㅎㅎ
혹시라도 시간이 되신다면 리뷰 부탁드립니다🙇‍♂️
감사합니다!!

bactoria

수정님 마지막까지 마션 잘 해주셨는데!! 리뷰가 늦었네요 😭

먼저, 미션 사이클에 너무 잘 구현해 주셔서 리뷰하기가 편했어요. 💯
몇가지 코멘트 남겨두었는데 확인 부탁드려요.
반영은 필수로 하지 않으셔도 되는데, 여유가 되신다면 반영해보셔도 좋아요!!🙌
리뷰요청 주시면 머지하도록 할게요! 고생 많으셨습니다 🙇‍♂️

bactoria · 2024-04-07T01:26:10Z

src/main/java/nextstep/line/Line.java

+    public Line(String name, String color, int extraFare, Section section) {
+        this(name, color, section);
+        this.extraFare = extraFare;
+    }


Line40의 extraFare 는 주생성자에서 초기화를 해주어야 할 것 같은데, 보조생성자에서 초기화를 해주고 있네요.
[엘레강트 오브젝트] 1-2장 생성자 하나를 주 생성자로 만드세요 글을 참고하시면 좋을 것 같습니다!

(+ 현재 주생성자는 테스트 코드에서만 사용되고 있는데 제거하고, 보조생성자를 주생성자로 만들면 어떨끼요?)

bactoria · 2024-04-11T12:51:08Z

src/main/java/nextstep/path/Path.java

@@ -9,11 +10,13 @@ public class Path {
    private List<Station> stations;
    private int distance;
    private int duration;
+    private List<Line> usedLine;


A역에서 C역까지의 경로라고 하면 A역 ---------> B역 ---------> C역 으로 볼 수도 있지만,
A역 -----신분당선-----> B역 -----2호선------> C역 으로 생각할 수도 있을 것 같아요.

이번 단계의 노선 추가요금 정책으로 인해 usedLine 가 추가되었네요 👏
다른 방법으로는 stations 대신에 sections(구간)를 넣어준다면 usedLine을 별도로 만들지 않아도 될 것 같아요!
이 방법은 어떠신가요?

경로(Path)는 결국 구간들(Sections)의 일급컬렉션으로 볼 수 있을 것 같아요!
(이렇게 만든다면 distance와 duration 필드도 제거가 가능합니다 😆)

bactoria · 2024-04-11T12:51:55Z

src/main/java/nextstep/path/fare/Fare.java

@@ -1,19 +1,42 @@
 package nextstep.path.fare;

+import java.util.Objects;
+
 public class Fare {


값객체 💯

bactoria · 2024-04-11T12:54:19Z

src/test/java/nextstep/path/fare/DiscountPolicyTest.java

+
+    @DisplayName("나이에 따라 요금을 할인한다")
+    @ParameterizedTest
+    @CsvSource({"6, 450", "13, 720", "19, 1250"})


경계구간 테스트 💯

bactoria · 2024-04-11T12:54:55Z

src/test/java/nextstep/path/fare/FareTest.java

+
+    @DisplayName("요금 더하기")
+    @Test
+    void plus() {


도메인 모델에 대한 테스트도 잘 해주셨네요 😄

moonstal1506 · 2024-04-17T11:35:45Z

마지막까지 리뷰해주셔서 감사합니다🙏
준오님 덕분에 미션도 끝까지 할 수 있었고 배운점이 많았습니다!!
다시 한번 감사드립니다:)

moonstal1506 · 2024-04-17T11:54:24Z

src/main/java/nextstep/path/Path.java

    private List<Station> stations;
-    private int distance;
-    private int duration;
+    private List<PathSection> pathSections;

-    public Path(List<Station> stations, int distance, int duration) {
+    public Path(List<Station> stations, List<PathSection> pathSections) {
        this.stations = stations;
-        this.distance = distance;
-        this.duration = duration;
+        this.pathSections = pathSections;
    }


stations 대신 sections(구간) 추가에 대한 피드백을 받고

생성자로 pathSections만 받아서 응답값(PathResponse)에 필요한getStations()를 구현하는 방법과

SubwayMap의 findPath에서 path.getVertexList()를 하면 stations를 바로 얻을 수 있다는 점에서 stations와 pathSections를 받는 방법이 고민되었습니다!

public Path findPath(Station source, Station target, PathType pathType) { GraphPath<Station, PathSection> path = getGraphPath(source, target, pathType); return new Path(path.getVertexList(), path.getEdgeList()); }

1번은 stations 구하는 로직을 한번 더 작성해야 한다는 점,
2번은 pathSection로도 구현 가능한데 stations를 또 받는다는 점에서
어떤 방식을 고르면 좋을까요🤔

moonstal1506 added 13 commits April 1, 2024 22:19

docs: 2단계 피드백 내용 추가

9706b1b

refactor: 출발지/도착지 파라미터로 전달 및 PathFinder SubwayMap로 역할 이동

135c69a

refactor: Fare 원시값 포장 객체로 변경

813d9bb

docs: 요금 정책 추가 요구사항 작성

6c0b7e4

test: 노선별 추가 요금 인수테스트 작성

aee66b7

test: 노선별 추가 요금 단위테스트 작성

9e84b32

feat: 노선별 추가 요금 구현

1f25188

test: 로그인 사용자 연령별 요금 계산 인수테스트 작성

8f9d1d4

test: 연령별 요금 할인 단위테스트 작성

3795fda

feat: 연령별 요금 할인 로직 작성

8abc39d

feat: 토큰에 나이 추가

4835fd4

feat: 로그인 여부에 따라 LoginMember 생성 로직 분리

95311ce

feat: 로그인 사용자 연령별 요금 계산 로직 작성

ffb7dc5

bactoria requested changes Apr 11, 2024

View reviewed changes

moonstal1506 added 2 commits April 16, 2024 00:00

refactor: Line 생성자 하나로 변경

2ae14b7

refactor: Path에 Sections 추가

a6ed904

moonstal1506 commented Apr 17, 2024

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

🚀 3단계 - 요금 정책 추가 #488

🚀 3단계 - 요금 정책 추가 #488

moonstal1506 commented Apr 3, 2024

bactoria left a comment

bactoria Apr 7, 2024

bactoria Apr 11, 2024

bactoria Apr 11, 2024

bactoria Apr 11, 2024

bactoria Apr 11, 2024

moonstal1506 commented Apr 17, 2024 •

edited

Loading

moonstal1506 Apr 17, 2024 •

edited

Loading

🚀 3단계 - 요금 정책 추가 #488

Are you sure you want to change the base?

🚀 3단계 - 요금 정책 추가 #488

Conversation

moonstal1506 commented Apr 3, 2024

bactoria left a comment

Choose a reason for hiding this comment

bactoria Apr 7, 2024

Choose a reason for hiding this comment

bactoria Apr 11, 2024

Choose a reason for hiding this comment

bactoria Apr 11, 2024

Choose a reason for hiding this comment

bactoria Apr 11, 2024

Choose a reason for hiding this comment

bactoria Apr 11, 2024

Choose a reason for hiding this comment

moonstal1506 commented Apr 17, 2024 • edited Loading

moonstal1506 Apr 17, 2024 • edited Loading

Choose a reason for hiding this comment

moonstal1506 commented Apr 17, 2024 •

edited

Loading

moonstal1506 Apr 17, 2024 •

edited

Loading