[refac] divided method responsibility

Team-Hankki · Jul 12, 2024 · c9db421 · c9db421
1 parent fd3ff54
commit c9db421
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 6 deletions.
diff --git a/src/main/java/org/hankki/hankkiserver/api/store/service/HeartCommandService.java b/src/main/java/org/hankki/hankkiserver/api/store/service/HeartCommandService.java
@@ -28,7 +28,7 @@ public HeartCreateResponse createHeart(final StorePostCommand storePostCommand)
         Long storeId = storePostCommand.storeId();
         validateCreateStoreHeart(userId, storeId);
         saveStoreHeart(userId, storeId);
-        updateStoreHeartCount(storeId, false);
+        increaseStoreHeartCount(storeId);
         return HeartCreateResponse.of(storeId, true);
     }
 
@@ -37,7 +37,7 @@ public HeartDeleteResponse deleteHeart(final StoreDeleteCommand storeDeleteComma
         Long storeId = storeDeleteCommand.storeId();
         validateDeleteStoreHeart(userId, storeId);
         heartDeleter.deleteHeart(userId,storeId);
-        updateStoreHeartCount(storeId, true);
+        decreaseStoreHeartCount(storeId);
         return HeartDeleteResponse.of(storeId, false);
     }
 
@@ -58,7 +58,11 @@ private void saveStoreHeart(final Long userId, final Long storeId) {
         heartUpdater.saveHeart(heart);
     }
 
-    private void updateStoreHeartCount(final Long storeId, final boolean isDeleted) {
-        storeFinder.getStore(storeId).updateHearCount(isDeleted);
+    private void increaseStoreHeartCount(final Long storeId) {
+        storeFinder.getStore(storeId).updateHearCount(false);
+    }
+
+    private void decreaseStoreHeartCount(final Long storeId) {
+        storeFinder.getStore(storeId).updateHearCount(true);
     }
 }
diff --git a/src/main/java/org/hankki/hankkiserver/domain/store/model/Store.java b/src/main/java/org/hankki/hankkiserver/domain/store/model/Store.java
@@ -39,9 +39,9 @@ public class Store extends BaseTimeEntity {
     public void updateHearCount(boolean isDeleted) {
         if (isDeleted) {
             heartCount--;
-        } else {
+        }
+        else {
             heartCount++;
         }
     }
-
 }