Skip to content

Commit 08d314f

Browse files
committed
refactor: routineType nullable하지 않도록 수정
1 parent ade58cd commit 08d314f

2 files changed

Lines changed: 14 additions & 49 deletions

File tree

src/main/java/bitnagil/bitnagil_backend/routine/request/DeleteRoutineByDayRequest.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,9 @@ public class DeleteRoutineByDayRequest {
2626
private UUID routineId;
2727

2828
@Schema(description = "루틴에 대한 타입 값입니다.",
29-
example = "CHANGED_ROUTINE")
29+
example = "CHANGED_ROUTINE",
30+
required = true)
31+
@NotNull
3032
private RoutineType routineType;
3133

3234
@Schema(description = "세부루틴 완료 여부 정보를 담은 리스트입니다.",

src/main/java/bitnagil/bitnagil_backend/routine/service/RoutineService.java

Lines changed: 11 additions & 48 deletions
Original file line numberDiff line numberDiff line change
@@ -162,65 +162,28 @@ public void deleteRoutine(User user, UUID routineId) {
162162
public void deleteRoutineByDay(User user, DeleteRoutineByDayRequest request) {
163163
LocalDateTime now = LocalDateTime.now();
164164

165-
if (request.getRoutineType() == null) {
166-
Routine routine = routineValidator.validateRoutineOwnership(request.getRoutineId(), user, now);
165+
if (request.getRoutineType() == RoutineType.ROUTINE) {
166+
Routine routine = routineValidator.validateRoutine(user, request.getRoutineId(), request.getHistorySeq());
167167

168168
// 변경 루틴으로 전환
169-
ChangedRoutine changedRoutineForDelete = ChangedRoutine.builder()
170-
.changedRoutinePk(new HistoryPk(UUID.randomUUID(), 1L))
171-
.changedRoutineName(routine.getName())
172-
.changedExecutionTime(routine.getExecutionTime())
173-
.originalRoutineDate(request.getPerformedDate())
174-
.changedRoutineDate(request.getPerformedDate())
175-
.historyStartDateTime(now)
176-
.historyEndDateTime(TimeUtils.END_DATE_TIME)
177-
.changedDivCode(ChangedDivCode.TODAY_DELETE)
178-
.userId(routine.getUserId())
179-
.routineId(routine.getRoutinePk().getId())
180-
.build();
181-
169+
ChangedRoutine changedRoutineForDelete = routineFactory.createChangedRoutineForDelete(request, routine, now);
182170
changedRoutineRepository.save(changedRoutineForDelete);
183171

184-
// 변경 서브루틴으로 전환
185172
List<SubRoutine> subRoutines = subRoutineRepository.findByRoutineId(routine.getRoutinePk().getId());
186173

174+
// 변경 서브루틴으로 전환
187175
for (SubRoutine subRoutine : subRoutines) {
188-
ChangedSubRoutine changedSubRoutineForDelete = ChangedSubRoutine.builder()
189-
.changedSubRoutinePk(new HistoryPk(UUID.randomUUID(), 1L))
190-
.changedSubRoutineName(subRoutine.getName())
191-
.historyStartDateTime(now)
192-
.historyEndDateTime(TimeUtils.END_DATE_TIME)
193-
.changedRoutineId(changedRoutineForDelete.getChangedRoutinePk().getId())
194-
.sortOrder(subRoutine.getSortOrder())
195-
.build();
196-
176+
ChangedSubRoutine changedSubRoutineForDelete =
177+
routineFactory.createChangedSubRoutineForDelete(subRoutine, now, changedRoutineForDelete);
197178
changedSubRoutineRepository.save(changedSubRoutineForDelete);
198179
}
199180
}
200-
else {
201-
if (request.getRoutineType() == RoutineType.ROUTINE) {
202-
Routine routine = routineValidator.validateRoutine(user, request.getRoutineId(), request.getHistorySeq());
203-
204-
// 변경 루틴으로 전환
205-
ChangedRoutine changedRoutineForDelete = routineFactory.createChangedRoutineForDelete(request, routine, now);
206-
changedRoutineRepository.save(changedRoutineForDelete);
181+
else if (request.getRoutineType() == RoutineType.CHANGED_ROUTINE) {
182+
ChangedRoutine changedRoutine = routineValidator.validateChangedRoutine(user, request.getRoutineId(),
183+
request.getHistorySeq());
207184

208-
List<SubRoutine> subRoutines = subRoutineRepository.findByRoutineId(routine.getRoutinePk().getId());
209-
210-
// 변경 서브루틴으로 전환
211-
for (SubRoutine subRoutine : subRoutines) {
212-
ChangedSubRoutine changedSubRoutineForDelete =
213-
routineFactory.createChangedSubRoutineForDelete(subRoutine, now, changedRoutineForDelete);
214-
changedSubRoutineRepository.save(changedSubRoutineForDelete);
215-
}
216-
}
217-
else if (request.getRoutineType() == RoutineType.CHANGED_ROUTINE) {
218-
ChangedRoutine changedRoutine = routineValidator.validateChangedRoutine(user, request.getRoutineId(),
219-
request.getHistorySeq());
220-
221-
// 기존 변경 루틴의 결정 코드를 "오늘만 루틴 삭제"로 변경
222-
changedRoutine.updateChangedDivCode(ChangedDivCode.TODAY_DELETE);
223-
}
185+
// 기존 변경 루틴의 결정 코드를 "오늘만 루틴 삭제"로 변경
186+
changedRoutine.updateChangedDivCode(ChangedDivCode.TODAY_DELETE);
224187
}
225188

226189
// routineCompletionId에 해당하는 루틴 완료 여부 데이터 삭제

0 commit comments

Comments
 (0)