Conversation
- (10 20, 20 30) (40 30, 20 30) 형태에서 (10 20, 20 30) (20 30, 40 30) 형태로
- FE와의 협의로 길 상세정보에 갈림길 노드좌표 포함하는 형태로 변경 - 차량 내비게이션이 아니라 도보 상세정보 형태로 응답 변경
mikekks
left a comment
mikekks
left a comment
There was a problem hiding this comment.
고생하셨습니다 ~! 궁금한 부분 질문남겼습니다 !
| accumulatedDistance = calculateDistance(nowRoute)/2; | ||
| checkPointNodeCoordinates = getCenter(now, nxt); | ||
| checkPointType = DirectionType.CAUTION; |
There was a problem hiding this comment.
getCenter 메서드를 Route 내부로 넣어두는건 어떻게 생각하시는지 궁금합니다 !!
There was a problem hiding this comment.
오 그게 훨씬 더 좋은 방향인것 같습니다! 반영한 뒤 머지하겠습니다!!
There was a problem hiding this comment.
생각해보니 getCenter를 Route 내부에서 정의하게되면 메서드가
public Map<String,Double> getCenter(){
return Map.of("lat", (node1.getCoordinates().getY() + node2.getCoordinates().getY())/2
, "lng", (node1.getCoordinates().getX() + node2.getCoordinates().getX())/2);
}
이렇게 될텐데, 만약 Lazy loading 상태인(fetch join을 하지 않은) Route에서 getCenter를 호출할 경우 추가 쿼리문이 나갈 위험성이 있을것 같습니다.
게다가 현재 Route에서 node1, node2가 (fetch = LAZY) 로 선언되어있기 때문에 더더욱 추가 쿼리문의 가능성이 높아보입니다.
의견 남겨주시면 확인 후 머지하겠습니다!
There was a problem hiding this comment.
말씀주신 것에 동감합니다!
그래서 전 항상 기능 개발할때 "해당 기능에서 발생하는 쿼리도 의도대로 잘 동작하는지" 까지도 고려해서 기능개발하는 게 좋다고 생각하는데요! 어떻게 생각하시는지 궁금합니다!
There was a problem hiding this comment.
네 좋습니다! 의도대로 동작하는지 스스로도 점검하고, 앞으로 코드리뷰할때도 꼼꼼하게 보겠습니다!
2 participants
📝 PR 타입
🚀 변경 사항
🧪 테스트 결과