[Spring 지하철 노선도 관리 - 3단계] 다니(이다은) 미션 제출합니다.

build.gradle

@@ -16,6 +16,7 @@ dependencies {
 	// spring
 	implementation 'org.springframework.boot:spring-boot-starter-web'
 	implementation 'org.springframework.boot:spring-boot-starter-jdbc'
+	implementation 'org.springframework.boot:spring-boot-starter-validation'
 
 	// handlebars
 	implementation 'pl.allegro.tech.boot:handlebars-spring-boot-starter:0.3.0'

README.md → docs/README.md

@@ -79,7 +79,49 @@ npm run dev
 - [x] H2 log/console 확인 
 
 ### 스프링 JDBC H2 DB 테스트
-- [x] DAO 테스트 작성 
+- [x] DAO 테스트 작성
+<br>
+
+> 3 단계
+
+### 노선 추가 기능 수정 
+- [x] 3가지 정보 추가
+  - [x] 상행 종점 (upStationId)
+  - [x] 하행 종점 (downStationId)
+  - [x] 두 종점간의 거리 (distance)
+- [x] 노선 추가 시 상행-하행 구간 등록 
+
+### 구간 추가
+- [x] 3가지 정보 추가
+  - [x] 상행 종점 (upStationId)
+  - [x] 하행 종점 (downStationId)
+  - [x] 두 종점간의 거리 (distance)
+- [x] 하나의 노선에서 종점역 추가될 경우
+  - 새 구간 하행 - 기존 구간 상행 비교 --> 상행 종점 변경
+  - 새 구간 상행 - 기존 구간 하행 비교 --> 하행 종점 변경
+- [x] 하나의 노선에서 중간역 추가될 경우 재배치
+  - [x] Distance 변경
+  - 새 구간 상행 - 기존 구간 상행 비교 --> 중간 추가
+  - 새 구간 하행 - 기존 구간 하행 비교 --> 중간 추가
+- [x] 중간역 등록 시 기존 구간 길이보다 크거나 같으면 등록 불가 
+- [x] 해당 노선에 이미 등록된 구간 추가 불가
+- [x] 상행역과 하행역 둘 중 하나는 무조건 해당 노선에 포함
+
+### 구간 제거 
+- [x] URI 정보 `/lines/{line_id}/sections?stationId={station_id}`
+- [x] 종점이 제거될 경우 다음으로 오던 역이 종점
+- [x] 중간역이 제거될 경우 재배치
+  - [x] Distance 변경
+- [x] 구간이 하나인 노선에서 마지막 구간 제거 불가
+
+### 노선 조회 기능 수정
+- [x] 상행 -> 하행 역 목록 차례로 출력
+
+### 지하철 역 / 지하철 노선 / 지하철 구간 구조 설계
+|Station |Line |Section |
+|:------:|:---:|:------:|
+|        |     |lineId  |
+|        |     |조회 시 구간 정렬|
 
 <br>
 

docs/TODO.md

@@ -0,0 +1,2 @@
+- 테스트 코드 메소드 분리
+- AssertJ 메소드 활용

src/main/java/wooteco/subway/exception/SubwayException.java

@@ -6,8 +6,8 @@ public class SubwayException extends RuntimeException {
     private final HttpStatus httpStatus;
     private final String message;
 
-    public SubwayException(HttpStatus httpStatus, String message) {
-        this.httpStatus = httpStatus;
+    public SubwayException(String message) {
+        this.httpStatus = HttpStatus.BAD_REQUEST;
         this.message = message;
     }
 

src/main/java/wooteco/subway/exception/line/LineDuplicationException.java

@@ -0,0 +1,9 @@
+package wooteco.subway.exception.line;
+
+import wooteco.subway.exception.SubwayException;
+
+public class LineDuplicationException extends SubwayException {
+    public LineDuplicationException() {
+        super("이미 존재하는 노선색깔입니다.");
+    }
+}

src/main/java/wooteco/subway/exception/line/LineLengthException.java

@@ -0,0 +1,9 @@
+package wooteco.subway.exception.line;
+
+import wooteco.subway.exception.SubwayException;
+
+public class LineLengthException extends SubwayException {
+    public LineLengthException() {
+        super("2자 이상의 이름을 입력해주세요.");
+    }
+}

src/main/java/wooteco/subway/exception/line/LineNonexistenceException.java

@@ -0,0 +1,10 @@
+package wooteco.subway.exception.line;
+
+import wooteco.subway.exception.SubwayException;
+
+public class LineNonexistenceException extends SubwayException {
+
+    public LineNonexistenceException() {
+        super("존재하지 않는 노선입니다.");
+    }
+}

src/main/java/wooteco/subway/exception/line/LineSuffixException.java

@@ -0,0 +1,9 @@
+package wooteco.subway.exception.line;
+
+import wooteco.subway.exception.SubwayException;
+
+public class LineSuffixException extends SubwayException {
+    public LineSuffixException() {
+        super("-선으로 끝나는 이름을 입력해주세요.");
+    }
+}

src/main/java/wooteco/subway/exception/section/SectionAdditionException.java

@@ -0,0 +1,9 @@
+package wooteco.subway.exception.section;
+
+import wooteco.subway.exception.SubwayException;
+
+public class SectionAdditionException extends SubwayException {
+    public SectionAdditionException() {
+        super("추가할 수 없는 구간입니다.");
+    }
+}

src/main/java/wooteco/subway/exception/section/SectionDeletionException.java

@@ -0,0 +1,9 @@
+package wooteco.subway.exception.section;
+
+import wooteco.subway.exception.SubwayException;
+
+public class SectionDeletionException extends SubwayException {
+    public SectionDeletionException() {
+        super("삭제할 수 없는 구간입니다.");
+    }
+}

src/main/java/wooteco/subway/exception/section/SectionDuplicationException.java

@@ -0,0 +1,9 @@
+package wooteco.subway.exception.section;
+
+import wooteco.subway.exception.SubwayException;
+
+public class SectionDuplicationException extends SubwayException {
+    public SectionDuplicationException() {
+        super("상행역과 하행역은 중복되지 않게 입력해주세요.");
+    }
+}

src/main/java/wooteco/subway/exception/station/StationDuplicationException.java

@@ -0,0 +1,9 @@
+package wooteco.subway.exception.station;
+
+import wooteco.subway.exception.SubwayException;
+
+public class StationDuplicationException extends SubwayException {
+    public StationDuplicationException() {
+        super("이미 존재하는 역입니다.");
+    }
+}

src/main/java/wooteco/subway/exception/station/StationLengthException.java

@@ -0,0 +1,9 @@
+package wooteco.subway.exception.station;
+
+import wooteco.subway.exception.SubwayException;
+
+public class StationLengthException extends SubwayException {
+    public StationLengthException() {
+        super("2자 이상의 역 이름을 입력해주세요.");
+    }
+}

src/main/java/wooteco/subway/exception/station/StationNonexistenceException.java

@@ -0,0 +1,12 @@
+package wooteco.subway.exception.station;
+
+import wooteco.subway.exception.SubwayException;
+
+public class StationNonexistenceException extends SubwayException {
+
+    public StationNonexistenceException() {
+        super("존재하지 않는 역입니다.");
+    }
+}
+
+

src/main/java/wooteco/subway/exception/station/StationSuffixException.java

@@ -0,0 +1,9 @@
+package wooteco.subway.exception.station;
+
+import wooteco.subway.exception.SubwayException;
+
+public class StationSuffixException extends SubwayException {
+    public StationSuffixException() {
+        super("-역으로 끝나는 이름을 입력해주세요.");
+    }
+}

src/main/java/wooteco/subway/line/Line.java

@@ -1,6 +1,7 @@
 package wooteco.subway.line;
 
-import wooteco.subway.exception.LineSuffixException;
+import wooteco.subway.exception.line.LineLengthException;
+import wooteco.subway.exception.line.LineSuffixException;
 
 import java.util.Objects;
 
@@ -14,14 +15,14 @@ public class Line {
     public Line() {
     }
 
-    public Line(Long id, String name, String color) {
-        this.id = id;
-        this.name = name;
-        this.color = color;
+    public Line(String name, String color) {
+        this(null, name, color);
     }
 
-    public Line(String name, String color) {
+    public Line(Long id, String name, String color) {
         validateSuffix(name);
+        validateLength(name);
+        this.id = id;
         this.name = name;
         this.color = color;
     }
@@ -36,6 +37,12 @@ private boolean isNotEndsWithLine(String name) {
         return !name.endsWith(SUFFIX);
     }
 
+    private void validateLength(String name) {
+        if (name.length() < 2) {
+            throw new LineLengthException();
+        }
+    }
+
     public Long getId() {
         return id;
     }

src/main/java/wooteco/subway/line/LineController.java

@@ -2,7 +2,9 @@
 
 import org.springframework.http.ResponseEntity;
 import org.springframework.web.bind.annotation.*;
+import wooteco.subway.station.StationResponse;
 
+import javax.validation.Valid;
 import java.net.URI;
 import java.util.List;
 import java.util.stream.Collectors;
@@ -18,15 +20,15 @@ public LineController(LineService lineService) {
     }
 
     @PostMapping
-    public ResponseEntity<LineResponse> createLine(@RequestBody LineRequest lineRequest) {
+    public ResponseEntity<LineResponse> createLine(@Valid @RequestBody LineRequest lineRequest) {
         Line newLine = lineService.add(lineRequest);
         LineResponse lineResponse = new LineResponse(newLine);
         return ResponseEntity.created(URI.create("/lines/" + newLine.getId())).body(lineResponse);
     }
 
     @GetMapping
     public ResponseEntity<List<LineResponse>> showLines() {
-        List<Line> lines = lineService.lines();
+        List<Line> lines = lineService.findAll();
         List<LineResponse> lineResponses = lines.stream()
                 .map(LineResponse::new)
                 .collect(Collectors.toList());
@@ -35,13 +37,15 @@ public ResponseEntity<List<LineResponse>> showLines() {
 
     @GetMapping(value = "/{id}")
     public ResponseEntity<LineResponse> showLine(@PathVariable Long id) {
-        Line line = lineService.line(id);
-        LineResponse lineResponse = new LineResponse(line);
+        Line line = lineService.findById(id);
+        List<StationResponse> sortedStations = lineService.sortedStationsByLineId(id);
+        LineResponse lineResponse = new LineResponse(line, sortedStations);
         return ResponseEntity.ok().body(lineResponse);
     }
 
     @PutMapping(value = "/{id}")
-    public ResponseEntity<Object> updateLine(@PathVariable Long id, @RequestBody LineRequest lineRequest) {
+    public ResponseEntity<Void> updateLine(@PathVariable Long id,
+                                           @Valid @RequestBody LineRequest lineRequest) {
         lineService.update(id, lineRequest);
         return ResponseEntity.ok().build();
     }

src/main/java/wooteco/subway/line/LineDao.java

@@ -8,11 +8,13 @@ public interface LineDao {
 
     List<Line> findAll();
 
-    void update(Long id, String name, String color);
-
-    Line findById(Long id);
+    Optional<Line> findById(Long id);
 
     Optional<Line> findByName(String name);
 
+    Optional<Line> findByColor(String color);
+
+    void update(Long id, String name, String color);
+
     void delete(Long id);
 }