refactor: QueryDSL Repository를 CommandRepository와 QueryRepository로 분리(#110) #111
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
chaiminwoo0223
force-pushed
the
refactor/110
branch
from
1e6337e to
0b6ce32
Compare
…110) * feat: MemberCommandRepository, MemberCommandRepositoryAdapter 구현 * feat: TripCommandRepository, TripCommandRepositoryAdapter 구현 * feat: StampCommandRepository, StampCommandRepositoryAdapter 구현 * feat: MissionCommandRepository, MissionCommandRepositoryAdapter 구현 * feat: DailyMissionCommandRepository, DailyMissionCommandRepositoryAdapter 구현 * feat: StudyLogCommandRepository, StudyLogCommandRepositoryAdapter 구현 * feat: StudyLogDailyMissionCommandRepository, StudyLogDailyMissionCommandRepositoryAdapter 구현 * feat: PomodoroCommandRepository, PomodoroCommandRepositoryAdapter 구현 * feat: TripReportQueryRepository, TripReportQueryRepositoryAdapter 구현 * feat: StampQueryRepository, StampQueryRepositoryAdapter에 findNextStampOrderByTripId 메서드 추가 * feat: StampQueryService에 getNextStampOrderByTrip 메서드 추가 * feat: StampQueryService에 getStampsToShiftAfterDeleted 메서드 추가 * feat: StampCommandService에 shiftStampOrders 메서드 추가 * refactor: TripQueryRepositoryAdapter에서 Q 클래스 static import 적용으로 QueryDSL 가독성 향상 * refactor: StampQueryRepositoryAdapter에서 Q 클래스 static import 적용으로 QueryDSL 가독성 향상 * refactor: MissionQueryRepositoryAdapter에서 Q 클래스 static import 적용으로 QueryDSL 가독성 향상 * refactor: DailyMissionQueryRepositoryAdapter에서 Q 클래스 static import 적용으로 QueryDSL 가독성 향상 * refactor: StudyLogQueryRepositoryAdapter에서 Q 클래스 static import 적용으로 QueryDSL 가독성 향상 * refactor: StudyLogDailyMissionQueryRepositoryAdapter에서 Q 클래스 static import 적용으로 QueryDSL 가독성 향상 * refactor: PomodoroQueryRepositoryAdapter에서 Q 클래스 static import 적용으로 QueryDSL 가독성 향상 * refactor: DailyGoalQueryRepositoryAdapter에서 Q 클래스 static import 적용으로 QueryDSL 가독성 향상 * refactor: TripReportQueryRepositoryAdapter에서 Q 클래스 static import 적용으로 QueryDSL 가독성 향상 * refactor: TripReportStudyLogQueryRepositoryAdapter에서 Q 클래스 static import 적용으로 QueryDSL 가독성 향상 * refactor: DailyGoalQueryRepository -> DailyGoalCommandRepository 이름 변경 * refactor: DailyGoalQueryRepositoryAdapter -> DailyGoalCommandRepositoryAdapter 이름 변경 * refactor: TripReportQueryRepository -> TripReportCommandRepository 이름 변경 * refactor: TripReportQueryRepositoryAdapter -> TripReportCommandRepositoryAdapter 이름 변경 * refactor: TripReportStudyLogQueryRepository -> TripReportStudyLogCommandRepository 이름 변경 * refactor: TripReportStudyLogQueryRepositoryAdapter -> TripReportStudyLogCommandRepositoryAdapter 이름 변경 * refactor: StampQueryRepository, StampQueryRepositoryAdapter에서 findMaxStampOrderByTripId 메서드 삭제 * refactor: StampCommandService에서 computeNextStampOrder 메서드 삭제 * refactor: StampCommandService에서 createStamp, createStamps 메서드 int nextOrder 매개변수 추가 * refactor: StampCommandService에서 createStamp, createStamps 메서드 비즈니스 로직 개선 * refactor: StampCommandService에서 deleteStamp 메서드 비즈니스 로직 개선 * refactor: TripFacade에서 createTrip 메서드 비즈니스 로직 개선 * refactor: StampFacade에서 createStamp 메서드 비즈니스 로직 개선 * refactor: StampFacade에서 deleteStamp 메서드 비즈니스 로직 개선 * test: MemberCommandServiceTest 단위 테스트 수정 * test: TripCommandServiceTest 단위 테스트 수정 * test: StampCommandServiceTest 단위 테스트 수정 * test: MissionCommandServiceTest 단위 테스트 수정 * test: DailyMissionCommandServiceTest 단위 테스트 수정 * test: StudyLogCommandServiceTest 단위 테스트 수정 * test: StudyLogDailyMissionCommandServiceTest 단위 테스트 수정 * test: PomodoroCommandServiceTest 단위 테스트 수정 * test: DailyGoalCommandServiceTest 단위 테스트 수정 * test: TripReportCommandServiceTest 단위 테스트 수정 * test: TripReportStudyLogCommandServiceTest 단위 테스트 수정 * test: StampQueryServiceTest에 GetNextStampOrderByTrip 단위 테스트 추가 * test: StampQueryServiceTest에 GetStampsToShiftAfterDeleted 단위 테스트 추가 * test: StampCommandServiceTest에 ShiftStampOrders 단위 테스트 추가 * test: StampFixture에 createStampWithName 메서드 추가
chaiminwoo0223
force-pushed
the
refactor/110
branch
from
0b6ce32 to
8489d65
Compare
hisonghy
approved these changes
Nov 4, 2025
Contributor
hisonghy
left a comment
hisonghy
left a comment
There was a problem hiding this comment.
QueryDSL 레포지토리까지 분리하면서 더 코드가 깔끔해졌네요!
고샹하셨습니다 병합부탁드려요~
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
📌 작업 내용 및 특이사항
✅ CommandRepository 구현 및 명령(등록, 검증, 삭제) 로직 분리
MemberCommandRepository,MemberCommandRepositoryAdapter구현TripCommandRepository,TripCommandRepositoryAdapter구현StampCommandRepository,StampCommandRepositoryAdapter구현MissionCommandRepository,MissionCommandRepositoryAdapter구현DailyMissionCommandRepository,DailyMissionCommandRepositoryAdapter구현StudyLogCommandRepository,StudyLogCommandRepositoryAdapter구현StudyLogDailyMissionCommandRepository,StudyLogDailyMissionCommandRepositoryAdapter구현PomodoroCommandRepository,PomodoroCommandRepositoryAdapter구현TripReportQueryRepository,TripReportQueryRepositoryAdapter구현✅ 스탬프 생성 관련 로직 개선
StampQueryService.getNextStampOrderByTrip()에서 받은 nextOrder를 이용해 스탬프 생성 로직 개선StampQueryRepository,StampQueryRepositoryAdapter에서findMaxStampOrderByTripId메서드 삭제StampCommandService.computeNextStampOrder()삭제✅ 스탬프 삭제 관련 로직 개선
StampCommandService.deleteStamp()에서 스탬프 순서 변경 로직 분리StampCommandService.deleteStamp()는 소프트 삭제만 담당✅ QueryRepository 메서드 추가
StampQueryRepository,StampQueryRepositoryAdapter에findNextStampOrderByTripId메서드 추가✅ QueryService 메서드 추가
StampQueryService에getNextStampOrderByTrip메서드 추가StampQueryService에getStampsToShiftAfterDeleted메서드 추가StampCommandService에shiftStampOrders메서드 추가✅ 테스트 추가
StampQueryServiceTest에GetNextStampOrderByTrip단위 테스트 추가StampQueryServiceTest에GetStampsToShiftAfterDeleted단위 테스트 추가StampCommandServiceTest에ShiftStampOrders단위 테스트 추가StampFixture에createStampWithName메서드 추가✅ 테스트 수정
MemberCommandServiceTest단위 테스트 수정TripCommandServiceTest단위 테스트 수정StampCommandServiceTest단위 테스트 수정MissionCommandServiceTest단위 테스트 수정DailyMissionCommandServiceTest단위 테스트 수정StudyLogCommandServiceTest단위 테스트 수정StudyLogDailyMissionCommandServiceTest단위 테스트 수정PomodoroCommandServiceTest단위 테스트 수정DailyGoalCommandServiceTest단위 테스트 수정TripReportCommandServiceTest단위 테스트 수정TripReportStudyLogCommandServiceTest단위 테스트 수정🌱 관련 이슈
🔍 참고사항(선택)
QueryRepositoryAdapter에서 Q 클래스 static import 적용으로 QueryDSL 가독성 향상DailyGoalQueryRepository->DailyGoalCommandRepository이름 변경TripReportQueryRepository->TripReportCommandRepository이름 변경TripReportStudyLogQueryRepository->TripReportStudyLogCommandRepository이름 변경📚 기타(선택)