Merge pull request #157 from Money-Touch/fix/#156

[#156] 🐛Fix: 고정비 등록 시 메모 사항 제거 및 고정비&내 소비 루틴 목록 조회 시 연도, 월 필터링 추가

Author: paul0755

src/main/java/com/server/money_touch/domain/consumptionRecord/converter/consumptionRecord/ConsumptionRecordConverter.java

@@ -14,26 +14,39 @@
 import java.time.LocalDate;
 import java.time.LocalDateTime;
 import java.util.List;
+import java.util.Optional;
 
 public class ConsumptionRecordConverter {
 
     // 일일 소비 기록 시 소비 카테고리 엔티티 생성
-    public static ConsumptionRecord toDailyConsumptionRecord(User user, ConsumptionCategory consumptionCategory, HouseholdConsumptionRequest.DailyConsumptionCreateDTO requestDTO, Boolean isPublic) {
-        return ConsumptionRecord.builder()
+    public static ConsumptionRecord toDailyConsumptionRecord(
+            User user,
+            ConsumptionCategory consumptionCategory,
+            HouseholdConsumptionRequest.DailyConsumptionCreateDTO requestDTO,
+            Boolean isPublic
+    ) {
+        var builder = ConsumptionRecord.builder()
                 .user(user)
                 .consumptionCategory(consumptionCategory)
                 .amount(requestDTO.getAmount())
                 .content(requestDTO.getContent())
-                .memo(requestDTO.getMemo())
                 .consumeDate(requestDTO.getConsumeDate())
                 .isPublic(isPublic) // 일일 소비 기록은 피드 없이 가계부에만 등록
                 .commentCount(0)
                 .wiseCount(0)
                 .wasteCount(0)
-                .viewCount(0)
-                .build();
+                .viewCount(0);
+
+        // memo가 있을 때(공백 제외)만 세팅
+        Optional.ofNullable(requestDTO.getMemo())
+                .map(String::trim)
+                .filter(s -> !s.isEmpty())
+                .ifPresent(builder::memo);
+
+        return builder.build();
     }
 
+
     // 소비 기록 응답
     public static ConsumptionRecordResponse.ConsumptionRecordCreateResultDTO toConsumptionRecordCreateResultDTO(Long consumptionRecordId){
         return ConsumptionRecordResponse.ConsumptionRecordCreateResultDTO.builder()

src/main/java/com/server/money_touch/domain/fixedConsumption/controller/FixedConsumptionController.java

@@ -36,7 +36,7 @@ public class FixedConsumptionController {
 
     @Operation(
             summary = "고정비 등록 API",
-            description = "고정비 등록 API 입니다. 금액, 카테고리, 항목명, 메모를 RequestBody로 입력받아 고정비를 등록합니다."
+            description = "고정비 등록 API 입니다. 금액, 카테고리, 항목명, 메모를 RequestBody로 입력받아 고정비를 등록합니다. 메모가 없다면 RequestBody에 포함하지 않아도 됩니다."
     )
     @ApiSuccessCodeExample(resultClass = FixedConsumptionResponse.FixedConsumptionCreateResultDTO.class)
     @ApiErrorCodeExamples({
@@ -107,12 +107,14 @@ public ApiResponse<String> deleteFixedConsumption(@PathVariable Long fixedConsum
             @ApiErrorCodeExample(value = ErrorStatus.class, name = "_INTERNAL_SERVER_ERROR"),
     })
     @Parameters({
+            @Parameter(name = "year", description = "조회하려는 소비 연도", example = "2025", required = true),
+            @Parameter(name = "month", description = "조회하려는 소비 월", example = "7", required = true),
             @Parameter(name = "cursorId", description = "커서 (이전 요청의 마지막 cursorId). 첫 요청 시 생략", example = "3", required = false)
     })
     @GetMapping("list")
-    public ApiResponse<FixedConsumptionResponse.FixedConsumptionCursorResultDTO> getFixedConsumptions(@RequestParam(required = false) Long cursorId, HttpServletRequest servletRequest) {
+    public ApiResponse<FixedConsumptionResponse.FixedConsumptionCursorResultDTO> getFixedConsumptions(@RequestParam(required = true) Integer year, @RequestParam(required = true) Integer month, @RequestParam(required = false) Long cursorId, HttpServletRequest servletRequest) {
         Long userId = authUtil.getUserIdFromRequest(servletRequest);
-        FixedConsumptionResponse.FixedConsumptionCursorResultDTO response = fixedConsumptionQueryService.getFixedConsumptions(userId, cursorId);
+        FixedConsumptionResponse.FixedConsumptionCursorResultDTO response = fixedConsumptionQueryService.getFixedConsumptions(userId, year, month, cursorId);
         return ApiResponse.onSuccess(response);
     }
 

src/main/java/com/server/money_touch/domain/fixedConsumption/dto/FixedConsumptionRequest.java

@@ -29,7 +29,6 @@ public static class FixedConsumptionCreateDTO{
         private String content;
 
         @Schema(description = "메모", example = "가족 공유 요금제")
-        @NotNull(message = "메모는 필수입니다.")
         @Size(max = 1000, message = "항목명은 1000자 이하로 입력해주세요.")
         private String memo;
     }

src/main/java/com/server/money_touch/domain/fixedConsumption/entity/FixedConsumption.java

@@ -17,7 +17,7 @@ public class FixedConsumption extends BaseEntity {
     @Column(nullable = false, length = 20)
     private String fixedConsumptionContent;
 
-    @Column(nullable = false, length = 1000)
+    @Column(length = 1000)
     private String fixedConsumptionMemo;
 
     @Column(nullable = false, length = 20)

src/main/java/com/server/money_touch/domain/fixedConsumption/repository/FixedConsumptionRepositoryCustom.java

@@ -8,5 +8,5 @@
 
 public interface FixedConsumptionRepositoryCustom {
     // 커서 기반 고정비 목록 조회
-    Slice<FixedConsumption> findFixedConsumptionsByCursor(Long userId, Long cursorId, Pageable pageable);
+    Slice<FixedConsumption> findFixedConsumptionsByCursor(Long userId, Long cursorId, int year, int month, Pageable pageable);
 }

src/main/java/com/server/money_touch/domain/fixedConsumption/repository/FixedConsumptionRepositoryImpl.java

@@ -10,6 +10,8 @@
 import org.springframework.data.domain.SliceImpl;
 import org.springframework.stereotype.Repository;
 
+import java.time.LocalDate;
+import java.time.LocalDateTime;
 import java.util.List;
 
 @RequiredArgsConstructor
@@ -21,13 +23,23 @@ public class FixedConsumptionRepositoryImpl implements FixedConsumptionRepositor
 
     // 커서 기반 고정비 목록 조회
     @Override
-    public Slice<FixedConsumption> findFixedConsumptionsByCursor(Long userId, Long cursorId, Pageable pageable) {
+    public Slice<FixedConsumption> findFixedConsumptionsByCursor(
+            Long userId, Long cursorId, int year, int month, Pageable pageable) {
+
         BooleanBuilder condition = new BooleanBuilder();
         condition.and(fixed.user.id.eq(userId));
+
+        // 커서 조건
         if (cursorId != null) {
-            condition.and(fixed.id.lt(cursorId)); // id 역순으로 작아야 이후 커서
+            condition.and(fixed.id.lt(cursorId));
         }
 
+        // ✅ 해당 달보다 이후에 생성된 고정비는 보이지 않도록 필터링
+        LocalDate firstDay = LocalDate.of(year, month, 1);
+        LocalDateTime nextMonthStart = firstDay.plusMonths(1).atStartOfDay();
+
+        condition.and(fixed.createdAt.before(nextMonthStart));
+
         int pageSize = pageable.getPageSize();
         List<FixedConsumption> results = queryFactory
                 .selectFrom(fixed)
@@ -38,7 +50,7 @@ public Slice<FixedConsumption> findFixedConsumptionsByCursor(Long userId, Long c
 
         boolean hasNext = results.size() > pageSize;
         if (hasNext) {
-            results.remove(pageSize); // Slice에 반환할 범위까지만 유지
+            results.remove(pageSize);
         }
 
         return new SliceImpl<>(results, pageable, hasNext);

src/main/java/com/server/money_touch/domain/fixedConsumption/service/FixedConsumptionCommandServiceImpl.java

@@ -30,8 +30,6 @@
 import java.time.LocalDate;
 import java.time.LocalDateTime;
 import java.util.Map;
-import java.util.Optional;
-import java.util.concurrent.atomic.AtomicInteger;
 import java.util.stream.Collectors;
 
 

src/main/java/com/server/money_touch/domain/fixedConsumption/service/FixedConsumptionQueryService.java

@@ -8,5 +8,5 @@ public interface FixedConsumptionQueryService {
     Boolean existsFixedConsumptionById(Long fixedConsumptionId);
 
     // 고정비 목록 조회 (커서 기반 무한스크롤)
-    FixedConsumptionResponse.FixedConsumptionCursorResultDTO getFixedConsumptions(@ExistUser Long userId, Long cursorId);
+    FixedConsumptionResponse.FixedConsumptionCursorResultDTO getFixedConsumptions(@ExistUser Long userId, Integer year, Integer month, Long cursorId);
 }

src/main/java/com/server/money_touch/domain/fixedConsumption/service/FixedConsumptionQueryServiceImpl.java

@@ -42,37 +42,33 @@ public Boolean existsFixedConsumptionById(Long fixedConsumptionId) {
      * - 페이지 사이즈는 고정(PAGE_SIZE)
      * - 커서 기반 페이징 방식으로, 마지막 조회된 고정비 ID 이후 데이터를 가져옵니다.
      * - 응답에는 고정비 목록, 다음 커서 ID 및 페이징 정보가 포함됩니다.
-     *
-     * @param userId   사용자 ID
-     * @param cursorId 마지막으로 조회된 고정비 ID (null이면 첫 페이지)
-     * @return FixedConsumptionCursorResultDTO (고정비 목록 + 페이징 정보)
      */
     @Override
-    public FixedConsumptionResponse.FixedConsumptionCursorResultDTO getFixedConsumptions(Long userId, Long cursorId) {
+    public FixedConsumptionResponse.FixedConsumptionCursorResultDTO getFixedConsumptions(
+            Long userId, Integer year, Integer month, Long cursorId) {
+
         // 사용자 조회
         User user = userRepository.findById(userId)
                 .orElseThrow(() -> new ErrorHandler(ErrorStatus.USER_NOT_FOUND));
 
         Pageable pageable = PageRequest.of(0, PAGE_SIZE);
 
-        // 고정비 목록 조회 (Slice)
+        // 고정비 목록 조회 (Slice) - year, month 추가
         Slice<FixedConsumption> slice = fixedConsumptionRepository
-                .findFixedConsumptionsByCursor(userId, cursorId, pageable);
+                .findFixedConsumptionsByCursor(userId, cursorId, year, month, pageable);
 
-        // 고정비 엔티티 → DTO 변환
         List<FixedConsumptionResponse.FixedConsumptionDetailDTO> content = slice.getContent().stream()
                 .map(FixedConsumptionConverter::toFixedConsumptionDetailDTO)
-                .collect(Collectors.toList());
+                .toList();
 
         // 다음 커서 ID 설정
         Long nextCursorId = content.isEmpty() ? null : content.get(content.size() - 1).getFixedConsumptionId();
 
-        log.info("고정비 목록 조회(커서 기반 무한스크롤) 완료 - userId: {}, cursorId: {}, nextCursorId: {}", userId, cursorId, nextCursorId);
         return FixedConsumptionConverter.toFixedConsumptionCursorResultDTO(
                 content,
                 slice.hasNext(),
                 nextCursorId,
-                cursorId == null // 첫 페이지 여부
+                cursorId == null
         );
     }
 }

src/main/java/com/server/money_touch/domain/routine/controller/RoutineController.java

@@ -75,18 +75,27 @@ public ApiResponse<RoutineResponse.RoutineCreateResultDTO> postRoutine(@Valid @R
     // 내 소비 루틴 목록 조회
     @Operation(
             summary = "내 소비 루틴 목록 조회 API (커서 기반 무한스크롤)",
-            description = "가계부에서 사용자가 등록한 소비 루틴 목록을 스크롤 형식으로 조회하는 API입니다."
+            description = "가계부에서 사용자가 등록한 소비 루틴 목록을 스크롤 형식으로 조회하는 API입니다. 특정 연도/월별 조회가 가능합니다."
     )
     @ApiErrorCodeExamples({
             @ApiErrorCodeExample(value = ErrorStatus.class, name = "USER_NOT_FOUND"),
             @ApiErrorCodeExample(value = ErrorStatus.class, name = "_BAD_REQUEST"),
             @ApiErrorCodeExample(value = ErrorStatus.class, name = "_INTERNAL_SERVER_ERROR"),
     })
-    @Parameter(name = "cursorId", description = "커서(이전 요청에서 마지막 소비 루틴 아이디), 첫번째 요청일 시에는 파라미터에 포함하지 않아도 됩니다.", example = "10", required = false)
+    @Parameters({
+            @Parameter(name = "cursorId", description = "커서(이전 요청에서 마지막 루틴 ID), 첫 요청 시 생략 가능", example = "10", required = false),
+            @Parameter(name = "year", description = "조회할 연도", example = "2025", required = true),
+            @Parameter(name = "month", description = "조회할 월", example = "7", required = true)
+    })
     @GetMapping("/users")
-    public ApiResponse<RoutineResponse.MyRoutineListDTO> getMyRoutines(@RequestParam(required = false) Long cursorId, HttpServletRequest servletRequest) {
+    public ApiResponse<RoutineResponse.MyRoutineListDTO> getMyRoutines(
+            @RequestParam int year,
+            @RequestParam int month,
+            @RequestParam(required = false) Long cursorId,
+            HttpServletRequest servletRequest) {
+
         Long userId = authUtil.getUserIdFromRequest(servletRequest);
-        RoutineResponse.MyRoutineListDTO response = routineQueryService.getMyRoutineList(userId, cursorId);
+        RoutineResponse.MyRoutineListDTO response = routineQueryService.getMyRoutineList(userId, year, month, cursorId);
         return ApiResponse.onSuccess(response);
     }