Skip to content

Conversation

@zyovn
Copy link
Member

@zyovn zyovn commented Nov 9, 2025

[7팀] 치이이즈 백엔드 코드리뷰

✨ 리뷰를 요청드리는 주요 부분

가장 고민되는 지점을 아래 형식으로 정리했습니다.

파일경로 : 조언이 필요한 포인트

  • src/main/java/com/cheeeese/album/application/AlbumService.java : 현재 AlbumService 내에서 앨범 관련 비즈니스 로직이 한 클래스에 모여 있어서 구조적으로 좀 복잡하다는 생각이 들었습니다. 혹시 현업에서는 이런 경우 비즈니스 로직을 어떤 기준으로 분리하거나 클래스 단위로 쪼개는지 궁금합니다.
  • src/main/java/com/cheeeese/photo/application/PhotoQueryService.java : PhotoQueryService에서 최대 64명의 사용자가 최대 2000장의 사진을 조회할 수 있는 상황이라 Redis 캐싱을 활용해 조회 기능을 구현한 상태입니다. 이런 방식이 구조적으로 적절한지 그리고 Redis 메모리 사용 측면에서 문제 소지가 없는지 한번 점검받고 싶습니다.

💬 7팀이 고민하고 있는 부분

  • 7일이 지나면 앨범이 만료되면서 photo 테이블에 있는 4장의 사진을 새로운 cheese4cut이라는 테이블에 저장을 해야하는 구조인데, 현재로는 앨범이 생성되면서 redis에 TTL을 넣고 백그라운드 작업을 1~5초(예정)으로 돌려서 만료를 감지할 예정인데 이 구조에 대해서 어떻게 생각하시는 지 궁금합니다. 기획과 이야기를 해봤을 때 거의 실시간으로 만료를 감지하고 싶다는 요구가 있었습니다.

💌 요청드리는 리뷰 방향

  • 설계 방향의 확장성
  • 과도하거나 부족한 추상화 여부
  • 성능 최적화 여지가 있는 부분
  • 네이밍, 폴더 구조, 모듈 분리 방향성
  • 코드 라인 단위 피드백뿐만 아니라, 구조/설계 관점에서의 개선 제안을 주시면 많은 도움이 될 것 같습니다.

바쁘신 와중에 시간 내어 리뷰해주셔서 감사합니다.
편하게 피드백 부탁드립니다! 🥹

zyovn and others added 30 commits November 3, 2025 12:41
fix: photo status 관련 로직 수정
feat: UT를 위한 로직 임시 수정
dahyun24 and others added 30 commits November 27, 2025 00:32
Added detailed sections for API documentation, ERD, system architecture, technology stack, CI/CD, and monitoring.
feat: 서비스 운영 핵심 지표 수집을 위한 비즈니스 로그 추가
fix: 사진 1시간 다운로드 여부 오류
docs: Expand README with API and architecture details
test: 사진 조회 관련 테스트 코드 작성
docs: 앨범 내 사진 목록 조회 swagger 수정
feat: 사진 생성 제한 및 최대 사진 수 수정
[CEEZ-33] 앨범 생성 API DTO 수정
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.

3 participants