[Fix/116] 이미지 관련 오류 해결

[ddhi7]

1. #⃣ 연관된 이슈
   • 관련 이슈를 명시해주세요.
   • 예: #이슈번호, #이슈번호
2. 📝 작업 내용
   • 이번 PR에서 작업한 내용을 간략히 설명해주세요.
   • 필요한 경우 이미지 첨부 가능.
3. 📸 스크린샷 (선택)
   • 작업 내용을 시각적으로 표현할 스크린샷을 포함하세요.
4. 💬 리뷰 요구사항 (선택)
   • 리뷰어가 특히 검토해주었으면 하는 부분이 있다면 작성해주세요.
   • 예: "메서드 XXX의 이름을 더 명확히 하고 싶은데, 좋은 아이디어가 있으신가요?"

Summary by CodeRabbit

• Refactor
  • Optimized board image synchronization logic for improved efficiency.
  • Enhanced photo album image retrieval with batched processing to improve performance.

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

[coderabbitai]

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

Two Java service classes are optimized: BoardService refactors image synchronization to use Set-based keyName diffing instead of URL comparison and inline deletion, while PhotoAlbumServiceImpl switches from per-album to batched image fetching with a new helper method for computing first-image maps.

Changes

Cohort / File(s)	Summary
Image Synchronization Optimization src/main/java/cc/backend/board/service/BoardService.java	Replaces List with Set for newKeyNames and existingKeyNames to improve membership checks; removes intermediate URL computation and refactors deletion to inline forEach; updates addition filter to use keyName-based diffing with Set and switches result collection to toList().
Batched Image Fetching src/main/java/cc/backend/photoAlbum/service/PhotoAlbumServiceImpl.java	Adds new private helper method getFirstImageMapForPhotoAlbums() to batch fetch representative images and resolve presigned URLs; simplifies getAllRecentPhotoAlbumList DTO construction to directly use first-image map; updates nextPage calculation to inline derive from albumPage.hasNext().

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• PR #123: Related to PhotoAlbumServiceImpl changes that modify per-album image fetching to a batched flow using a helper method to build albumId→presigned URL maps.
• PR #121: Related to PhotoAlbumServiceImpl pagination refactoring that transitions from cursor-based to page-number pagination and nextPage handling.
• PR #101: Related to presigned-URL issuance and image service/DTO changes that support keyName-to-URL mapping via imageService.

Poem

🐰 With Sets we dance, no Lists to compare,
Batch first-images through the digital air,
KeyNames diffed swift, presigned URLs gleam,
Optimized flows in each micro-scheme! ✨

✨ 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/116

---

📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 9ff9454 and 959e17f.

📒 Files selected for processing (2)

• src/main/java/cc/backend/board/service/BoardService.java
• src/main/java/cc/backend/photoAlbum/service/PhotoAlbumServiceImpl.java

---

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.}