Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FINDY-10] refactor: bookmark 정책 변경 #11

Merged
merged 11 commits into from
Oct 12, 2024

Conversation

Hoya324
Copy link
Member

@Hoya324 Hoya324 commented Oct 12, 2024

✅ 작업 내용

  • Bookmark의 장소 카테고리 종속성 제거: Bookmark와 장소의 종속성을 제거하여, 카테고리 변경에 영향을 받지 않도록 개선했습니다.
  • Bookmark 새로운 타입 정의: Bookmark의 타입을 새롭게 정의하여 명확한 분류와 확장성을 확보했습니다.
  • BookmarkType에 따른 장소 저장 규칙 설정: BookmarkType에 따라 장소 저장 규칙을 설정하여, 타입별로 저장 로직이 달라지도록 구현했습니다.
  • Bookmark 정책 변경에 따른 코드 수정: Bookmark의 정책이 변경됨에 따라, 불필요한 코드와 종속성을 제거했습니다.
  • 에러 메시지 정의: Bookmark 정책 위반 시의 에러 메시지를 명확히 정의하여 예외 처리 시 가독성을 높였습니다.
  • Enum 대분류 코드 수정: Bookmark 대분류 Enum 코드를 개선하여 가독성과 유지보수성을 높였습니다.

🤔 고민 했던 부분

  • 카테고리 종속성 제거와 확장성 고려: 기존의 Bookmark 구조에서는 카테고리에 대한 종속성이 강했으나, 이를 제거하면서도 카테고리의 유연성을 유지할 방법을 고민했습니다. 이를 위해 BookmarkType을 새롭게 정의하여, 확장성 있는 구조로 전환했습니다.
  • BookmarkType에 따른 저장 로직 정의: 장소 저장 시 BookmarkType에 따라 다르게 처리될 수 있도록 저장 규칙을 설정하는 과정에서 정책과의 일관성을 유지하도록 신경 썼습니다.

🔊 도움이 필요한 부분

  • Bookmark 정책 적용의 적절성 검토: BookmarkType에 따른 저장 로직과 정책 변경에 대한 타당성 검토가 필요합니다. 각 정책이 다양한 상황에서 올바르게 적용될 수 있는지 의견 부탁드립니다.

@Hoya324 Hoya324 added 🎯리팩토링 기존의 기능을 디벨롭한 pr입니다. 🧪테스트 test를 추가한 pr입니다. labels Oct 12, 2024
@Hoya324 Hoya324 self-assigned this Oct 12, 2024
Copy link

Unit Test Results

22 tests   22 ✔️  3s ⏱️
10 suites    0 💤
10 files      0

Results for commit ba85123.

Copy link

Code Coverage

File Coverage [100%] 🍏
Bookmark.java 100% 🍏
BookmarkType.java 100% 🍏
RegisterPlaceService.java 100% 🍏
MajorCategory.java 100% 🍏
Total Project Coverage 98.01% 🍏

@Hoya324 Hoya324 merged commit 2d427af into develop Oct 12, 2024
2 checks passed
@Hoya324 Hoya324 deleted the refactor/Bookmark-정책-변경 branch October 12, 2024 09:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🎯리팩토링 기존의 기능을 디벨롭한 pr입니다. 🧪테스트 test를 추가한 pr입니다.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant