Skip to content

Conversation

@Hoya324
Copy link
Member

@Hoya324 Hoya324 commented Nov 20, 2024

✅ 작업 내용

  • 기존 카테고리 처리 방식으로는 네이버 지도의 카테고리 형태에 대해 적절한 처리를 하기 어려웠습니다.
  • 이를 해결하기 위해, 제공하는 대분류, 중분류가 아닌경우 '>'로 구분된 형태로 저장하도록 변경했습니다.

🤔 고민 했던 부분

  • "관리 중인 enum 타입이 아닌데 괜찮을까?" 를 고민했으나 모든 카테고리를 확인할 수 없고, 이를 예외처리하거나 기타로 처리하는 것보단, 카테고리의 역할이 view의 역할 뿐이므로, 확장성이 큰 String으로 저장하는 것으로 결정했습니다.
  • 다만 중분류>소분류 로 구부되는 카테고리에 대해서는 대분류 > 중분류로 바꿔주는 로직은 유지합니다.

@Hoya324 Hoya324 added the 🎯리팩토링 기존의 기능을 디벨롭한 pr입니다. label Nov 20, 2024
@Hoya324 Hoya324 self-assigned this Nov 20, 2024
@github-actions
Copy link

github-actions bot commented Nov 20, 2024

Unit Test Results

79 tests   79 ✔️  12s ⏱️
27 suites    0 💤
27 files      0

Results for commit 6ddce35.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

Code Coverage

File Coverage [99.38%] 🍏
Category.java 100% 🍏
FindPlaceService.java 100% 🍏
MiddleCategory.java 100% 🍏
BatchRegisterMarkerService.java 98.85% 🍏
CategoryResolver.java 96.47% 🍏
RegisterMarkerService.java 92.94% 🍏
Total Project Coverage 87.01% 🍏

@Hoya324 Hoya324 merged commit cb25c6d into develop Nov 20, 2024
2 checks passed
@Hoya324 Hoya324 deleted the refactor/카테고리-다각화 branch November 20, 2024 06:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🎯리팩토링 기존의 기능을 디벨롭한 pr입니다.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants