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

get- path-detail 과 post- path -favorites API를 구현, 리팩토링했습니다. #18

Closed
wants to merge 1 commit into from

Conversation

Moon-1C
Copy link
Collaborator

@Moon-1C Moon-1C commented Jun 1, 2024

//todo

nativeQuery = true)
List<TrafficEntity> findClosetTrafficByLocation(
@Param("longitude") Double longitude, @Param("latitude") Double latitude);

@Query(
value =
"SELECT * FROM traffic "
Copy link

Choose a reason for hiding this comment

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

주어진 코드 패치에 대한 간략한 코드 리뷰:

  1. 주변 교통 데이터 조회 쿼리 (findClosetTrafficByLocation) 사용시 주의사항:

    • ST_DISTANCE 함수를 통해 거리를 계산할 때 정확성과 성능을 위해 인덱스를 확인할 것.
    • 입력값(:longitude, :latitude) 유효성 검사 필요 (null 체크 등)
    • Polygon 생성 부분의 좌표 계산이 정확한지 확인
    • Subquery를 사용하기 때문에 큰 데이터셋에서는 성능 이슈가 발생할 수 있음
  2. SQL 쿼리와 지역적 기능 관련 테스트를 추가하여 작동 여부 확인이 필요.

  3. @Query 어노테이션 명확하게 사용 및 설명 주석 추가 권장.

  4. 오타나 문법적 오류가 있는지 세심히 확인하고 수정.

  5. 잠재적인 보안 문제 시 주변 코드와 함께 검토.

  6. Repository 메소드 네이밍 명확성을 높일 수 있도록 고민.

  7. 코드의 가독성을 높일 수 있도록 줄 바꾸기 등 포맷팅 확인.

  8. 코드 리뷰 이후 동료나 팀 내 다른 멤버들과 리뷰 결과 공유.

코드 최적화 및 버그 리스크를 줄이는 방향으로 진행하는 것이 중요합니다.

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.

2 participants