-
Notifications
You must be signed in to change notification settings - Fork 2
Closed
Description
상태 관리 도메인 Converter 분리 및 책임 분리
📌 Description
상태 관리(Status) 도메인에서의 DTO ↔ Entity 변환 책임을 Converter 클래스로 분리하는 리팩토링 작업 수행
기존 문제점 진단:
- 기존
StatusServiceImpl클래스 내부에toDto(),toEntity()변환 메서드가 직접 구현된 수동 방식으로 존재 - 해당 로직은 단순 변환이 아닌 책임 단위를 가지므로, Service 내부에 존재하는 것은 단일 책임 원칙을 위배하게 됨
주요 문제:
| 항목 | 문제 내용 |
|---|---|
| 🎯 책임 분리 부족 | Service가 비즈니스 로직 외에 변환 책임까지 가짐 |
| 🔁 재사용성 부족 | 다른 계층에서 DTO 변환 재사용 어려움 |
| 🧪 테스트 불편 | DTO 변환 로직만 별도로 테스트 불가 |
✅ Check List
- 기존
toDto(),toEntity()책임 분리 완료 -
StatusServiceImpl내 변환 로직 제거 - 단일 책임 원칙(SRP) 만족
- 재사용성 및 테스트 가능성 확보
Metadata
Metadata
Assignees
Labels
No labels