[SCRUM-259] 송정동 행정구역 분류 로직 수정 (#113)#115
Merged
moonxxpower merged 6 commits intodevelopfrom Aug 29, 2025
Hidden character warning
The head ref may contain hidden characters: "SCRUM-259-\uc1a1\uc815\ub3d9-\ud589\uc815\uad6c\uc5ed-\ubd84\ub958-\ub85c\uc9c1-\uc218\uc815"
Merged
Conversation
Area.WESTERN_NAKDONGRIVER으로 분류되어야할 "강서구 송정동"이 Area.SONGJEONG_GIJANG로 분류되는 오류를 수정하기 위해 extractArea 메서드의 조건문에 구 단위 체크("해운대구")를 추가했습니다.
카카오 지도 API에서 주소 정보를 받아오지 못하는 등의 오류가 발생했을 때, 서비스를 중단하지 않고 Area, Theme를 ETC로 분류하여 처리하도록 개선했습니다.
ssggii
reviewed
Aug 27, 2025
Contributor
ssggii
left a comment
ssggii
left a comment
There was a problem hiding this comment.
수고하셨습니다! 코멘트로 제안드린 부분 확인 부탁드립니다:)
두루누비 코스 생성 로직에서 Area, Theme 할당하는 부분도 수정이 필요해보이는데, 수현님이 현재 피드백 바탕으로 코드 수정해주시면 제가 이어서 수정 커밋 올리겠습니다!
src/main/java/com/server/running_handai/domain/course/service/CourseDataService.java
Outdated
Show resolved
Hide resolved
src/main/java/com/server/running_handai/domain/course/service/CourseDataService.java
Outdated
Show resolved
Hide resolved
src/main/java/com/server/running_handai/domain/course/service/CourseDataService.java
Outdated
Show resolved
Hide resolved
src/main/java/com/server/running_handai/domain/course/service/CourseDataService.java
Outdated
Show resolved
Hide resolved
src/main/java/com/server/running_handai/domain/course/service/CourseDataService.java
Outdated
Show resolved
Hide resolved
#113) 기존에 Map<String, String>으로 구성했던 주소 정보를 타입 안정성, 확장성, 불변성, 가독성 등의 이유로 Record로 변경했습니다. 추가적으로 KakaoMapService가 주소 관련 모든 기능을 담당하도록 기존에 CourseDataService에 있던 extractDistrictNameAndDongName를 kakaoMapService로 이동했습니다. 이 과정에서 textToNull 헬퍼 메서드를 추가해 가독성을 높였습니다.
Area, Theme의 결정 로직을 기존 CourseDataService에서 Enum으로 이동하여 코드 재사용성을 향상시켰습니다.
…me 메서드로 통일 (#113) 두루누비 코스의 Area와 Theme 결정 방식을 공통 메서드(extractArea, extractTheme)를 사용하도록 변경했습니다. 지역 및 테마 처리 경로를 통일하여 코드의 일관성을 높이고, 향후 로직 확장이 용이하도록 개선했습니다.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
✏️ 연관 이슈
#113
⛳ 작업 내용
Area,ThemeETC로 분류되게 수정💬리뷰 요구사항
extractArea에서 조건문에 구 단위 체크만 수정했는데, 혹시 제가 놓친 부분이 있다면 말씀해주세요!📍 참고사항
Area,ThemeEnum을ETC로 분류하면서SpotCategory도UNKNOWN대신ETC로 수정했습니다.