지도 범위 내 마커형 공개 다이어리 조회 API 구현

[dnzp75]

📌 PR 제목

지도 범위 내 마커형 다이어리 조회 API 구현

• 지도 범위 내 클러스터형 공개 다이어리 조회 API 구현 (줌레벨 기준 시/도, 시군구 클러스터) #31
  -> 이전 브랜치 병합하여 계속 작업 진행

✨ 작업 사항 흐름

• 클러스터 모드 OFF 상태에서 지도 영역 내 다이어리 목록 조회
• 썸네일, 제목, 좋아요 수, 위경도, 생성일 등 반환
• VisibilityType.PUBLIC 조건으로 필터링

🔍 변경 이유

✅ 체크리스트

• 코드가 정상적으로 동작하는지 확인
• 관련 테스트 코드 작성 및 통과 여부 확인
• 문서화(README 등) 필요 여부 확인 및 반영
• 리뷰어가 알아야 할 사항 추가 설명

📸 스크린샷 (선택)

📌 참고 사항

[sonarqubecloud]

Quality Gate passed

Issues
13 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
2.4% Duplication on New Code

See analysis details on SonarQube Cloud

[github-actions]

Claude의 전체 변경사항 및 관련 파일에 대한 리뷰:

개선된 사항:

• 다중 데이터베이스 구성 (MySQL과 PostgreSQL)을 위한 설정 클래스 추가
• 지도 기능을 위한 새로운 도메인 (map) 추가
• QueryDSL 설정 개선

주요 이슈:

1. 트랜잭션 관리 개선 필요

   • 제안: MapService에서 트랜잭션 범위를 확장하여 데이터 일관성 보장

   @Transactional
   public void increaseRegionDiaryCount(Double lat, Double lon) {
       // 현재 코드
   }

2. 예외 처리 강화

   • 제안: MapService에서 지역을 찾지 못했을 때 명시적인 예외 처리

   public void increaseRegionDiaryCount(Double lat, Double lon) {
       SidoAreas sido = sidoAreasRepository.findRegionByLatLon(lat, lon)
           .orElseThrow(() -> new NotFoundRegionException("시도 지역을 찾을 수 없습니다."));
       // 나머지 코드
   }

3. 테스트 코드 개선

   • 제안: MapServiceTest에서 더 다양한 시나리오 테스트 추가

   @Test
   @DisplayName("유효하지 않은 좌표로 다이어리 클러스터 조회 시 빈 리스트 반환")
   void getDiaryClusters_invalidCoordinates_returnsEmptyList() {
       // 테스트 코드 구현
   }

관련 파일에 대한 영향 분석:

• DiaryService와 MapService 간의 의존성 추가로 인해, 다이어리 생성 시 지역 정보 업데이트 로직이 추가됨. 이는 다이어리 관련 테스트 코드 수정 필요성을 야기할 수 있음.
• 새로운 PostgreSQL 관련 설정으로 인해, 애플리케이션의 데이터베이스 연결 설정 및 관련 프로퍼티 파일 수정이 필요할 수 있음.
• Map 도메인 추가로 인해, API 문서화(예: Swagger) 업데이트가 필요할 수 있음.

전반적인 의견:
코드베이스가 지도 기능 추가로 인해 복잡해졌지만, 전반적으로 잘 구조화되어 있습니다. 다만, 트랜잭션 관리와 예외 처리에 대한 추가적인 주의가 필요하며, 새로운 기능에 대한 더 광범위한 테스트 coverage가 권장됩니다.