src/main/java/balancetalk/file/domain/repository/FileRepositoryCustom.java (1)

Line range hint 8-13: 메서드 통합 검토 제안

현재 비슷한 목적을 가진 세 개의 메서드가 존재합니다:

• findImgUrlsByResourceIdAndFileType
• findIdsByResourceIdAndFileType
• findAllByResourceIdAndFileType

각각의 메서드가 File 엔티티의 특정 필드만 조회하는 것으로 보입니다. 성능 최적화가 특별히 필요한 경우가 아니라면, 하나의 메서드로 통합하고 필요한 정보만 매핑하여 사용하는 것을 고려해보시기 바랍니다.

src/main/java/balancetalk/file/domain/repository/FileRepositoryImpl.java (1)

35-41: 구현이 올바르지만 코드 재사용성을 개선할 수 있습니다.

새로운 메서드의 구현이 적절하며 기존 패턴을 잘 따르고 있습니다. 하지만 다른 두 메서드(findImgUrlsByResourceIdAndFileType, findIdsByResourceIdAndFileType)가 이 메서드를 기반으로 리팩토링될 수 있을 것 같습니다.

다음과 같이 리팩토링하는 것을 고려해보세요:

+    @Override
+    public List<File> findAllByResourceIdAndFileType(Long resourceId, FileType fileType) {
+        return queryFactory.select(file)
+                .from(file)
+                .where(file.fileType.eq(fileType), file.resourceId.eq(resourceId))
+                .fetch();
+    }
+
     @Override
     public List<String> findImgUrlsByResourceIdAndFileType(Long resourceId, FileType fileType) {
-        List<File> images = queryFactory.selectFrom(file)
-                .where(file.fileType.eq(fileType), file.resourceId.eq(resourceId))
-                .fetch();
+        List<File> images = findAllByResourceIdAndFileType(resourceId, fileType);
         return images.stream()
                 .map(File::getS3Url)
                 .toList();
     }

     @Override
     public List<Long> findIdsByResourceIdAndFileType(Long resourceId, FileType fileType) {
-        return queryFactory.select(file.id)
-                .from(file)
-                .where(file.fileType.eq(fileType), file.resourceId.eq(resourceId))
-                .fetch();
+        return findAllByResourceIdAndFileType(resourceId, fileType).stream()
+                .map(File::getId)
+                .toList();
     }

src/main/java/balancetalk/file/domain/FileHandler.java (1)

63-68: 성능 최적화를 위한 제안

대량의 파일을 처리할 때 성능 향상을 위해 병렬 처리를 고려해보시는 것은 어떨까요?

다음과 같은 구현을 제안드립니다:

 @Transactional
 public void deleteFiles(List<File> files) {
+    if (files.isEmpty()) {
+        return;
+    }
+
+    // S3 파일 삭제를 병렬로 처리
+    files.parallelStream().forEach(file -> {
         try {
-            s3Operations.deleteObject(bucket, file.getS3Key());
+            s3Operations.deleteObject(bucket, file.getS3Key());
         } catch (Exception e) {
+            throw new FileDeleteException("S3 파일 삭제 중 오류 발생: " + file.getS3Key(), e);
         }
-    }
+    });
+
     fileRepository.deleteAll(files);
 }

src/main/java/balancetalk/talkpick/application/TalkPickService.java (1)

101-106: 로깅 추가 제안

파일 삭제 작업의 모니터링과 디버깅을 위해 로깅을 추가하는 것이 좋습니다. 특히 파일 삭제 시작/완료 시점과 삭제된 파일의 수를 기록하면 유용할 것 같습니다.

private void deleteFiles(Long talkPickId) {
+   log.debug("Starting file deletion for talkPickId: {}", talkPickId);
    List<File> files = fileRepository.findAllByResourceIdAndFileType(talkPickId, TALK_PICK);
    if (files.isEmpty()) {
+       log.debug("No files found for talkPickId: {}", talkPickId);
        return;
    }
+   log.info("Deleting {} files for talkPickId: {}", files.size(), talkPickId);
    fileHandler.deleteFiles(files);
+   log.debug("File deletion completed for talkPickId: {}", talkPickId);
}

Configuration used: CodeRabbit UI
Review profile: CHILL

#!/bin/bash
# Description: FileRepositoryCustom 인터페이스 선언과 사용처 확인

# 인터페이스 선언 확인
echo "Checking interface declaration..."
rg -A 2 "findAllByResourceIdAndFileType.*FileType.*\)" --type java

# 메서드 사용처 확인
echo "Checking usage..."
rg "findAllByResourceIdAndFileType\(" --type java