Merge pull request #20 from TeamUStory/fix/paper-count

fix:paperCount, 동시성 문제 해결
TeamUStory · Sep 12, 2024 · cd86ccf · cd86ccf
2 parents 6368331 + 2f6a4d4
commit cd86ccf
Showing 1 changed file with 7 additions and 5 deletions.
diff --git a/src/main/java/com/elice/ustory/domain/paper/service/PaperService.java b/src/main/java/com/elice/ustory/domain/paper/service/PaperService.java
@@ -222,14 +222,16 @@ public Paper validatePaper(Long paperId) {
         return paper;
     }
 
-    public Integer countPapersByWriterId(Long userId) {
+    public int countPapersByWriterId(Long userId) {
         List<Paper> findByWriterId = paperRepository.findByWriterId(userId);
-        for(Paper paper : findByWriterId) {
-            if (paper.isDeleted()) {
-                findByWriterId.remove(paper);
+        int count = 0;
+
+        for (Paper paper : findByWriterId) {
+            if (!paper.isDeleted()) {
+                count++;
             }
         }
-        return findByWriterId.size();
+        return count;
     }
 
     // 작성자를 제외한 멤버들에게 코멘트를 달아달라고 알림 전송, 트랜잭션?