Merge pull request #50 from MAKAR-iOS/feat/#49

[Feat/#49] 최근 경로 리스트 내 존재하지 않는 경로를 삭제할 경우 예외 처리
MAKAR-iOS · Jul 30, 2024 · 847cf14 · 847cf14
2 parents 722049d + a168843
commit 847cf14
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/src/main/java/makar/dev/common/status/ErrorStatus.java b/src/main/java/makar/dev/common/status/ErrorStatus.java
@@ -52,6 +52,7 @@ public enum ErrorStatus {
     NOT_FOUND_LINE_STATION(HttpStatus.NOT_FOUND, 404, "지하철 노선도 역 리스트가 존재하지 않습니다."),
     NOT_FOUND_ROUTE(HttpStatus.NOT_FOUND, 404, "존재하지 않는 경로입니다."),
     NOT_FOUND_NOTI(HttpStatus.NOT_FOUND, 404, "존재하지 않는 알림입니다."),
+    NOT_FOUND_IN_RECENT_ROUTE_LIST(HttpStatus.NOT_FOUND, 404, "최근 경로 리스트 내 존재하지 않는 경로입니다."),
 
 
     /**

diff --git a/src/main/java/makar/dev/domain/User.java b/src/main/java/makar/dev/domain/User.java
@@ -6,6 +6,8 @@
 import lombok.Getter;
 import lombok.NoArgsConstructor;
 import lombok.Setter;
+import makar.dev.common.exception.GeneralException;
+import makar.dev.common.status.ErrorStatus;
 
 import java.util.List;
 
@@ -89,7 +91,10 @@ public void addRecentRouteList(Route route) {
         }
     }
     public void removeRecentRouteList(Route route) {
-        this.recentRouteList.remove(route);
+        boolean removed = this.recentRouteList.remove(route);
+        if (!removed) {
+            throw new GeneralException(ErrorStatus.NOT_FOUND_IN_RECENT_ROUTE_LIST);
+        }
     }
     public void clearRecentRouteList() {
         this.recentRouteList.clear();