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

M3-173 축소시 픽셀 안보이게 수정 #25

Merged
merged 3 commits into from
Jul 11, 2024

Conversation

qjvk2880
Copy link
Contributor

작업 내용*

  • 일정 수준 이상 축소 시 픽셀이 안보이게끔 수정
  • 지도의 범위를 동적으로 계산하여 픽셀을 가져오는 반경을 조정
  • 지도 화면에 일부 랜드마크를 표시하여 편의성 개선

고민한 내용*

  • 픽셀이 안보이게 되는 zoom level은 14.3으로 설정 (사진 참고)
  • 지도의 범위를 동적으로 계산하기 위해 지도의 왼쪽 위 , 오른쪽 아래 점 사이 거리를 구함
  • 이 때 haversine 공식을 사용했다.

리뷰 요구사항

  • map controller에 점점 많은 함수가 생기는 것 같은데 다음 스프린트 때 리팩토링 해도 괜찮을 것 같습니다..!

스크린샷

스크린샷 2024-07-11 오후 12 28 22

@qjvk2880 qjvk2880 self-assigned this Jul 11, 2024
Copy link
Contributor

@tkdals802 tkdals802 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

맵이 더 이뻐지고
거리를 구하는 공식이 좋은것 같습니다

Copy link
Member

@koomin1227 koomin1227 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

줌 상태에 따라 가져오는 픽셀 수가 달라서 더 효율적으로 동작할 것 같네요. 말씀하신것 같이 mapController 의 역할을 분리 해야되긴할 것 같네요

double _calculateDistance(LatLng start, LatLng end) {
const double earthRadius = 6371000;

final double dLat = _toRadians(end.latitude - start.latitude);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

dLat 은 double Latitude 의 줄임말인가요?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

differential Latitude 라는 의미로 사용했는데 의미가 명확하지 않네요.. 수정하겠습니다..!

@qjvk2880 qjvk2880 merged commit 256a2d5 into develop Jul 11, 2024
1 check passed
@qjvk2880 qjvk2880 deleted the feature/M3-173-hidePixelsOnOverZoomOut branch July 12, 2024 01:25
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.

3 participants