🚀 1단계 - 경로 조회 타입 추가 #112

loop-study · 2022-02-17T14:05:17Z

안녕하세요! 리뷰어님
마지막 주차를 진행하게 된 박현철입니다. 🙇🏻‍♀️
이번 첫단계를 진행하면서 문서화 커스터마이징으로 작성했지만
gradle이 제대로 인식이 안되어서 원하는 모습으로 출력이 되지 않고 있습니다 😭

이를 해결할 수 있는 방법이 있다면 키워드 부탁드리겠습니다.

그리고 부족한 점이 많아 확인 부탁드리겠습니다.
감사합니다. 🙇🏻‍♀️

2022. 02. 17. README.md - 실습 사항 정리 Resolves : loop-study See also : junwoochoi

2022. 02. 17. build.gradle - 코드 수정 index.adoc - 코드 수정 Resolves : loop-study See also : junwoochoi

2022. 02. 17. PathDocumentation - 문서화 실습 PathSteps - 문서화 도우미 클래스 Resolves : loop-study See also : junwoochoi

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

2022. 02. 17. PathAcceptanceTest - 노선과 구간에 인자로 소요시간 추가 Resolves : loop-study See also : junwoochoi

2022. 02. 17. LineRequest, SectionRequest - 소요시간 추가 - 관련 테스트도 수정 Resolves : loop-study See also : junwoochoi

2022. 02. 17. PathAcceptanceTest - 테스트 명명 수정 PathDocumentation - 테스트 스텁빙에 소요시간 추가 PathResponse - 소요시간 추가 Resolves : loop-study See also : junwoochoi

2022. 02. 17. PathAcceptanceTest, PathDocumentation, PathSteps - 시간과 거리를 구분하는 인자 추가 Resolves : loop-study See also : junwoochoi

2022. 02. 17. 추가된 소요시간에 관련된 클래스 수정 Resolves : loop-study See also : junwoochoi

2022. 02. 17. PathController - 경로 조회 메서드에 시간과 유형 타입을 구분하는 인자 추가 SubwayMap - 유형 타입 구분으로 로직 변경 Path - 최단 시간 메서드 호출 추가 관련된 테스트들 수정 Resolves : loop-study See also : junwoochoi

2022. 02. 17. SubwayMapTest, PathDocumentation - 최단 시간 테스트 추가 Resolves : loop-study See also : junwoochoi

2022. 02. 17. PathAcceptanceTest, PathDocumentation, PathSteps - 유형에 관한 중복 코드 개선 Resolves : loop-study See also : junwoochoi

2022. 02. 17. PathSteps - 각 필드 설명 추가 Resolves : loop-study See also : junwoochoi

2022. 02. 17. PathAcceptanceTest - 구간을 좀 더 구분하기 위한 거리 및 시간 수정 Resolves : loop-study See also : junwoochoi

junwoochoi

안녕하세요! 원하는 대로 안나온다는 말씀이 혹시 아래 사진 같은 모습을 원하셨던 걸까요?

아마 asciidoc/index.adoc에 스니펫 추가를 안해주셔서 그런 것 같습니다.
https://docs.spring.io/spring-restdocs/docs/current/reference/html5/#getting-started-using-the-snippets
요문서를 확인하시면 조금 도움이 되실 것 같아요 :)

추가적으로 몇가지 코멘트 달아보았는데 확인 부탁드립니다! 🙏

README.md

src/docs/asciidoc/index.adoc

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

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

junwoochoi · 2022-02-18T02:14:40Z

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

    private List<Line> lines;
+    private String type;


Type을 enum을 활용해서 표현해보시면 어떨까요?
https://velog.io/@nawhew/%EC%9E%90%EB%B0%94-Enum%EC%9D%84-%EB%8D%94-%EC%9E%98%EC%93%B0%EA%B8%B0-%EC%9C%84%ED%95%9C-%EB%B0%A9%EB%B2%95

junwoochoi · 2022-02-18T02:16:17Z

src/main/java/nextstep/subway/ui/PathController.java

-        return ResponseEntity.ok(pathService.findPath(source, target));
+    public ResponseEntity<PathResponse> findPath(@RequestParam Long source,
+                                                 @RequestParam Long target,
+                                                 @RequestParam String type) {


type=BLABLA처럼 이상한 값이 들어와도 DISTANCE만 아니면 다 Duration기준으로 조회가 되고 있을 것 같네요 ㅠㅠ

유효성을 잊어버렸네요...🙈
빠르게 수정하겠습니다.

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

2022. 02. 17. PathSteps.java - 문서자동화 커스터마이징 추가 index.adoc, path.adoc - 커스터마이징 문서 추가 Resolves : loop-study See also : junwoochoi

2022. 02. 17. 여러 검증이 존재하면 assertAll로 묶음 처리 Resolves : loop-study See also : junwoochoi

2022. 02. 17. String PathType을 Enum으로 변경 - 관련된 로직 수정 - 관련된 테스트 변경 Resolves : loop-study See also : junwoochoi

loop-study · 2022-02-19T04:07:11Z

안녕하세요 리뷰어님!
피드백 반영하여 제출합니다.
커스터마이징도 원하는 대로 출력이 되었습니다.

확인 부탁드리겠습니다!
감사합니다!🙇🏻‍♀️

junwoochoi

안녕하세요! 1단계 미션 피드백도 잘 적용해주시고 문서화도 잘 적용해주셨습니다!
다만, 기존 구간이 삭제될 때 상행역 하행역의 소요시간을 합쳐주는 것 이외에도

신논현역 --- 7km, 10분 ----------- 양재역

에서 중간에 강남역이 추가되는 겨우, 시간을 쪼개주는 거리 뿐만 아니라 시간도 쪼개줘야하는 데 해당 부분 적용이 미흡한 것 같습니다 한번 확인 부탁드립니다 🙏

신논현역 --3km, 4분 --- 강남역 ----4km, 6분 ---- 양재역

src/docs/asciidoc/index.adoc

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

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

2022. 02. 17. Sections - 중간구역 추가에 다른 소요시간 누락 수정 LineTest - 중간구역 추가에 누락된 소요시간 검증 추가 Resolves : loop-study See also : junwoochoi

loop-study · 2022-02-19T09:20:52Z

안녕하세요 리뷰어님.
놓친 부분 알려주셔서 감사합니다.
중간 구역 단위테스트에 소요시간 검증도 추가하여 확인하도록 변경했습니다!
확인 부탁드립니다.
감사합니다.

junwoochoi

피드백 적용해주신 부분 확인했습니다!!!
스텝2도 화이팅입니다!😁🙌

loop-study added 14 commits February 17, 2022 17:53

docs : 실습 요구사항 정리

5df022c

2022. 02. 17. README.md - 실습 사항 정리 Resolves : loop-study See also : junwoochoi

refactor : 문서 관련 예제코드 수정

5377e96

2022. 02. 17. build.gradle - 코드 수정 index.adoc - 코드 수정 Resolves : loop-study See also : junwoochoi

feat : 문서화 실습

681f2c6

2022. 02. 17. PathDocumentation - 문서화 실습 PathSteps - 문서화 도우미 클래스 Resolves : loop-study See also : junwoochoi

docs : step1 정리

1fad3e4

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

test : 지하철 경로 인수테스트 수정

7e44f79

2022. 02. 17. PathAcceptanceTest - 노선과 구간에 인자로 소요시간 추가 Resolves : loop-study See also : junwoochoi

test : 지하철 경로 수정

a6e183e

2022. 02. 17. LineRequest, SectionRequest - 소요시간 추가 - 관련 테스트도 수정 Resolves : loop-study See also : junwoochoi

test : 지하철 경로 찾기 테스트 수정

2d62504

2022. 02. 17. PathAcceptanceTest - 테스트 명명 수정 PathDocumentation - 테스트 스텁빙에 소요시간 추가 PathResponse - 소요시간 추가 Resolves : loop-study See also : junwoochoi

test : 경로 찾기 테스트 호출 인자 추가

dd281e8

2022. 02. 17. PathAcceptanceTest, PathDocumentation, PathSteps - 시간과 거리를 구분하는 인자 추가 Resolves : loop-study See also : junwoochoi

test : 구간 관련 시간 인자 추가

1c47e9c

2022. 02. 17. 추가된 소요시간에 관련된 클래스 수정 Resolves : loop-study See also : junwoochoi

test : 테스트 추가

c0a2266

2022. 02. 17. SubwayMapTest, PathDocumentation - 최단 시간 테스트 추가 Resolves : loop-study See also : junwoochoi

refactor : 코드 개선

61d9083

2022. 02. 17. PathAcceptanceTest, PathDocumentation, PathSteps - 유형에 관한 중복 코드 개선 Resolves : loop-study See also : junwoochoi

refactor : 문서화 커스터마이징

7abb27c

2022. 02. 17. PathSteps - 각 필드 설명 추가 Resolves : loop-study See also : junwoochoi

refactor : 경로 인수테스트 수정

a4503e1

2022. 02. 17. PathAcceptanceTest - 구간을 좀 더 구분하기 위한 거리 및 시간 수정 Resolves : loop-study See also : junwoochoi

junwoochoi requested changes Feb 18, 2022

View reviewed changes

loop-study added 3 commits February 19, 2022 12:27

feat : 문서화 커스터마이징

817a8e8

2022. 02. 17. PathSteps.java - 문서자동화 커스터마이징 추가 index.adoc, path.adoc - 커스터마이징 문서 추가 Resolves : loop-study See also : junwoochoi

refactor : assertAll 추가

7fad4d7

2022. 02. 17. 여러 검증이 존재하면 assertAll로 묶음 처리 Resolves : loop-study See also : junwoochoi

refactor : PathType 열거형으로 변경

64e0c96

2022. 02. 17. String PathType을 Enum으로 변경 - 관련된 로직 수정 - 관련된 테스트 변경 Resolves : loop-study See also : junwoochoi

junwoochoi requested changes Feb 19, 2022

View reviewed changes

src/docs/asciidoc/index.adoc Show resolved Hide resolved

src/test/java/nextstep/subway/acceptance/LineAcceptanceTest.java Show resolved Hide resolved

src/main/java/nextstep/subway/domain/PathType.java Show resolved Hide resolved

feat : 중간구역 추가에 따른 누락된 부분 수정

9b8b373

2022. 02. 17. Sections - 중간구역 추가에 다른 소요시간 누락 수정 LineTest - 중간구역 추가에 누락된 소요시간 검증 추가 Resolves : loop-study See also : junwoochoi

junwoochoi approved these changes Feb 19, 2022

View reviewed changes

junwoochoi merged commit 581663c into next-step:loop-study Feb 19, 2022

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

🚀 1단계 - 경로 조회 타입 추가 #112

🚀 1단계 - 경로 조회 타입 추가 #112

loop-study commented Feb 17, 2022

junwoochoi left a comment

junwoochoi Feb 18, 2022

junwoochoi Feb 18, 2022

loop-study Feb 19, 2022

loop-study commented Feb 19, 2022 •

edited

junwoochoi left a comment

loop-study commented Feb 19, 2022

junwoochoi left a comment

🚀 1단계 - 경로 조회 타입 추가 #112

🚀 1단계 - 경로 조회 타입 추가 #112

Conversation

loop-study commented Feb 17, 2022

junwoochoi left a comment

Choose a reason for hiding this comment

junwoochoi Feb 18, 2022

Choose a reason for hiding this comment

junwoochoi Feb 18, 2022

Choose a reason for hiding this comment

loop-study Feb 19, 2022

Choose a reason for hiding this comment

loop-study commented Feb 19, 2022 • edited

junwoochoi left a comment

Choose a reason for hiding this comment

loop-study commented Feb 19, 2022

junwoochoi left a comment

Choose a reason for hiding this comment

loop-study commented Feb 19, 2022 •

edited