From 455f954fbde0dc4175833dedf42cfa1455eb481a Mon Sep 17 00:00:00 2001
From: RyuKwanKon <rhks33rhs@gmail.com>
Date: Sun, 14 Jan 2024 01:47:00 +0900
Subject: [PATCH] =?UTF-8?q?[fix]=20#43=20=EC=9D=BC=EC=B9=98=EC=9C=A8=20?=
 =?UTF-8?q?=EA=B3=84=EC=82=B0=EC=9D=84=20=EC=9C=84=ED=95=9C=20service=20ut?=
 =?UTF-8?q?il=20=EA=B0=9D=EC=B2=B4=20=EC=A0=81=EC=9A=A9?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../domain/post/service/PostService.java      | 27 +++++--------------
 .../domain/scrap/service/ScrapService.java    | 17 +-----------
 2 files changed, 8 insertions(+), 36 deletions(-)

diff --git a/src/main/java/org/gachon/checkmate/domain/post/service/PostService.java b/src/main/java/org/gachon/checkmate/domain/post/service/PostService.java
index a6a8933..b6b132c 100644
--- a/src/main/java/org/gachon/checkmate/domain/post/service/PostService.java
+++ b/src/main/java/org/gachon/checkmate/domain/post/service/PostService.java
@@ -19,7 +19,6 @@
 import org.gachon.checkmate.domain.post.dto.support.PostSearchDto;
 import org.gachon.checkmate.domain.post.entity.Post;
 import org.gachon.checkmate.domain.post.repository.PostRepository;
-import org.gachon.checkmate.domain.post.utils.PostSortType;
 import org.gachon.checkmate.domain.post.utils.PostSortingUtils;
 import org.gachon.checkmate.domain.scrap.repository.ScrapRepository;
 import org.gachon.checkmate.global.error.exception.EntityNotFoundException;
@@ -32,10 +31,10 @@
 
 import java.time.LocalDate;
 import java.time.temporal.ChronoUnit;
-import java.util.Comparator;
 import java.util.List;
 import java.util.stream.Collectors;
 
+import static org.gachon.checkmate.domain.checkList.utils.MatchRateCalculator.getAccuracy;
 import static org.gachon.checkmate.global.error.ErrorCode.*;
 import static org.gachon.checkmate.global.utils.PagingUtils.convertPaging;
 
@@ -110,12 +109,6 @@ public PostStateUpdateResponseDto updateMyPostState(Long userId, Long postId, Po
         return PostStateUpdateResponseDto.of(post);
     }
 
-    private void validatePostWriter(User user, Post post) {
-        if (!post.getUser().getId().equals(user.getId())) {
-            throw new ForbiddenException(NOT_POST_WRITER);
-        }
-    }
-
     private List<PostSearchElementResponseDto> createPostSearchResponseDto(Page<PostSearchDto> postSearchDtoList, CheckList checkList) {
         return postSearchDtoList.stream()
                 .map(postSearchDto ->
@@ -126,22 +119,16 @@ private List<PostSearchElementResponseDto> createPostSearchResponseDto(Page<Post
                 .collect(Collectors.toList());
     }
 
-    private int getAccuracy(PostCheckList postCheckList, CheckList checkList) {
-        int count = 0;
-        count += postCheckList.getCleanType().compareRateTo(checkList.getCleanType());
-        count += postCheckList.getDrinkType().compareRateTo(checkList.getDrinkType());
-        count += postCheckList.getHomeType().compareRateTo(checkList.getHomeType());
-        count += postCheckList.getLifePatternType().compareRateTo(checkList.getLifePatternType());
-        count += postCheckList.getNoiseType().compareRateTo(checkList.getNoiseType());
-        count += postCheckList.getSleepType().compareRateTo(checkList.getSleepType());
-        count += postCheckList.getSmokeType().compareRateTo(checkList.getSmokeType());
-        return (int) (count / 7) * 100;
-    }
-
     private int getRemainDate(LocalDate endDate) {
         return (int) LocalDate.now().until(endDate, ChronoUnit.DAYS);
     }
 
+    private void validatePostWriter(User user, Post post) {
+        if (!post.getUser().getId().equals(user.getId())) {
+            throw new ForbiddenException(NOT_POST_WRITER);
+        }
+    }
+
     private void validateDuplicateTitle(String title) {
         if (postRepository.existsByTitle(title))
             throw new InvalidValueException(INVALID_POST_TITLE);
diff --git a/src/main/java/org/gachon/checkmate/domain/scrap/service/ScrapService.java b/src/main/java/org/gachon/checkmate/domain/scrap/service/ScrapService.java
index 7cb2339..b520a9c 100644
--- a/src/main/java/org/gachon/checkmate/domain/scrap/service/ScrapService.java
+++ b/src/main/java/org/gachon/checkmate/domain/scrap/service/ScrapService.java
@@ -2,7 +2,6 @@
 
 import lombok.RequiredArgsConstructor;
 import org.gachon.checkmate.domain.checkList.entity.CheckList;
-import org.gachon.checkmate.domain.checkList.entity.PostCheckList;
 import org.gachon.checkmate.domain.checkList.repository.CheckListRepository;
 import org.gachon.checkmate.domain.member.entity.User;
 import org.gachon.checkmate.domain.member.repository.UserRepository;
@@ -26,6 +25,7 @@
 import java.util.List;
 import java.util.stream.Collectors;
 
+import static org.gachon.checkmate.domain.checkList.utils.MatchRateCalculator.getAccuracy;
 import static org.gachon.checkmate.global.error.ErrorCode.*;
 
 @RequiredArgsConstructor
@@ -65,21 +65,6 @@ private List<PostSearchElementResponseDto> createPostSearchResponseDto(Page<Post
                 .collect(Collectors.toList());
     }
 
-    private int getAccuracy(PostCheckList postCheckList, CheckList checkList) {
-        int count = 0;
-        count += getRateForFrequencyElement(postCheckList.getCleanType().getCode(), checkList.getCleanType().getCode(), 4);
-        count += getRateForFrequencyElement(postCheckList.getDrinkType().getCode(), checkList.getDrinkType().getCode(), 3);
-        count += getRateForFrequencyElement(postCheckList.getHomeType().getCode(), checkList.getHomeType().getCode(), 3);
-        count = postCheckList.getLifePatternType().equals(checkList.getLifePatternType()) ? count + 1 : count;
-        count = postCheckList.getNoiseType().equals(checkList.getNoiseType()) ? count + 1 : count;
-        count = postCheckList.getSleepType().equals(checkList.getSleepType()) ? count + 1 : count;
-        return (int) (count / 6) * 100;
-    }
-
-    private int getRateForFrequencyElement(String firstEnumCode, String secondEnumCode, int size) {
-        return 1 - Math.abs(Integer.parseInt(firstEnumCode) - Integer.parseInt(secondEnumCode)) / size;
-    }
-
     private int getRemainDate(LocalDate endDate) {
         return (int) LocalDate.now().until(endDate, ChronoUnit.DAYS);
     }