-
Notifications
You must be signed in to change notification settings - Fork 0
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
[#31] [FRATURE] 응급실 길찾기 #45
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
고생하셨습니다 !! 👏
저희 팀 다들 정말 실력 뭐죠 너무 잘 하시는데요!!
그 메서드 명 몇 개만 추천드려 봤는데 그거 말고는 저는 따로 코멘트 남길 부분이 없는 거 같습니다
그나저나 응답값 진짜 장난 아니긴 하네요,, ㅋㅋㅋ
|
||
@RequiredArgsConstructor | ||
@Component | ||
public class EmergencyRoomDirections { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
별건 아닌데 Directions
라는 이름이 엔티티에 가깝다는 느낌이 들어서
혹시 Router
이런 이름은 어떠신가용
@@ -7,4 +7,6 @@ public interface EmergencyRoomClient { | |||
List<RealTimeEmergencyRoomInfo> getRealTimeInfo(Region region); | |||
|
|||
List<EmergencyRoomInfo> getEmergencyRoomInfoData(); | |||
|
|||
EmergencyRoomDirectionsInfo directionsEmergencyRoom(String start, String end); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
getRouteToEmergencyRoom
이런거 어떨까요 아니면 find
?
@@ -41,4 +41,8 @@ public double distanceTo(Coordinate targetCoordinate) { | |||
|
|||
return Math.round(EARTH_RADIUS_KM * c * 100) / 100.0; | |||
} | |||
|
|||
public String convertorLatitudeAndLongitude() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
메서드명은 동사로 명명하는 게 좋을 거 같아서 convert
가 나을 것 같습니당
아니면 getGps
..? 음 애매하네요 ㅋㅋㅋ
감사합니다 ㅎㅎ |
https://api.ncloud-docs.com/docs/ai-naver-mapsdirections-driving#%EC%9D%91%EB%8B%B5 |
6bb3394
to
695d261
Compare
✨ 구현 기능 명세
✅ PR Point
😭 어려웠던 점