Skip to content

refactor: 지도프래그먼트 기능 분리 #14

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

minjun011026
Copy link
Collaborator

Summary

  • 모든 기능 구현 및 UI 코드가 MapFragment 코드 하나에 전부 구현되어 있어 지나치게 결합도가 높고 가독성이 좋지 않음
  • 기존의 MapFragment를 MVVM과 안드로이드 권장 아키텍처를 참고하여 구현
  • 경로 생성기와 팝업뷰는 각각RouteParser, StorePopupView로 클래스를 분리
  • 경로 색상이 UI와 통일성이 없다고 판단하여 기존의 파란색에서 보라색으로 변경

Describe your changes

  • MapFragment : 최대한 View로서의 역할만 수행하도록 연산기능을 대부분 ViewModel로 이전
  • MapViewModel : 필터링, 팝업과 마커 구체화, 좌표연산과 같은 가공, 연산을 수행
  • RouteParser : 현재 위치로부터 술집까지의 경로 연산만을 수행
  • StorePopUpView : 팝업뷰를 생성해주는 클래스를 별도로 분리
  • 경로 색상을 7165E3로 변경
  • 경로 색상 비교

To reviewers

  • 아키텍처 설계 측면에 대해서 리뷰해주시면 감사하겠습니다.
  • Deprecated된 Geocode관련 코드는 API 33 이후부터 도입된 함수를 사용하는데 현재 프로젝트 api가 33보다 낮아 주석처리했습니다.

@minjun011026 minjun011026 changed the title refactor: 지도프래그먼트 MVVM 적용 및 기능 분리 refactor: 지도프래그먼트 기능 분리 Feb 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant