[TB-14] 영수증 수정 API 기능 구현 (#21)

baekjaehyuk · web-flow · commit 3d99f0156b85 · 2025-04-12T22:49:11.000+09:00
diff --git a/src/main/java/com/ClubAccount_BE/receipt/adapter/in/web/CreateReceiptApi.java b/src/main/java/com/ClubAccount_BE/receipt/adapter/in/web/CreateReceiptApi.java
@@ -1,7 +1,7 @@
 package com.ClubAccount_BE.receipt.adapter.in.web;
 
 import com.ClubAccount_BE.core.meta.LoginUser;
-import com.ClubAccount_BE.receipt.adapter.in.web.dto.request.CreateReceiptRequestDto;
+import com.ClubAccount_BE.receipt.adapter.in.web.dto.request.ReceiptRequest;
 import com.ClubAccount_BE.receipt.adapter.in.web.dto.response.CreateReceiptResponseDto;
 import com.ClubAccount_BE.user.domain.User;
 import io.swagger.v3.oas.annotations.Operation;
@@ -16,6 +16,6 @@ public interface CreateReceiptApi {
     CreateReceiptResponseDto createReceipt(
             @LoginUser User user,
             @RequestPart(value = "image") MultipartFile image,
-            @RequestPart(value = "request") CreateReceiptRequestDto createReceiptRequestDto
+            @RequestPart(value = "request") ReceiptRequest receiptRequest
     );
 }
diff --git a/src/main/java/com/ClubAccount_BE/receipt/adapter/in/web/CreateReceiptController.java b/src/main/java/com/ClubAccount_BE/receipt/adapter/in/web/CreateReceiptController.java
@@ -1,7 +1,7 @@
 package com.ClubAccount_BE.receipt.adapter.in.web;
 
 import com.ClubAccount_BE.core.meta.LoginUser;
-import com.ClubAccount_BE.receipt.adapter.in.web.dto.request.CreateReceiptRequestDto;
+import com.ClubAccount_BE.receipt.adapter.in.web.dto.request.ReceiptRequest;
 import com.ClubAccount_BE.receipt.adapter.in.web.dto.response.CreateReceiptResponseDto;
 import com.ClubAccount_BE.receipt.application.port.in.CreateReceiptUseCase;
 import com.ClubAccount_BE.user.domain.User;
@@ -23,8 +23,8 @@ public class CreateReceiptController implements CreateReceiptApi {
     public CreateReceiptResponseDto createReceipt(
             @LoginUser User user,
             @RequestPart(value = "image", required = false) MultipartFile image,
-            @RequestPart(value = "request") CreateReceiptRequestDto createReceiptRequestDto
+            @RequestPart(value = "request") ReceiptRequest receiptRequest
     ) {
-        return createReceiptUseCase.createReceipt(user, image, createReceiptRequestDto);
+        return createReceiptUseCase.createReceipt(user, image, receiptRequest);
     }
 }
diff --git a/src/main/java/com/ClubAccount_BE/receipt/adapter/in/web/UpdateReceiptApi.java b/src/main/java/com/ClubAccount_BE/receipt/adapter/in/web/UpdateReceiptApi.java
@@ -0,0 +1,21 @@
+package com.ClubAccount_BE.receipt.adapter.in.web;
+
+import com.ClubAccount_BE.core.meta.LoginUser;
+import com.ClubAccount_BE.receipt.adapter.in.web.dto.request.ReceiptRequest;
+import com.ClubAccount_BE.user.domain.User;
+import io.swagger.v3.oas.annotations.Operation;
+import io.swagger.v3.oas.annotations.tags.Tag;
+import org.springframework.web.bind.annotation.PathVariable;
+import org.springframework.web.bind.annotation.RequestBody;
+
+@Tag(name = "Update Receipt", description = "영수증 수정 API")
+public interface UpdateReceiptApi {
+
+    @Operation(summary = "영수증 수정", description = "파싱된 영수증 정보를 수정한다.")
+    Long updateReceipt(
+            @LoginUser User user,
+            @PathVariable("receiptId") Long receiptId,
+            @RequestBody ReceiptRequest receiptRequest
+    );
+
+}
diff --git a/src/main/java/com/ClubAccount_BE/receipt/adapter/in/web/UpdateReceiptController.java b/src/main/java/com/ClubAccount_BE/receipt/adapter/in/web/UpdateReceiptController.java
@@ -0,0 +1,29 @@
+package com.ClubAccount_BE.receipt.adapter.in.web;
+
+import com.ClubAccount_BE.core.meta.LoginUser;
+import com.ClubAccount_BE.receipt.adapter.in.web.dto.request.ReceiptRequest;
+import com.ClubAccount_BE.receipt.application.port.in.UpdateReceiptUseCase;
+import com.ClubAccount_BE.user.domain.User;
+import lombok.RequiredArgsConstructor;
+import org.springframework.web.bind.annotation.PathVariable;
+import org.springframework.web.bind.annotation.PutMapping;
+import org.springframework.web.bind.annotation.RequestBody;
+import org.springframework.web.bind.annotation.RequestMapping;
+import org.springframework.web.bind.annotation.RestController;
+
+@RestController
+@RequiredArgsConstructor
+@RequestMapping("/api/v1/receipts")
+public class UpdateReceiptController implements UpdateReceiptApi {
+
+    private final UpdateReceiptUseCase updateReceiptUseCase;
+
+    @PutMapping("/{receiptId}")
+    public Long updateReceipt(
+            @LoginUser User user,
+            @PathVariable("receiptId") Long receiptId,
+            @RequestBody ReceiptRequest receiptRequest
+    ) {
+        return updateReceiptUseCase.updateReceipt(user, receiptId, receiptRequest);
+    }
+}
diff --git a/src/main/java/com/ClubAccount_BE/receipt/adapter/in/web/dto/request/ReceiptItemRequest.java b/src/main/java/com/ClubAccount_BE/receipt/adapter/in/web/dto/request/ReceiptItemRequest.java
@@ -3,7 +3,7 @@
 import io.swagger.v3.oas.annotations.media.Schema;
 import java.math.BigDecimal;
 
-public record CreateReceiptItemRequestDto(
+public record ReceiptItemRequest(
 
         @Schema(description = "영수증 아이템 이름", example = "치킨")
         String name,
@@ -13,7 +13,7 @@ public record CreateReceiptItemRequestDto(
 
         @Schema(description = "영수증 아이템 총 가격", example = "20000")
         BigDecimal totalPrice,
-        
+
         @Schema(description = "영수증 아이템 수량", example = "2")
         int quantity
 ) {
diff --git a/src/main/java/com/ClubAccount_BE/receipt/adapter/in/web/dto/request/ReceiptRequest.java b/src/main/java/com/ClubAccount_BE/receipt/adapter/in/web/dto/request/ReceiptRequest.java
@@ -6,7 +6,7 @@
 import java.time.LocalDate;
 import java.util.List;
 
-public record CreateReceiptRequestDto(
+public record ReceiptRequest(
 
         @Schema(
                 description = "영수증 분류 카테고리",
@@ -29,7 +29,7 @@ public record CreateReceiptRequestDto(
         String etc,
 
         @Schema(description = "영수증 내 아이템 리스트")
-        List<CreateReceiptItemRequestDto> receiptItems
+        List<ReceiptItemRequest> receiptItems
 ) {
 
 }
diff --git a/src/main/java/com/ClubAccount_BE/receipt/adapter/out/ReceiptRepositoryAdapter.java b/src/main/java/com/ClubAccount_BE/receipt/adapter/out/ReceiptRepositoryAdapter.java
@@ -4,11 +4,11 @@
 import com.ClubAccount_BE.receipt.adapter.out.persistence.repository.ReceiptRepository;
 import com.ClubAccount_BE.receipt.application.port.out.CreateReceiptPort;
 import com.ClubAccount_BE.receipt.application.port.out.FindReceiptPort;
+import com.ClubAccount_BE.receipt.application.port.out.UpdateReceiptPort;
 import com.ClubAccount_BE.receipt.domain.Receipt;
 import com.ClubAccount_BE.receipt.domain.ReceiptItem;
 import com.ClubAccount_BE.receipt.mapper.ReceiptMapper;
 import com.ClubAccount_BE.user.domain.User;
-import com.ClubAccount_BE.user.mapper.UserMapper;
 import java.util.List;
 import lombok.RequiredArgsConstructor;
 import org.springframework.data.domain.Page;
@@ -17,16 +17,16 @@
 
 @Component
 @RequiredArgsConstructor
-public class ReceiptRepositoryAdapter implements CreateReceiptPort, FindReceiptPort {
+public class ReceiptRepositoryAdapter
+        implements CreateReceiptPort, FindReceiptPort, UpdateReceiptPort {
 
     private final ReceiptRepository receiptRepository;
 
     @Override
     public Long createReceipt(Receipt receipt, List<ReceiptItem> receiptItems) {
 
         ReceiptEntity receiptEntity = ReceiptMapper.toEntity(receipt);
-        ReceiptMapper.toEntity(receiptItems, receiptEntity);
-
+        receiptEntity.replaceReceiptItem(receiptItems);
         return receiptRepository
                 .save(receiptEntity)
                 .getId();
@@ -42,8 +42,22 @@ public Page<Receipt> getReceipts(User user, Pageable pageable) {
     @Override
     public Receipt getReceipt(User user, Long receiptId) {
         return receiptRepository
-                .findByUserAndId(UserMapper.toEntity(user), receiptId)
+                .findById(receiptId)
                 .map(ReceiptMapper::toDomain)
                 .orElseThrow(() -> new IllegalArgumentException("Receipt not found"));
     }
+
+    @Override
+    public Long updateReceipt(
+            Long receiptId,
+            Receipt receipt,
+            List<ReceiptItem> receiptItems
+    ) {
+        ReceiptEntity receiptEntity = receiptRepository.findReceiptById(receiptId)
+                .orElseThrow(() -> new IllegalArgumentException("Receipt not found"));
+
+        receiptEntity.updateReceipt(receipt);
+        receiptEntity.replaceReceiptItem(receiptItems);
+        return receiptEntity.getId();
+    }
 }
diff --git a/src/main/java/com/ClubAccount_BE/receipt/adapter/out/persistence/entity/ReceiptEntity.java b/src/main/java/com/ClubAccount_BE/receipt/adapter/out/persistence/entity/ReceiptEntity.java
@@ -1,7 +1,10 @@
 package com.ClubAccount_BE.receipt.adapter.out.persistence.entity;
 
 import com.ClubAccount_BE.core.entity.TimeBaseEntity;
+import com.ClubAccount_BE.receipt.domain.Receipt;
+import com.ClubAccount_BE.receipt.domain.ReceiptItem;
 import com.ClubAccount_BE.receipt.domain.type.ReceiptCategory;
+import com.ClubAccount_BE.receipt.mapper.ReceiptItemMapper;
 import com.ClubAccount_BE.user.adapter.out.persistence.entity.UserEntity;
 import jakarta.persistence.CascadeType;
 import jakarta.persistence.Column;
@@ -65,10 +68,26 @@ public class ReceiptEntity extends TimeBaseEntity {
     @OneToMany(mappedBy = "receipt", cascade = CascadeType.ALL, orphanRemoval = true)
     private List<ReceiptItemEntity> receiptItems = new ArrayList<>();
 
+    public void updateReceipt(Receipt receipt) {
+        this.category = receipt.getCategory();
+        this.categoryName = receipt.getCategoryName();
+        this.businessName = receipt.getBusinessName();
+        this.amount = receipt.getAmount();
+        this.date = receipt.getDate();
+        this.etc = receipt.getEtc();
+    }
+
     public void addReceiptItem(ReceiptItemEntity receiptItem) {
         this.receiptItems.add(receiptItem);
         if (receiptItem.getReceipt() != this) {
             receiptItem.addReceipt(this);
         }
     }
+
+    public void replaceReceiptItem(List<ReceiptItem> receiptItems) {
+        this.receiptItems.clear();
+        receiptItems.stream()
+                .map(ReceiptItemMapper::toEntity)
+                .forEach(this::addReceiptItem);
+    }
 }
diff --git a/src/main/java/com/ClubAccount_BE/receipt/adapter/out/persistence/repository/ReceiptRepository.java b/src/main/java/com/ClubAccount_BE/receipt/adapter/out/persistence/repository/ReceiptRepository.java
@@ -1,7 +1,6 @@
 package com.ClubAccount_BE.receipt.adapter.out.persistence.repository;
 
 import com.ClubAccount_BE.receipt.adapter.out.persistence.entity.ReceiptEntity;
-import com.ClubAccount_BE.user.adapter.out.persistence.entity.UserEntity;
 import java.util.Optional;
 import org.springframework.data.domain.Page;
 import org.springframework.data.domain.Pageable;
@@ -13,5 +12,5 @@ public interface ReceiptRepository extends JpaRepository<ReceiptEntity, Long> {
     Page<ReceiptEntity> findAllByUserId(Long id, Pageable pageable);
 
     @EntityGraph(attributePaths = "receiptItems")
-    Optional<ReceiptEntity> findByUserAndId(UserEntity user, Long id);
+    Optional<ReceiptEntity> findReceiptById(Long receiptId);
 }
diff --git a/src/main/java/com/ClubAccount_BE/receipt/application/port/in/CreateReceiptUseCase.java b/src/main/java/com/ClubAccount_BE/receipt/application/port/in/CreateReceiptUseCase.java
@@ -1,6 +1,6 @@
 package com.ClubAccount_BE.receipt.application.port.in;
 
-import com.ClubAccount_BE.receipt.adapter.in.web.dto.request.CreateReceiptRequestDto;
+import com.ClubAccount_BE.receipt.adapter.in.web.dto.request.ReceiptRequest;
 import com.ClubAccount_BE.receipt.adapter.in.web.dto.response.CreateReceiptResponseDto;
 import com.ClubAccount_BE.user.domain.User;
 import org.springframework.web.multipart.MultipartFile;
@@ -10,6 +10,6 @@ public interface CreateReceiptUseCase {
     CreateReceiptResponseDto createReceipt(
             User user,
             MultipartFile image,
-            CreateReceiptRequestDto createReceiptRequestDto
+            ReceiptRequest receiptRequest
     );
 }
diff --git a/src/main/java/com/ClubAccount_BE/receipt/application/port/in/FindReceiptUseCase.java b/src/main/java/com/ClubAccount_BE/receipt/application/port/in/FindReceiptUseCase.java
@@ -5,9 +5,7 @@
 import com.ClubAccount_BE.receipt.adapter.in.web.dto.response.FindReceiptResponseDto;
 import com.ClubAccount_BE.user.domain.User;
 import org.springframework.data.domain.Pageable;
-import org.springframework.stereotype.Component;
 
-@Component
 public interface FindReceiptUseCase {
 
     PagingResponse<FindReceiptResponseDto> getReceipts(User user, Pageable pageable);
diff --git a/src/main/java/com/ClubAccount_BE/receipt/application/port/in/UpdateReceiptUseCase.java b/src/main/java/com/ClubAccount_BE/receipt/application/port/in/UpdateReceiptUseCase.java
@@ -0,0 +1,9 @@
+package com.ClubAccount_BE.receipt.application.port.in;
+
+import com.ClubAccount_BE.receipt.adapter.in.web.dto.request.ReceiptRequest;
+import com.ClubAccount_BE.user.domain.User;
+
+public interface UpdateReceiptUseCase {
+
+    Long updateReceipt(User user, Long receiptId, ReceiptRequest receiptRequest);
+}
diff --git a/src/main/java/com/ClubAccount_BE/receipt/application/port/out/UpdateReceiptPort.java b/src/main/java/com/ClubAccount_BE/receipt/application/port/out/UpdateReceiptPort.java
@@ -0,0 +1,10 @@
+package com.ClubAccount_BE.receipt.application.port.out;
+
+import com.ClubAccount_BE.receipt.domain.Receipt;
+import com.ClubAccount_BE.receipt.domain.ReceiptItem;
+import java.util.List;
+
+public interface UpdateReceiptPort {
+
+    Long updateReceipt(Long receiptId, Receipt receipt, List<ReceiptItem> receiptItem);
+}
diff --git a/src/main/java/com/ClubAccount_BE/receipt/application/service/CreateReceiptService.java b/src/main/java/com/ClubAccount_BE/receipt/application/service/CreateReceiptService.java
@@ -1,14 +1,14 @@
 package com.ClubAccount_BE.receipt.application.service;
 
-import com.ClubAccount_BE.receipt.adapter.in.web.dto.request.CreateReceiptRequestDto;
+import com.ClubAccount_BE.receipt.adapter.in.web.dto.request.ReceiptRequest;
 import com.ClubAccount_BE.receipt.adapter.in.web.dto.response.CreateReceiptResponseDto;
 import com.ClubAccount_BE.receipt.application.port.in.CreateReceiptUseCase;
 import com.ClubAccount_BE.receipt.application.port.out.CreateReceiptPort;
 import com.ClubAccount_BE.receipt.application.port.out.UploadReceiptPort;
 import com.ClubAccount_BE.receipt.domain.Receipt;
 import com.ClubAccount_BE.receipt.domain.ReceiptItem;
+import com.ClubAccount_BE.receipt.domain.service.ReceiptItemEditor;
 import com.ClubAccount_BE.user.domain.User;
-import com.ClubAccount_BE.user.mapper.UserMapper;
 import java.util.List;
 import lombok.RequiredArgsConstructor;
 import org.springframework.stereotype.Service;
@@ -22,42 +22,27 @@ public class CreateReceiptService implements CreateReceiptUseCase {
 
     private final UploadReceiptPort uploadReceiptPort;
 
+    private final ReceiptItemEditor receiptItemEditor;
+
     @Override
     public CreateReceiptResponseDto createReceipt(
             User user,
             MultipartFile image,
-            CreateReceiptRequestDto createReceiptRequestDto
+            ReceiptRequest receiptRequest
     ) {
         Receipt receipt = Receipt.create(
-                UserMapper.toEntity(user),
-                createReceiptRequestDto.category(),
-                createReceiptRequestDto.categoryName(),
-                createReceiptRequestDto.businessName(),
-                createReceiptRequestDto.date(),
-                createReceiptRequestDto.amount(),
-                createReceiptRequestDto.etc(),
-                //TODO : 직접 등록시 기본 이미지로 설정 로직 추가
+                user,
+                receiptRequest.category(),
+                receiptRequest.categoryName(),
+                receiptRequest.businessName(),
+                receiptRequest.date(),
+                receiptRequest.amount(),
+                receiptRequest.etc(),
                 image == null ? "" : uploadReceiptPort.uploadReceipt(image)
         );
 
-        List<ReceiptItem> receiptItems = toReceiptItems(createReceiptRequestDto, receipt);
-
+        List<ReceiptItem> receiptItems = receiptItemEditor.toReceiptItems(receiptRequest, receipt);
         Long receiptId = createReceiptPort.createReceipt(receipt, receiptItems);
         return CreateReceiptResponseDto.of(receiptId, receipt);
     }
-
-    private List<ReceiptItem> toReceiptItems(
-            CreateReceiptRequestDto createReceiptRequestDto,
-            Receipt receipt
-    ) {
-        return createReceiptRequestDto.receiptItems().stream()
-                .map(receiptItem -> ReceiptItem.create(
-                        receipt,
-                        receiptItem.name(),
-                        receiptItem.price(),
-                        receiptItem.totalPrice(),
-                        receiptItem.quantity()
-                ))
-                .toList();
-    }
 }
diff --git a/src/main/java/com/ClubAccount_BE/receipt/application/service/UpdateReceiptService.java b/src/main/java/com/ClubAccount_BE/receipt/application/service/UpdateReceiptService.java
@@ -0,0 +1,44 @@
+package com.ClubAccount_BE.receipt.application.service;
+
+import com.ClubAccount_BE.receipt.adapter.in.web.dto.request.ReceiptRequest;
+import com.ClubAccount_BE.receipt.application.port.in.UpdateReceiptUseCase;
+import com.ClubAccount_BE.receipt.application.port.out.UpdateReceiptPort;
+import com.ClubAccount_BE.receipt.domain.Receipt;
+import com.ClubAccount_BE.receipt.domain.ReceiptItem;
+import com.ClubAccount_BE.receipt.domain.service.ReceiptItemEditor;
+import com.ClubAccount_BE.user.domain.User;
+import java.util.List;
+import lombok.RequiredArgsConstructor;
+import org.springframework.stereotype.Service;
+import org.springframework.transaction.annotation.Transactional;
+
+@Service
+@RequiredArgsConstructor
+@Transactional
+public class UpdateReceiptService implements UpdateReceiptUseCase {
+
+    private final UpdateReceiptPort updateReceiptPort;
+
+    private final ReceiptItemEditor receiptItemEditor;
+
+    @Override
+    public Long updateReceipt(
+            User user,
+            Long receiptId,
+            ReceiptRequest receiptRequest
+    ) {
+        Receipt receipt = Receipt.update(
+                receiptId,
+                user,
+                receiptRequest.category(),
+                receiptRequest.categoryName(),
+                receiptRequest.businessName(),
+                receiptRequest.date(),
+                receiptRequest.amount(),
+                receiptRequest.etc()
+        );
+
+        List<ReceiptItem> receiptItems = receiptItemEditor.toReceiptItems(receiptRequest, receipt);
+        return updateReceiptPort.updateReceipt(receiptId, receipt, receiptItems);
+    }
+}
diff --git a/src/main/java/com/ClubAccount_BE/receipt/domain/Receipt.java b/src/main/java/com/ClubAccount_BE/receipt/domain/Receipt.java
diff --git a/src/main/java/com/ClubAccount_BE/receipt/domain/ReceiptItem.java b/src/main/java/com/ClubAccount_BE/receipt/domain/ReceiptItem.java
diff --git a/src/main/java/com/ClubAccount_BE/receipt/domain/service/ReceiptItemEditor.java b/src/main/java/com/ClubAccount_BE/receipt/domain/service/ReceiptItemEditor.java
diff --git a/src/main/java/com/ClubAccount_BE/receipt/mapper/ReceiptItemMapper.java b/src/main/java/com/ClubAccount_BE/receipt/mapper/ReceiptItemMapper.java
diff --git a/src/main/java/com/ClubAccount_BE/receipt/mapper/ReceiptMapper.java b/src/main/java/com/ClubAccount_BE/receipt/mapper/ReceiptMapper.java
diff --git a/src/test/java/com/ClubAccount_BE/factory/receipt/ReceiptTestFactory.java b/src/test/java/com/ClubAccount_BE/factory/receipt/ReceiptTestFactory.java
diff --git a/src/test/java/com/ClubAccount_BE/receipt/mapper/ReceiptMapperTest.java b/src/test/java/com/ClubAccount_BE/receipt/mapper/ReceiptMapperTest.java
