🔥 fix : 상세조회시, 거리 표현 수정

[doup2001]

📌 작업한 내용

• 상세 조회 API에서 거리(distance) 값의 계산 혹은 표기 로직을 리팩터링하여 보다 정확하고 일관된 형태로 표시되도록 수정했습니다.
• 클라이언트(웹/앱)에서 상세 조회 시, 사용자에게 노출되는 거리 표현 포맷(단위, 반올림/절사 규칙 등)을 정리하고 코드에 반영했습니다.
  ​

🔍 참고 사항

• 기존 거리 계산/표현 방식에 의존하는 다른 API나 화면이 있는지 한번 더 확인이 필요합니다.
  ​- 거리 포맷(예: m/㎞ 전환 기준, 소수점 자리수 등)에 대한 기획/디자인 합의가 있다면, 이번 PR 내용이 해당 정책을 정확히 따르는지 리뷰 시 함께 봐주세요.

🖼️ 스크린샷

🔗 관련 이슈

✅ 체크리스트

• 로컬에서 빌드 및 테스트 완료
• 코드 리뷰 반영 완료
• 문서화 필요 여부 확인

Summary by CodeRabbit

릴리스 노트

• 새로운 기능

  • 주택단지 조회 시 통합 교통 정보 제공 추가 - 총 소요 시간, 거리, 상세 경로 정보를 함께 제공합니다.

• 개선 사항

  • 교통 정보 조회 방식이 개선되었습니다. 더 명확하고 완전한 교통 데이터를 제공합니다.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

개요

대중교통 경로 정보의 데이터 구조를 재설계했습니다. 기존의 세그먼트 목록 기반 방식에서 총 시간, 거리 및 세그먼트를 포함하는 통합 TransitInfoResponse로 변경하고, ComplexService에 새로운 getTransitInfo() 메서드를 추가하며 getSegments()를 deprecate했습니다.

변경사항

집합 / 파일(들)	변경 요약
응답 DTO 구조 변경 ComplexDetailResponse.java	distance 필드를 List<SegmentResponse>에서 TransitInfoResponse로 변경; 팩토리 메서드 시그니처 및 null 체크 로직 업데이트
새로운 Transit 정보 레코드 TransitInfoResponse.java	새로운 공개 레코드 추가: totalTime(문자열), totalTimeMinutes(숫자), totalDistance(km), segments(세그먼트 리스트); @Builder, @JsonInclude, @Schema 어노테이션 포함
서비스 계층 리팩토링 ComplexService.java	새로운 공개 메서드 getTransitInfo(String id, String pinPointId) 추가; 기존 getSegments() 메서드를 @Deprecated로 표시; ComplexDetailResponse 생성 로직을 TransitInfo 기반으로 변경
매퍼 유틸리티 확장 TransitResponseMapper.java	새로운 공개 메서드 toTransitInfoResponse(RootResult route) 추가: null 입력 처리, 총 시간/거리 계산, 세그먼트 목록 변환 후 TransitInfoResponse 반환

예상 코드 리뷰 노력

🎯 4 (복잡함) | ⏱️ ~45분

시

🐰 세그먼트들 묶어서 통합했네,
시간과 거리 한눈에 보네!
전환 과정도 우아하게,
새로운 TransitInfo 환영해~
여행 정보, 더 깔끔해졌어! 🚌✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	PR 제목이 변경 사항의 핵심을 명확히 반영하고 있습니다. '거리 표현 수정'은 거리 계산 및 표기 로직의 리팩터링이라는 주요 변경 내용과 일치합니다.
Description check	✅ Passed	PR 설명이 템플릿의 주요 섹션(작업 내용, 참고 사항, 체크리스트)을 포함하고 있으며, 변경 사항과 검토 시 고려해야 할 사항을 충분히 설명하고 있습니다.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix/대중교통-거리-시간-수정

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[github-actions]

Test Results

22 tests  ±0   22 ✅ ±0   1s ⏱️ -1s
 3 suites ±0    0 💤 ±0 
 3 files   ±0    0 ❌ ±0 

Results for commit 4dfc6b2. ± Comparison against base commit 08e3a27.

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (3)

src/main/java/com/pinHouse/server/platform/housing/complex/application/util/TransitResponseMapper.java (1)

241-263: LGTM! 새로운 TransitInfoResponse 생성 메서드가 잘 구현되었습니다.

null 체크와 기존 toSegmentResponses 재사용이 적절합니다. 다만, 거리 계산 로직(Math.round(route.totalDistance() / 100.0) / 10.0)이 toSummaryResponse (line 219)와 중복됩니다.

🔎 중복 제거를 위한 헬퍼 메서드 추출 제안

