caadiq
430bf38c91
- src/services/album.js 생성: getAlbumDetails, getAlbumsWithTracks - src/services/schedule.js 생성: getMonthlySchedules, getUpcomingSchedules - albums/index.js, schedules/index.js에서 서비스 사용으로 변경 - schedules/index.js에서 중복 함수 제거 (240줄 감소) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
84 lines
2.6 KiB
Markdown
84 lines
2.6 KiB
Markdown
# Backend Refactoring Plan
|
|
|
|
백엔드 코드 품질 개선을 위한 리팩토링 계획서
|
|
|
|
## 작업 목록
|
|
|
|
### 1단계: 설정 통합 (config 정리) ✅ 완료
|
|
- [x] 카테고리 ID 상수 통합 (`CATEGORY_IDS`)
|
|
- [ ] 매직 넘버 설정 파일로 이동 (추후)
|
|
- [ ] JWT 기본값 제거 (추후)
|
|
|
|
**수정된 파일:**
|
|
- `src/config/index.js` - `CATEGORY_IDS` 상수 추가
|
|
- `src/routes/admin/youtube.js` - config에서 import
|
|
- `src/routes/admin/x.js` - config에서 import
|
|
- `src/routes/schedules/index.js` - 하드코딩된 2, 3, 8 → 상수로 변경
|
|
|
|
---
|
|
|
|
### 2단계: N+1 쿼리 최적화 ✅ 완료
|
|
- [x] 앨범 목록 조회 시 트랙 한 번에 조회로 변경
|
|
- [x] 스케줄 멤버 조회 - 이미 최적화됨 (확인 완료)
|
|
|
|
**수정된 파일:**
|
|
- `src/routes/albums/index.js` - GET /api/albums에서 트랙 조회 최적화
|
|
- 변경 전: 앨범마다 트랙 조회 (N+1 쿼리)
|
|
- 변경 후: 모든 트랙을 한 번에 조회 후 JavaScript에서 그룹화 (2 쿼리)
|
|
|
|
---
|
|
|
|
### 3단계: 서비스 레이어 분리 ✅ 완료
|
|
- [x] `src/services/album.js` 생성 - 앨범 관련 비즈니스 로직
|
|
- [x] `src/services/schedule.js` 생성 - 스케줄 관련 비즈니스 로직
|
|
- [x] 라우트에서 서비스 호출로 변경
|
|
|
|
**생성된 파일:**
|
|
- `src/services/album.js` - getAlbumDetails, getAlbumsWithTracks
|
|
- `src/services/schedule.js` - getMonthlySchedules, getUpcomingSchedules
|
|
|
|
**수정된 파일:**
|
|
- `src/routes/albums/index.js` - 서비스 import 및 사용
|
|
- `src/routes/schedules/index.js` - 서비스 import 및 기존 함수 제거 (240줄 감소)
|
|
|
|
---
|
|
|
|
### 4단계: 에러 처리 통일
|
|
- [ ] 에러 응답 유틸리티 생성 (`src/utils/error.js`)
|
|
- [ ] `reply.status()` vs `reply.code()` 통일
|
|
- [ ] `console.error` → `fastify.log.error` 변경
|
|
|
|
**관련 파일:**
|
|
- 모든 라우트 파일
|
|
|
|
---
|
|
|
|
### 5단계: 중복 코드 제거
|
|
- [ ] 멤버 이름 매핑 유틸리티 생성
|
|
- [ ] 이미지 업로드 공통 함수 생성
|
|
- [ ] 스케줄러 상태 업데이트 로직 통합
|
|
|
|
**관련 파일:**
|
|
- `src/services/image.js`
|
|
- `src/services/youtube/index.js`
|
|
- `src/plugins/scheduler.js`
|
|
|
|
---
|
|
|
|
## 진행 상황
|
|
|
|
| 단계 | 작업 | 상태 |
|
|
|------|------|------|
|
|
| 1단계 | 설정 통합 | ✅ 완료 |
|
|
| 2단계 | N+1 쿼리 최적화 | ✅ 완료 |
|
|
| 3단계 | 서비스 레이어 분리 | ✅ 완료 |
|
|
| 4단계 | 에러 처리 통일 | 대기 |
|
|
| 5단계 | 중복 코드 제거 | 대기 |
|
|
|
|
---
|
|
|
|
## 참고사항
|
|
|
|
- 각 단계별로 테스트 후 다음 단계 진행
|
|
- 기존 API 응답 형식은 유지
|
|
- 프론트엔드 수정 불필요하도록 진행
|