[feat] 여행의 총 시간과 거리를 계산한다 (#289) #328

Jaeyoung22 · 2023-08-21T07:55:42Z

📌 관련 이슈

closed [feat] 여행의 총 시간과 총 이동 거리를 계산한다 #289

📁 작업 설명

여행의 총 시간을 계산합니다.

여행의 총 거리를 계산합니다.

하버-사인 공식이 아닌 구면 코사인 법칙을 이용해서 구했습니다.
일단 수식이 간단하여 쉬운 이해를 위하여 구면 코사인 법칙을 이용했습니다.
하버-사인 공식을 사용한 방법이 짧은 거리에서의 계산이 더 정확하고, 속도도 더 빠를 수 있으므로 추후 리팩터링 예정입니다.

github-actions · 2023-08-21T07:57:24Z

Test Results

199 tests 199 ✔️ 11s ⏱️
  43 suites     0 💤
  43 files     0 ❌

Results for commit 26000fc.

♻️ This comment has been updated with latest results.

greeng00se

방학인데 열일하시네요.. 👍 시간과 거리 계산 로직 뚝딱 만들어주셨네요 👍👍
방학 겸 시간도 조금 널널해서 리뷰 자세하게 남겨보았습니다. 😄
확인 부탁드립니다~ 🙇

6시 30분 추가 코멘트
계산 로직과 시간 구하는 로직이 비교적 간단하여 Route 클래스가 거리와 시간을 계산하는 책임을 가지게 하는 건 어떨까요?

backend/src/main/java/dev/tripdraw/domain/trip/RouteLength.java

greeng00se · 2023-08-21T08:41:00Z

backend/src/main/java/dev/tripdraw/domain/trip/RouteLength.java

+        double theta = startPoint.longitude() - endPoint.longitude();
+        Double latitude1 = startPoint.latitude();
+        Double latitude2 = endPoint.latitude();
+
+        double unitDistance = sin(toRadians(latitude1)) * sin(toRadians(latitude2))
+                + cos(toRadians(latitude1)) * cos(toRadians(latitude2)) * cos(toRadians(theta));


다음과 같이 메서드를 분리해보면 어떨까요?

private double calculateUnitDistance(Point startPoint, Point endPoint) { double start = startPoint.latitude(); double end = endPoint.latitude(); double theta = start - end; return sin(toRadians(start)) * sin(toRadians(end)) + cos(toRadians(start)) * cos(toRadians(end)) * cos(toRadians(theta)); }

추가로 double startRadians = toRadians(startPoint.latitude()); 와 같이 radians로 변수 추출 한다면 조금 더 최적화 할 수 있겠네요!

theta는 경도를 사용해서 구하기 때문에 제시해주신 방법은 오류가 있네요!

저도 변수명을 많이 고민했는데,
오히려 이런 부분은 수학적인 도메인이기 때문에 startLatitude, endLatitude 보다 1,2를 뒤에 붙이는 네이밍을 선택했습니다
이 부분은 어떻게 생각하실까요?

근거만 있다면 취사선택 하시면 될 것 같습니다!

저는 startLatitude, endLatitude 를 더 선호합니다만, 리오의 선택에 맡기겠습니다!

greeng00se · 2023-08-21T08:46:53Z

backend/src/main/java/dev/tripdraw/domain/trip/RouteLength.java

+    }
+
+    public String lengthInKm() {
+        return String.format("%.2f" + "km", length);


km도 view의 영역 아닐까 싶은데 어떻게 생각하시나요!

근데 이 부분 그러면 어떤 단위로 날아가는지는 api에 담는 것이 좋을까요 아니면 팀 내부 회의에서 정하는 것이 좋을까요?

단위는 변수명으로 정해도 되고, 팀 내부 회의에서 정해도 좋을 것 같습니다~

backend/src/test/java/dev/tripdraw/domain/trip/RouteLengthTest.java

backend/src/test/java/dev/tripdraw/domain/trip/RouteTest.java

backend/src/test/java/dev/tripdraw/domain/trip/TripDurationTest.java

greeng00se

관련 코멘트 재확인 부탁드립니다.
https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports

greeng00se · 2023-08-21T10:11:54Z

backend/src/main/java/dev/tripdraw/domain/trip/Trip.java

-import jakarta.persistence.ManyToOne;
-import java.time.Duration;
-import java.util.List;
+import jakarta.persistence.*;


사용하지 않는 것이 저희 컨벤션입니다. 확인부탁드립니다~

실례지만, 혹시 짜증내신 건가요?

Combi153

고생하셨습니다 혹시 관련한 공식에 대한 내용 자료를 어디서 볼 수 있을까요? 저도 궁금하네요!

Combi153 · 2023-09-01T04:41:37Z

backend/src/main/java/dev/tripdraw/application/TripService.java

+import java.util.List;
+
+import static dev.tripdraw.exception.member.MemberExceptionType.MEMBER_NOT_FOUND;
+import static dev.tripdraw.exception.trip.TripExceptionType.TRIP_NOT_FOUND;
+


제 눈에는 이미 위에 있는데, 왜 또 보이죠..? 그림자 분신술...

Combi153 · 2023-09-01T04:43:27Z

backend/src/main/java/dev/tripdraw/domain/trip/RouteLength.java

+    private static final double AVERAGE_DISTANCE_FOR_ONE_DEGREE_DIFFERENCE_IN_LATITUDE_ON_KOREA_REGION = 111.1;
+


영어 좀 치시네요 파파고 돌리셨나요?

Combi153 · 2023-09-01T04:48:22Z

backend/src/main/java/dev/tripdraw/domain/trip/RouteLength.java

+        double theta = startPoint.longitude() - endPoint.longitude();
+        Double latitude1 = startPoint.latitude();
+        Double latitude2 = endPoint.latitude();
+
+        double unitDistance = sin(toRadians(latitude1)) * sin(toRadians(latitude2))
+                + cos(toRadians(latitude1)) * cos(toRadians(latitude2)) * cos(toRadians(theta));


저는 startLatitude, endLatitude 를 더 선호합니다만, 리오의 선택에 맡기겠습니다!

Jaeyoung22 added 3 commits August 21, 2023 16:38

[feat] 여행 기간을 계산하는 기능 구현 (#289)

c19e472

[refactor] 여행 경로 조회 기능 수정 (#289)

45bd5c9

[fix] conflict 해결 (#289)

f8db27e

Jaeyoung22 requested review from greeng00se, beer-2000 and Combi153 August 21, 2023 07:55

Jaeyoung22 self-assigned this Aug 21, 2023

[refactor] 누락된 테스트 작성 (#289)

5d877a5

greeng00se requested changes Aug 21, 2023

View reviewed changes

Jaeyoung22 added 8 commits August 21, 2023 18:26

[refactor] 접근 제어자 변경 (#289)

56b6c5e

[refactor] primitive type으로 변경 (#289)

8e94e66

[refactor] 조건 오류 수정 (#289)

cb99c1a

[refactor] RouteLength 생성 로직 변경 (#289)

496ca5f

[refactor] duration 계산 로직 TripDuration 내부에 있도록 변경 (#289)

bda1c6e

[refactor] static import 정리 (#289)

b3f18dd

[refactor] 불필요한 개행 정리 (#289)

f51a50e

[refactor] 개행 추가 (#289)

26000fc

greeng00se requested changes Aug 21, 2023

View reviewed changes

Combi153 approved these changes Sep 1, 2023

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[feat] 여행의 총 시간과 거리를 계산한다 (#289) #328

[feat] 여행의 총 시간과 거리를 계산한다 (#289) #328

Jaeyoung22 commented Aug 21, 2023

github-actions bot commented Aug 21, 2023 •

edited

Loading

greeng00se left a comment •

edited

Loading

greeng00se Aug 21, 2023

greeng00se Aug 21, 2023

Jaeyoung22 Aug 21, 2023

greeng00se Aug 21, 2023

Combi153 Sep 1, 2023

greeng00se Aug 21, 2023

Jaeyoung22 Aug 21, 2023

greeng00se Aug 21, 2023

greeng00se left a comment

greeng00se Aug 21, 2023

Combi153 Sep 1, 2023 •

edited

Loading

Combi153 left a comment

Combi153 Sep 1, 2023

Combi153 Sep 1, 2023 •

edited

Loading

Combi153 Sep 1, 2023

		private static final double AVERAGE_DISTANCE_FOR_ONE_DEGREE_DIFFERENCE_IN_LATITUDE_ON_KOREA_REGION = 111.1;

[feat] 여행의 총 시간과 거리를 계산한다 (#289) #328

Are you sure you want to change the base?

[feat] 여행의 총 시간과 거리를 계산한다 (#289) #328

Conversation

Jaeyoung22 commented Aug 21, 2023

📌 관련 이슈

📁 작업 설명

github-actions bot commented Aug 21, 2023 • edited Loading

Test Results

greeng00se left a comment • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

greeng00se left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Combi153 Sep 1, 2023 • edited Loading

Choose a reason for hiding this comment

Combi153 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Combi153 Sep 1, 2023 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

github-actions bot commented Aug 21, 2023 •

edited

Loading

greeng00se left a comment •

edited

Loading

Combi153 Sep 1, 2023 •

edited

Loading

Combi153 Sep 1, 2023 •

edited

Loading