+    /** 거리(m)를 km로 변환 (소수점 1자리) */
+    private double toDistanceKm(double distanceM) {
+        return Math.round(distanceM / 100.0) / 10.0;
+    }
+
     private TransitRoutesResponse.SummaryResponse toSummaryResponse(RootResult route) {
         int totalMinutes = route.totalTime();
         int transferCount = countTransfers(route);

         return TransitRoutesResponse.SummaryResponse.builder()
                 .totalMinutes(totalMinutes)
-                .totalDistanceKm(Math.round(route.totalDistance() / 100.0) / 10.0)
+                .totalDistanceKm(toDistanceKm(route.totalDistance()))
                 // ...

src/main/java/com/pinHouse/server/platform/housing/complex/application/dto/response/ComplexDetailResponse.java (1)

60-64: isEmpty() 사용은 좋은 개선입니다.

equals("") 대신 isEmpty()를 사용하는 것이 더 명확하고 관용적입니다. 다만, NumberFormatException 방어 로직이 없습니다. totalHouseholds가 숫자가 아닌 문자열일 경우 예외가 발생할 수 있습니다.

🔎 NumberFormatException 방어 로직 제안

 .totalHouseholds(
-        document.getTotalHouseholds() == null || document.getTotalHouseholds().isEmpty()
-                ? 0
-                : Integer.parseInt(document.getTotalHouseholds())
+        parseTotalHouseholds(document.getTotalHouseholds())
 )

// 별도 헬퍼 메서드 추가
+private static Integer parseTotalHouseholds(String value) {
+    if (value == null || value.isEmpty()) {
+        return 0;
+    }
+    try {
+        return Integer.parseInt(value);
+    } catch (NumberFormatException e) {
+        return 0;
+    }
+}

src/main/java/com/pinHouse/server/platform/housing/complex/application/service/ComplexService.java (1)

478-486: @Deprecated 어노테이션에 추가 정보를 포함하세요.

주석으로 대체 메서드를 언급했지만, 표준 Javadoc @deprecated 태그를 사용하면 IDE와 문서 도구에서 더 잘 인식됩니다.

🔎 Deprecation 어노테이션 개선 제안

-    /// Segment 리스트 조회 (임대주택 상세조회용) - Deprecated, use getTransitInfo instead
-    @Deprecated
+    /**
+     * Segment 리스트 조회 (임대주택 상세조회용)
+     * @deprecated Use {@link #getTransitInfo(String, String)} instead.
+     */
+    @Deprecated(forRemoval = true)
     @Transactional(readOnly = true)
     public List<TransitRoutesResponse.SegmentResponse> getSegments(String id, String pinPointId) throws UnsupportedEncodingException {

📜 Review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 08e3a27 and 4dfc6b2.

📒 Files selected for processing (4)

• src/main/java/com/pinHouse/server/platform/housing/complex/application/dto/response/ComplexDetailResponse.java
• src/main/java/com/pinHouse/server/platform/housing/complex/application/dto/response/TransitInfoResponse.java
• src/main/java/com/pinHouse/server/platform/housing/complex/application/service/ComplexService.java
• src/main/java/com/pinHouse/server/platform/housing/complex/application/util/TransitResponseMapper.java

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: test

🔇 Additional comments (4)

src/main/java/com/pinHouse/server/platform/housing/complex/application/dto/response/ComplexDetailResponse.java (1)

48-49: 필드 타입 변경이 API 응답 구조에 영향을 줍니다.

distance 필드 타입이 List<SegmentResponse>에서 TransitInfoResponse로 변경되었습니다. 이는 API 응답 구조의 breaking change입니다. 클라이언트에서 기존 배열 형태의 distance를 사용하고 있었다면 수정이 필요합니다.

또한, 필드명 distance가 이제 TransitInfoResponse 타입을 담고 있어 의미가 모호해졌습니다. 명확성을 위해 transitInfo로 변경을 고려해 보세요.

src/main/java/com/pinHouse/server/platform/housing/complex/application/dto/response/TransitInfoResponse.java (1)

1-29: LGTM! 새로운 DTO가 잘 설계되었습니다.

TransitInfoResponse record가 적절한 어노테이션(@Builder, @JsonInclude, @Schema)과 함께 깔끔하게 정의되었습니다. 기존 TransitRoutesResponse.SegmentResponse를 재사용하여 일관성을 유지한 점도 좋습니다.

src/main/java/com/pinHouse/server/platform/housing/complex/application/service/ComplexService.java (2)

72-76: LGTM! getTransitInfo로의 전환이 잘 적용되었습니다.

getSegments 대신 getTransitInfo를 호출하고 ComplexDetailResponse.from에 transitInfo를 전달하는 변경이 적절합니다.

---

469-476: getTransitInfo 메서드가 잘 구현되었습니다.

템플릿 메서드 패턴을 활용한 구현이 깔끔합니다. 다만, 두 가지 확인이 필요합니다:

1. getEasyDistance는 Redis 캐싱을 사용하지만 getTransitInfo는 캐싱이 없습니다. 상세 조회 API에서 빈번하게 호출된다면 캐싱 추가를 고려해 보세요.
2. 이 메서드가 외부에서도 사용된다면 ComplexUseCase 인터페이스에 추가하는 것이 좋습니다.