🚀 3단계 - 지하철 구간 관리 #968

nice7677 · 2023-07-23T13:05:07Z

안녕하세요.
JPA를 안 쓴지 좀 돼서 많이 까먹어서 매핑 부분에서 코드가 좀 더럽습니다.

hiblue02

3단계 미션도 잘 구현해주셨네요.
Line과 Section, Station, Stations간의 관계에 대해 다시 한번 고민해주시면 좋을 것 같아요. 😅
코멘트 남겼으니 확인해주시고, 다시 리뷰 요청주세요!

hiblue02 · 2023-07-24T04:32:42Z

README.md

+ - [ ] 지하철 노선에 구간을 등록
+ - [ ] 새로운 구간의 상행역은 해당 노선에 등록되어있는 하행 종점역이어야 한다.
+ - [ ] 새로운 구간의 하행역은 해당 노선에 등록되어있는 역일 수 없다.
+ - [ ] 새로운 구간 등록시 위 조건에 부합하지 않는 경우 에러 처리한다.


구현한 사항은 체크 표시해주시면 좋을 것 같아요.

hiblue02 · 2023-07-24T04:35:33Z

src/main/java/subway/line/domain/Section.java

+    @ManyToOne
+    @JoinColumn(name = "station_id", nullable = false)
+    private Station station;
+


Station이 아니라 Stations가 들어가야 하지 않을까요? 🤔 Stations와 Section의 연결고리가 없어보여요.
그리고 line에서 section을 가지고 있어야 할까요? stations을 가지고 있어야 할까요? 요 부분에 대해 고민해봐주세요!

생각못했네요 그렇게하면 해결되는 문제였는데 돌고돌아서 만들었네요😅

hiblue02 · 2023-07-24T04:45:57Z

src/main/java/subway/line/dto/response/LineResponse.java

+    public static LineResponse of(Line route) {
+        return new LineResponse(route);
    }


정적 팩토리 메소드 사용 👍

hiblue02 · 2023-07-24T04:51:55Z

src/test/java/subway/line/SectionAcceptanceTest.java

+        Map<String, String> params = new HashMap<>();
+        params.put("upStationId", "1");
+        params.put("downStationId", "2");
+        params.put("distance", "10");
+
+        Long lineId = 1L;
+
+        ExtractableResponse<Response> createSectionResponse = RestAssuredUtils.create(new RestAssuredCondition(String.format(LINE_SECTION_API_URI, lineId), params));
+        assertThat(createSectionResponse.statusCode()).isEqualTo(500);
+
+
+        Map<String, String> stationParams = new HashMap<>();
+        stationParams.put("name", "을지로역");
+        RestAssuredUtils.create(new RestAssuredCondition(STATION_API_URI, stationParams));
+
+
+        params.put("upStationId", "2");
+        params.put("downStationId", "3");
+        params.put("distance", "10");
+        createSectionResponse = RestAssuredUtils.create(new RestAssuredCondition(String.format(LINE_SECTION_API_URI, lineId), params));
+        assertThat(createSectionResponse.statusCode()).isEqualTo(201);
+
+        ExtractableResponse<Response> deleteSectionResponse = RestAssuredUtils.delete(new RestAssuredCondition(String.format(LINE_SECTION_API_URI + "?stationId=%s", lineId, 3)));
+        assertThat(deleteSectionResponse.statusCode()).isEqualTo(204);
+
+        ExtractableResponse<Response> getSectionResponse = RestAssuredUtils.inquiry(new RestAssuredCondition(String.format(LINE_SECTION_API_URI, lineId)));
+
+        assertThat(getSectionResponse.statusCode()).isEqualTo(200);
+
+        List<SectionResponse> sectionResponses = getSectionResponse.body().jsonPath().getList(".", SectionResponse.class);
+
+        assertThat(sectionResponses.size()).isEqualTo(2);
+        assertThat(sectionResponses.get(0).getStationId()).isEqualTo(1L);
+        assertThat(sectionResponses.get(1).getStationId()).isEqualTo(2L);


인수 테스트 코드가 길고 테스트 중간중간 assertThat이 들어가 있어서 한눈에 안들어오는 것 같아요. 😅 given, when, then 주석으로 구분해주면 어떨까요? 테스트 시나리오도 같이 있으면 좋을 것 같아요.

hiblue02 · 2023-07-24T04:57:01Z

src/main/java/subway/line/service/LineService.java

+        return sectionService.getSections(line)
+                .stream()
+                .sorted(Comparator.comparing(Section::getId))
+                .map(section -> {
+                    return SectionResponse.of(section);
+                })
+                .collect(Collectors.toList());


id 순서로 sorting하는 이유가 있을까요? id를 순서 칼럼으로 사용해도 괜찮을까요? 🤔

hiblue02 · 2023-07-24T04:59:44Z

src/main/java/subway/line/service/LineService.java

+    private void validateSection(List<Section> sections, Section section) {
+
+    }


사용하지 않는 코드는 지워주세요.

hiblue02 · 2023-07-24T05:02:14Z

src/test/java/subway/line/SectionAcceptanceTest.java

+    @DisplayName("구간 제거 기능")
+    @Test
+    void deleteSection() {


구간 제거 예외 케이스에 대한 검증이 없어요. 😭

구간 관리 기능의 예외 케이스를 고려하기

-구간 등록과 제거 기능의 예외케이스들에 대한 시나리오를 정의

-인수 테스트를 작성하고 이를 만족시키는 기능을 구현

hiblue02 · 2023-07-24T05:03:03Z

src/test/java/subway/line/SectionAcceptanceTest.java

+    @DisplayName("구간 등록 기능")
+    @Test
+    void createSection() {


구간 등록 예외 케이스에 대한 검증이 없어요. 😭

구간 관리 기능의 예외 케이스를 고려하기

-구간 등록과 제거 기능의 예외케이스들에 대한 시나리오를 정의

-인수 테스트를 작성하고 이를 만족시키는 기능을 구현

nice7677 added 6 commits July 16, 2023 21:35

docs: step3 기능 요구사항 수정

381f5e8

refactor: 3단계 전 수정

67490c6

feat: section 추가

a39d069

feat: section 추가

9faf292

feat: section 등록 기능 추가

9d242ab

feat: section 제거 기능 추가

94546e8

hiblue02 requested changes Jul 24, 2023

View reviewed changes

nice7677 added 4 commits July 27, 2023 21:13

test: given when then 주석

dd4107c

docs: 문서 수정

00ace5c

test: 에러 처리 예외 테스트

31c2473

refactor: 안쓰는 코드 삭제

d780f95

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

🚀 3단계 - 지하철 구간 관리 #968

🚀 3단계 - 지하철 구간 관리 #968

nice7677 commented Jul 23, 2023

hiblue02 left a comment

hiblue02 Jul 24, 2023

hiblue02 Jul 24, 2023

nice7677 Jul 24, 2023

hiblue02 Jul 24, 2023

hiblue02 Jul 24, 2023

hiblue02 Jul 24, 2023

hiblue02 Jul 24, 2023

hiblue02 Jul 24, 2023

hiblue02 Jul 24, 2023

		private void validateSection(List<Section> sections, Section section) {

		}

🚀 3단계 - 지하철 구간 관리 #968

Are you sure you want to change the base?

🚀 3단계 - 지하철 구간 관리 #968

Conversation

nice7677 commented Jul 23, 2023

hiblue02 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment