🚀 1단계 - 지하철 구간 추가 기능 개선 #606

jjisun · 2023-07-18T09:34:26Z

지하철 구간 추가 기능 개선 구현

기존 : 구간은 종점에서만 추가 가능
변경 : 구간은 앞, 뒤, 중간 모두 추가 가능

: 새로운 역을 상행 종점으로 등록
: 새로운 역을 하행 종점으로 등록
: 역 사이에 새로운 역을 등록

예외 조건

: 역 사이에 새로운 역을 등록할 경우, 거리는 기존의 구간보다 짧아야 함
: 상행역과 하행역이 이미 노선에 모두 등록되어 있다면 추가할 수 없음
: 상행역과 하행역 둘 중 하나도 포함되어있지 않으면 추가할 수 없음

리뷰 잘부탁드립니다 :)

* step1: 지하철 목록 조회, 제거 인수 테스트 작성 * 2단계 지하철 노선 관리

# Conflicts: # src/test/java/subway/acceptance/SectionAcceptanceTest.java

junwoochoi

안녕하세요! 1단계 미션 진행해주신 부분 확인했습니다!
0단계 실습을 놓치고 계신 것 같아서 요 부분 확인 부탁드립니다 🙏
추가로 2주차 실습은 인수테스트와 단위테스트를 함께 하면서 강의중에 다뤘던 Outside In, Inside Out이 각각 어떤 장단점이 있는지, 어떤 방식이 더 선호하는 방식에 맞는지 생각해보시면서 진행해보시면 더 좋답니다 😄
1단계 미션에 대한 코멘트도 남겼으니 해당 부분 + 0단계 실습 확인 후 다시 요청 부탁드립니다!
추가로 이해가 안되는 부분이나 의논해보고 싶은 부분이 있으면 편하게 dm이나 채널에 메시지 주세요!

junwoochoi · 2023-07-18T12:28:02Z

src/main/java/subway/common/DatabaseCleaner.java

+    @Transactional
+    public void execute() {
+        entityManager.flush();
+        entityManager.createNativeQuery("SET REFERENTIAL_INTEGRITY FALSE").executeUpdate();
+
+        for (String tableName : tableNames) {
+            entityManager.createNativeQuery("TRUNCATE TABLE " + tableName).executeUpdate();
+        }
+
+        entityManager.createNativeQuery("SET REFERENTIAL_INTEGRITY TRUE").executeUpdate();
+    }


인수테스트 격리 👍

junwoochoi · 2023-07-18T12:32:36Z

src/test/java/subway/acceptance/SectionAcceptanceTest.java

+import subway.fixture.StationFixture;
+
+@DisplayName("지하철 구간 관련 기능")
+@SpringBootTest(webEnvironment = SpringBootTest.WebEnvironment.DEFINED_PORT)


RANDOM_PORT를 활용해보면 더 좋을 것 같아요~

junwoochoi · 2023-07-18T12:34:32Z

src/test/java/subway/acceptance/SectionAcceptanceTest.java

+    @LocalServerPort
+    int port;
+
+    @Autowired
+    DatabaseCleaner databaseCleaner;
+
+    Long lineId;
+
+    Map<String, Long> stationMaps = new HashMap<>();
+
+    @BeforeEach
+    void setUp() {
+        RestAssured.port = port;
+        databaseCleaner.execute();


아래 부분들이 인수테스트 클래스마다 모두 중복이 발생하고 있는데 중복을 줄일 수 있는 방법이 없을까요?

@SpringBootTest(webEnvironment = SpringBootTest.WebEnvironment.RANDOM_PORT) @LocalServerPort int port; @Autowired DatabaseCleaner databaseCleaner; @BeforeEach의 RestAssured.port = port; databaseCleaner.execute();

junwoochoi · 2023-07-18T12:36:33Z

src/test/java/subway/acceptance/SectionAcceptanceTest.java

+        var lineResponse = response.as(LineResponse.class);
+        assertThat(lineResponse.getStations().stream().map(StationResponse::getName)).containsAnyOf("정자역");


인수 테스트는 블랙박스 테스트로 보고 진행하니, 직접적인 구현에 의존되는 as(LineResponse.class)보다는 jsonPath위주로 활용해보시길 권해드립니다!
물론 개인 선호에 따라 지금처럼 바로 클래스에 매핑해서 활용해주셔도 좋습니다 :)

junwoochoi · 2023-07-18T12:37:06Z

src/test/java/subway/acceptance/SectionAcceptanceTest.java

+
+        // then
+        var lineResponse = response.as(LineResponse.class);
+        assertThat(lineResponse.getStations().stream().map(StationResponse::getName)).containsAnyOf("논현역");


응답에 논현역이 있는지만 확인하기보다 순서도 체크해주시면 더 좋겠습니다!

junwoochoi · 2023-07-18T12:50:42Z

src/main/java/subway/entity/Sections.java

+            if(o.getUpStation().equals(section.getUpStation())) {
+                replaceSections.add(section);
+                var distance = o.getDistance() - section.getDistance();
+                replaceSections.add(Section.builder()
+                    .upStation(section.getDownStation())
+                    .downStation(o.getDownStation())
+                    .distance(distance)
+                    .build());
+            } else if(o.getDownStation().equals(section.getDownStation())) {
+                var distance = o.getDistance() - section.getDistance();
+                replaceSections.add(Section.builder()
+                    .upStation(o.getUpStation())
+                    .downStation(section.getUpStation())
+                    .distance(distance)
+                    .build());
+                replaceSections.add(section);
+            } else {
+                replaceSections.add(o);
+            }
+        });
+
+        return replaceSections;


요부분 가독성을 개선해볼 수 있을까요?
객체지향 생활체조 2번 else예약어를 사용하지 않는다도 참고하면 도움이 되겠네요!

junwoochoi · 2023-07-18T12:52:20Z

src/main/java/subway/entity/Sections.java

+    public Section lastSection() {
+        return sections.get(sections.size() - 1);
+    }
+
+    public Section firstSection() {
+        return sections.get(0);
+    }


첫번째와 마지막 Section은 index순으로 가져와서는 알맞은 정보를 가져오기 힘듭니다. 어떤 조건의 Station이 상행, 하행 종점으로 볼 수 있을까요?

junwoochoi · 2023-07-18T12:53:42Z

src/main/java/subway/entity/Sections.java

+    }
+
+    public List<Station> stations() {
+        return this.sections.stream().map(o -> Arrays.asList(o.getUpStation(), o.getDownStation()))


요기서도 위 이미지 참고하여 순서에 맞는 응답을 만들어볼 수 있습니다~!

junwoochoi · 2023-07-18T12:53:59Z

src/main/java/subway/exception/ApiException.java

+
+import org.springframework.http.HttpStatus;
+
+public class ApiException extends RuntimeException {


커스텀 예외 처리 👍

junwoochoi · 2023-07-18T12:54:22Z

src/main/java/subway/exception/ApiExceptionHandler.java

+
+    @ExceptionHandler(ApiException.class)
+    public ResponseEntity<ErrorResponse> handleSubwayException(ApiException apiException) {
+        return new ResponseEntity<>(ErrorResponse.builder()


boorownie and others added 15 commits January 17, 2022 03:01

3rd init

df2d0b1

4th init

c66bb09

5th init

37f79e0

6th init

3244b9d

step1: 구글 페이지 접근 테스트

b01ce62

step1: 구글 페이지 접근 테스트 (next-step#763)

d202435

🚀 2단계 - 지하철 노선 관리 (#899)

053adea

* step1: 지하철 목록 조회, 제거 인수 테스트 작성 * 2단계 지하철 노선 관리

test: 지하철 구간 추가, 삭제 인수테스트 작성, 기존 테스트 수정

ecccf73

feat: 지하철 구간 추가, 삭제 구현

02b8012

test: 지하철 구간 추가 기능 개선 인수테스트 작성

80cea42

refactor: 지하철 구간 관리 리팩토링 리뷰 반영

c350315

Merge branch 'step3' into tdd-step1

81dc0c6

# Conflicts: # src/test/java/subway/acceptance/SectionAcceptanceTest.java

feature: 지하철 구간 추가 기능 개선

293651e

Merge branch 'step4' into step1

7cc45cf

Merge branch 'jjisun' into step1

10f13fa

junwoochoi requested changes Jul 18, 2023

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

🚀 1단계 - 지하철 구간 추가 기능 개선 #606

🚀 1단계 - 지하철 구간 추가 기능 개선 #606

jjisun commented Jul 18, 2023

junwoochoi left a comment

junwoochoi Jul 18, 2023

junwoochoi Jul 18, 2023

junwoochoi Jul 18, 2023

junwoochoi Jul 18, 2023

junwoochoi Jul 18, 2023

junwoochoi Jul 18, 2023

junwoochoi Jul 18, 2023

junwoochoi Jul 18, 2023

junwoochoi Jul 18, 2023

junwoochoi Jul 18, 2023

		var lineResponse = response.as(LineResponse.class);
		assertThat(lineResponse.getStations().stream().map(StationResponse::getName)).containsAnyOf("정자역");


		import org.springframework.http.HttpStatus;

		public class ApiException extends RuntimeException {

🚀 1단계 - 지하철 구간 추가 기능 개선 #606

Are you sure you want to change the base?

🚀 1단계 - 지하철 구간 추가 기능 개선 #606

Conversation

jjisun commented Jul 18, 2023

junwoochoi 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

Choose a reason for hiding this comment