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

NaverMapFragment 라이브러리화 #27

Open
zion830 opened this issue Dec 17, 2021 · 0 comments
Open

NaverMapFragment 라이브러리화 #27

zion830 opened this issue Dec 17, 2021 · 0 comments
Labels
고통스러움 예상치 못한 문제가 많이 생길수 있고 작업량이 많을 것으로 보입니다 업보 청산 2년 전에 저지른 과오

Comments

@zion830
Copy link
Contributor

zion830 commented Dec 17, 2021

AS IS

현재 앱에서 지도가 들어가는 화면이 여러개인데, 화면 별로 지도에 붙어야하는 기능이 조금씩 다릅니다..
예를들어 카테고리별로 가게 보는 화면에서는 드래그 제스처가 끝날때마다 이벤트를 발생시키는 로직이 필요하고,
홈화면에서는 가게 리스트 처럼 별개의 View에서 발생한 이벤트에 따라 지도에 박힌 마커 색이 바뀝니다.

지금은 모든 화면에서 공통으로 쓰이는 기능들이 포함된 open class로 NaverMapFragment가 있고, 얘를 상속받아서 화면별로 필요에 따라 조금씩 커스텀한 XXXNaverMapFragment 클래스를 만들어서 쓰고 있습니다

  • 대략 이런 관계 - NaverMapFragment <-- NearStoreNaverMapFragment <-<> HomeFragment

문제는 저 XXXNaverMapFragment 시리즈들이 숫자가 늘어나다보니까 뭐가 어디서 쓰이는지 헷갈리기도 하고
NaverMapFragment 에서 버그가 생겼을 경우, 오류를 수정했을때 상속받고 있는 모든 UI에 영향이 가서 건들기가 조금 힘든 상태입니다

TO BE

  • 구현을 app 쪽에서 알수 없도록 지도 관련 코드를 다른 모듈로 분리
  • 마커/위치/권한 관련 옵션들을 다 포함한 라이브러리화 하면 좋을거 같습니다 (그러면 사장님 앱에서도 매우 용이할듯.)
@zion830 zion830 added 업보 청산 2년 전에 저지른 과오 고통스러움 예상치 못한 문제가 많이 생길수 있고 작업량이 많을 것으로 보입니다 labels Dec 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
고통스러움 예상치 못한 문제가 많이 생길수 있고 작업량이 많을 것으로 보입니다 업보 청산 2년 전에 저지른 과오
Projects
None yet
Development

No branches or pull requests

1 participant