refactor/#415 글 삭제, 수정시 게시글 이미지의 soft delete를 위해 update문이 중복해서 나지 않도록 수정함

backend/src/main/java/edonymyeon/backend/image/postimage/domain/PostImageInfo.java

@@ -31,12 +31,6 @@ public PostImageInfo(final String storeName, final Post post) {
     }
 
     public static PostImageInfo of(final ImageInfo imageInfo, final Post post) {
-        final PostImageInfo postImageInfo = new PostImageInfo(imageInfo.getStoreName(), post);
-        //post.addPostImageInfo(postImageInfo);
-        return postImageInfo;
-    }
-
-    public void delete() {
-        this.deleted = true;
+        return new PostImageInfo(imageInfo.getStoreName(), post);
     }
 }

backend/src/main/java/edonymyeon/backend/image/postimage/domain/PostImageInfos.java

@@ -1,20 +1,21 @@
 package edonymyeon.backend.image.postimage.domain;
 
-import static edonymyeon.backend.global.exception.ExceptionInformation.IMAGE_STORE_NAME_INVALID;
-import static edonymyeon.backend.global.exception.ExceptionInformation.POST_IMAGE_COUNT_INVALID;
-
 import edonymyeon.backend.global.exception.EdonymyeonException;
 import edonymyeon.backend.image.domain.ImageInfo;
 import edonymyeon.backend.post.domain.Post;
 import jakarta.persistence.Embeddable;
 import jakarta.persistence.OneToMany;
-import java.util.ArrayList;
-import java.util.List;
 import lombok.AccessLevel;
 import lombok.Getter;
 import lombok.NoArgsConstructor;
 import org.hibernate.annotations.BatchSize;
 
+import java.util.ArrayList;
+import java.util.List;
+
+import static edonymyeon.backend.global.exception.ExceptionInformation.IMAGE_STORE_NAME_INVALID;
+import static edonymyeon.backend.global.exception.ExceptionInformation.POST_IMAGE_COUNT_INVALID;
+
 @NoArgsConstructor(access = AccessLevel.PROTECTED)
 @Getter
 @Embeddable
@@ -53,11 +54,6 @@ private boolean isInvalidImageCount(final Integer imageCount) {
         return imageCount > MAX_IMAGE_COUNT;
     }
 
-    public void addAll(final List<PostImageInfo> imagesToAdd) {
-        validateImageCount(this.postImageInfos.size() + imagesToAdd.size());
-        this.postImageInfos.addAll(imagesToAdd);
-    }
-
     public void add(final PostImageInfo postImageInfo) {
         if (this.postImageInfos.contains(postImageInfo)) {
             return;
@@ -72,14 +68,14 @@ private void validateImageAdditionCount() {
         }
     }
 
-    public void update(final List<String> remainedStoreNames, final List<PostImageInfo> newPostImageInfos) {
+    public List<Long> getImageIdsToDeleteBy(final List<String> remainedStoreNames, final List<PostImageInfo> newPostImageInfos) {
         final List<PostImageInfo> imagesToDelete = findImagesToDelete(remainedStoreNames);
         int updatedImageCount = this.postImageInfos.size() - imagesToDelete.size() + newPostImageInfos.size();
         validateImageCount(updatedImageCount);
 
-        imagesToDelete.forEach(PostImageInfo::delete);
-        postImageInfos.removeAll(imagesToDelete);
-        postImageInfos.addAll(newPostImageInfos);
+        return imagesToDelete.stream()
+                .map(ImageInfo::getId)
+                .toList();
     }
 
     private List<PostImageInfo> findImagesToDelete(final List<String> remainedStoreNames) {
@@ -93,18 +89,6 @@ private List<PostImageInfo> findImagesToDelete(final List<String> remainedStoreN
         return unmatchedPostImageInfos;
     }
 
-    // todo : 여기 부분 맞게 했나요? 헷갈립니다.
-    public void delete(final List<PostImageInfo> deletedPostImageInfos) {
-        // 어쨌든 deleted = false 인 놈들만 가지고 있어야 하니 지워져야 할 녀석들을 리스트에서 뺀다.
-        this.postImageInfos.removeAll(deletedPostImageInfos);
-        // 지워져야 하는 녀석들을 soft delete
-        deletedPostImageInfos.forEach(PostImageInfo::delete);
-    }
-
-    public void deleteAll() {
-        this.postImageInfos.forEach(PostImageInfo::delete);
-    }
-
     public boolean isEmpty() {
         return this.postImageInfos.isEmpty();
     }

backend/src/main/java/edonymyeon/backend/image/postimage/repository/PostImageInfoRepository.java

@@ -1,20 +1,22 @@
 package edonymyeon.backend.image.postimage.repository;
 
 import edonymyeon.backend.image.postimage.domain.PostImageInfo;
-import java.util.List;
 import org.springframework.data.jpa.repository.JpaRepository;
 import org.springframework.data.jpa.repository.Modifying;
 import org.springframework.data.jpa.repository.Query;
 import org.springframework.data.repository.query.Param;
 
+import java.util.List;
+
 public interface PostImageInfoRepository extends JpaRepository<PostImageInfo, Long> {
 
     List<PostImageInfo> findAllByPostId(final Long postId);
 
     @Modifying
-    @Query("delete from PostImageInfo pi where pi.post.id=:postId")
+    @Query("update PostImageInfo pi set pi.deleted = true  where pi.post.id=:postId")
     void deleteAllByPostId(@Param("postId") final Long postId);
 
-    @Query(value = "select * from post_image_info", nativeQuery = true)
-    List<PostImageInfo> findAllImages();
+    @Modifying
+    @Query("update PostImageInfo pi set pi.deleted = true  where pi.id in (:imageIds)")
+    void deleteAllByIds(@Param("imageIds") final List<Long> imageIds);
 }

backend/src/main/java/edonymyeon/backend/post/application/PostService.java

@@ -1,9 +1,5 @@
 package edonymyeon.backend.post.application;
 
-import static edonymyeon.backend.global.exception.ExceptionInformation.MEMBER_ID_NOT_FOUND;
-import static edonymyeon.backend.global.exception.ExceptionInformation.POST_ID_NOT_FOUND;
-import static edonymyeon.backend.global.exception.ExceptionInformation.POST_MEMBER_NOT_SAME;
-
 import edonymyeon.backend.global.exception.EdonymyeonException;
 import edonymyeon.backend.image.application.ImageService;
 import edonymyeon.backend.image.domain.ImageType;
@@ -18,14 +14,17 @@
 import edonymyeon.backend.post.application.event.PostDeletionEvent;
 import edonymyeon.backend.post.domain.Post;
 import edonymyeon.backend.post.repository.PostRepository;
-import java.util.List;
-import java.util.Objects;
 import lombok.RequiredArgsConstructor;
 import org.springframework.context.ApplicationEventPublisher;
 import org.springframework.stereotype.Service;
 import org.springframework.transaction.annotation.Transactional;
 import org.springframework.web.multipart.MultipartFile;
 
+import java.util.List;
+import java.util.Objects;
+
+import static edonymyeon.backend.global.exception.ExceptionInformation.*;
+
 @RequiredArgsConstructor
 @Service
 public class PostService {
@@ -90,13 +89,12 @@ public void deletePost(final MemberId memberId, final Long postId) {
         final Post post = findPostById(postId);
         checkWriter(member, post);
 
-        // soft delete 시킬 때, 실제 이미지는 보관된다.
-        // todo: 이미지 삭제.. 한번에..
         // todo: 소비내역 삭제할 때, 이벤트 대신 인터페이스로 변경
         applicationEventPublisher.publishEvent(new PostDeletionEvent(post.getId()));
         thumbsService.deleteAllThumbsInPost(postId);
         commentService.deleteAllCommentsInPost(postId);
         post.delete();
+        postImageInfoRepository.deleteAllByPostId(postId);
     }
 
     private Post findPostById(final Long postId) {
@@ -126,13 +124,18 @@ public PostIdResponse updatePost(
         final List<String> remainedImageNames = imageService.convertToStoreName(request.originalImages(), ImageType.POST);
 
         if(isImagesEmpty(imageFilesToAdd)) {
-            post.updateImages(remainedImageNames);
+            final List<Long> imageIdsToDelete = post.getImageIdsToDeleteBy(remainedImageNames);
+            postImageInfoRepository.deleteAllByIds(imageIdsToDelete);
             return new PostIdResponse(postId);
         }
 
         final PostImageInfos imagesToAdd = PostImageInfos.of(post, imageService.saveAll(imageFilesToAdd, ImageType.POST));
-        post.updateImages(remainedImageNames, imagesToAdd); //이때 기존 이미지중 삭제되는 것들은 softDelete
+
+        final List<Long> imageIdsToDelete = post.getImageIdsToDeleteBy(remainedImageNames, imagesToAdd);
+        postImageInfoRepository.deleteAllByIds(imageIdsToDelete); //이때 기존 이미지중 삭제되는 것들은 softDelete
+
         postImageInfoRepository.saveAll(imagesToAdd.getPostImageInfos()); // //새로 추가된 이미지들을 DB에 저장
+
         return new PostIdResponse(postId);
     }
 }

backend/src/main/java/edonymyeon/backend/post/domain/Post.java

@@ -52,7 +52,6 @@ public class Post extends TemporalRecord {
     @JoinColumn(nullable = false)
     private Member member;
 
-    // TODO: cascade
     private PostImageInfos postImageInfos;
 
     @ColumnDefault("0")
@@ -125,6 +124,7 @@ private void validateMember(final Member member) {
         }
     }
 
+    // todo: 이 부분 docs test에서만 쓰고 있어요
     public void addPostImageInfo(final PostImageInfo postImageInfo) {
         this.postImageInfos.add(postImageInfo);
     }
@@ -156,18 +156,16 @@ private void updatePrice(final Long price) {
 
     /**
      * 게시글 수정시 사용, 새로 추가되는 이미지가 없고 기존 이미지에 대한 수정만 일어나는 경우
-     * -> imageNamesToMaintain을 제외하고 삭제한다.
      */
-    public void updateImages(final List<String> remainedImageNames) {
-        postImageInfos.update(remainedImageNames, Collections.emptyList());
+    public List<Long> getImageIdsToDeleteBy(final List<String> remainedImageNames) {
+        return this.postImageInfos.getImageIdsToDeleteBy(remainedImageNames, Collections.emptyList());
     }
 
     /**
      * 게시글 수정시 사용, 새로 추가되는 이미지도 있는 경우
-     * -> imageNamesToMaintain을 제외하고 삭제 후, imagesToAdd를 추가한다.
      */
-    public void updateImages(final List<String> remainedImageNames, final PostImageInfos imagesToAdd) {
-        this.postImageInfos.update(remainedImageNames, imagesToAdd.getPostImageInfos());
+    public List<Long> getImageIdsToDeleteBy(final List<String> remainedImageNames, final PostImageInfos imagesToAdd) {
+        return this.postImageInfos.getImageIdsToDeleteBy(remainedImageNames, imagesToAdd.getPostImageInfos());
     }
 
     public boolean isSameMember(final Member member) {
@@ -179,7 +177,7 @@ public boolean isSameMember(final Long memberId) {
     }
 
     public Member getMember() {
-        return member;
+        return this.member;
     }
 
     public Long getWriterId() {
@@ -212,8 +210,6 @@ public void updateView(final Member member) {
     }
 
     public void delete() {
-        //lazyLoading 문제로 repository를 통해 직접 postImageInfos를 제거해주는 것이 필요하다.
-        this.postImageInfos.deleteAll();
         this.deleted = true;
     }