🚀 Step1 - 경로 조회 타입 추가 #338

hugehoo · 2023-04-04T07:03:24Z

안녕하세요 리뷰어님
시간이 많이 늦었지만, 늦게라도 완주하고자 PR 올립니다.
과정이 끝났음에도 리뷰 해주셔서 정말 감사합니다!

사전 실습 이미지: PathDocumentation을 문서화했습니다.

- PathFinder 인터페이스로 추상화 - Duration 기준 탐색 PathFinder 생성 - SubwayMap -> DistancePathFinder 수정

heesunjeong

안녕하세요 :) 마지막 미션 리뷰를 맡게된 정희선입니다! 마지막까지 잘 부탁드려요~!
문서와 실습과 1단계 미션 구현 잘 해주셨습니다! 👍
몇가지 조금 더 참고해보시면 좋을 부분을 코멘트로 추가했는데요, 확인부탁드려요!

heesunjeong · 2023-04-04T13:32:37Z

src/main/java/nextstep/subway/domain/path/DistancePathFinder.java

 private List<Line> lines;

- public SubwayMap(List<Line> lines) {
+ public DistancePathFinder(List<Line> lines) {


DistancePathFinder와 DurationPathFinder의 로직이 Section의 어떤 값을 사용하는지만 다르고 거의 동일한데요,
중복을 제거하는 리팩토링을 도전해보시면 어떨까요? 한가지 방법으로는 enum을 조금 더 활용해보셔도 좋을 것 같습니다 :)

heesunjeong · 2023-04-04T13:38:50Z

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

+ int 최소시간 = Math.min(경로_1_최소시간, 경로_2_최소시간);
+ assertThat(duration).isEqualTo(최소시간);


테스트 코드에서는 로직을 사용하는 것보다 하드코딩된 값으로 검증해보시면 어떨까요?
https://jojoldu.tistory.com/615

heesunjeong · 2023-04-04T13:39:37Z

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

+ .when().get("/paths?source={sourceId}&target={targetId}&searchType={type}", source, target, type)
+ .then().log().all().extract();
+ }
+
 private ExtractableResponse<Response> 두_역의_최단_거리_경로_조회를_요청(Long source, Long target) {


PathSteps을 활용하여 중복을 줄여볼 수 있을 것 같습니다~!

heesunjeong · 2023-04-04T13:48:30Z

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


+@DisplayName("Subway Path TEST 문서화")
 public class PathDocumentation extends Documentation {


더 풍부한 문서를 위해서 스니펫을 활용해서 문서를 커스텀해보시면 어떨까요?!

heesunjeong · 2023-04-04T13:49:13Z

src/main/java/nextstep/subway/domain/path/DistancePathFinder.java

+import nextstep.subway.domain.SectionEdge;
+import nextstep.subway.domain.Station;
+
+public class DistancePathFinder implements PathFinder {


새로 작성해주신 코드에 대해서 단위테스트를 작성해보시면 어떨까요?!

heesunjeong · 2023-04-04T14:11:32Z

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

+ }
+
+ /**
+ * Circular 경로는 생성 불가.


작성해주셨던 인수조건을 삭제하신 이유가 있으실까요? 😢

hugehoo added 9 commits March 27, 2023 11:50

feature: PathDocumentation 완성

ec3b539

이미지 추가

9907799

fix: duration 인자 추가

b8fec66

feature: Domain Path 패키지 생성

0e0524a

- PathFinder 인터페이스로 추상화 - Duration 기준 탐색 PathFinder 생성 - SubwayMap -> DistancePathFinder 수정

feature: 경로조회 비즈니스 로직 수정

1b8e993

test: 경로조회타입 추가 인수테스트

6cebf91

fix: Code 정리

45cebca

feature: 인수테스트 문서화

0567c1a

code 정리

f18af95

heesunjeong requested changes Apr 4, 2023

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

🚀 Step1 - 경로 조회 타입 추가 #338

🚀 Step1 - 경로 조회 타입 추가 #338

hugehoo commented Apr 4, 2023

heesunjeong left a comment

heesunjeong Apr 4, 2023

heesunjeong Apr 4, 2023

heesunjeong Apr 4, 2023

heesunjeong Apr 4, 2023

heesunjeong Apr 4, 2023

heesunjeong Apr 4, 2023

		int 최소시간 = Math.min(경로_1_최소시간, 경로_2_최소시간);
		assertThat(duration).isEqualTo(최소시간);


		@DisplayName("Subway Path TEST 문서화")
		public class PathDocumentation extends Documentation {

🚀 Step1 - 경로 조회 타입 추가 #338

Are you sure you want to change the base?

🚀 Step1 - 경로 조회 타입 추가 #338

Conversation

hugehoo commented Apr 4, 2023

heesunjeong left a comment

Choose a reason for hiding this comment

heesunjeong Apr 4, 2023

Choose a reason for hiding this comment

heesunjeong Apr 4, 2023

Choose a reason for hiding this comment

heesunjeong Apr 4, 2023

Choose a reason for hiding this comment

heesunjeong Apr 4, 2023

Choose a reason for hiding this comment

heesunjeong Apr 4, 2023

Choose a reason for hiding this comment

heesunjeong Apr 4, 2023

Choose a reason for hiding this comment