[BE] 방 비교 결과 조회 API에 기능을 추가 구현한다. #215

tkdgur0906 · 2024-08-03T09:52:35Z

❗ Issue

[BE] 방 비교 결과 조회 API에 기능을 추가 구현한다. #197

✨ 구현한 기능

체크리스트 랭킹 계산 기능
질문 미 선택 시 null 처리
논리적 삭제 정책 반영

📢 논의하고 싶은 내용

미 선택 질문에 대한 답변을 전체 점수 계산에 반영할 지에 대한 여부

🎸 기타

...good/src/main/java/com/bang_ggood/checklist/dto/response/ChecklistWithScoreReadResponse.java

backend/bang-ggood/src/main/java/com/bang_ggood/checklist/repository/ChecklistRepository.java

backend/bang-ggood/src/main/java/com/bang_ggood/checklist/service/ChecklistService.java

JINU-CHANG

수고많으셨습니다 👏👏👏
궁금한부분 몇개 코멘트 남겨두었어요~

backend/bang-ggood/src/test/java/com/bang_ggood/checklist/service/ChecklistServiceTest.java

JINU-CHANG · 2024-08-04T10:51:20Z

backend/bang-ggood/src/main/java/com/bang_ggood/checklist/service/ChecklistService.java

+        List<Integer> ranks = ChecklistRank.calculateRanksByDescendingScores(scores);
+
+        for (int idx = 0; idx < checklistsWithScore.size(); idx++) {
+            ChecklistWithScoreReadResponse checklistWithScore = checklistsWithScore.get(idx);
+            checklistWithScore.assignRank(ranks.get(idx));
+        }


.sorted(Comparator.comparing(ChecklistWithScoreReadResponse::getScore).reversed())

이미 정렬이 되어 있으니 이 순서로 for문 돌면서 rank 부여하면 될 것 같은데 ChecklistRank.calculateRanksByDescendingScores(scores)의 역할을 잘 모르겠어요 🥲

전체적인 구현방식을 바꿔 인덱스 대신 객체에 직접 값을 할당하는 방법으루 수정하였습니다!

shin-jisong

간단하게 리뷰 남겼습니다! 수고하셨어요~

shin-jisong · 2024-08-05T05:41:49Z

backend/bang-ggood/src/main/java/com/bang_ggood/checklist/domain/Grade.java

@@ -19,6 +19,9 @@ public enum Grade {
    }

    public static Grade from(String grade) {
+        if(grade == null) {


공백이 제대로 들어가지 않은 것 같아요
정책상 수정될 코드 같기는 하네요~

정책에 맞게 수정해놨습니다!

shin-jisong · 2024-08-05T05:43:33Z

...good/src/main/java/com/bang_ggood/checklist/dto/response/ChecklistWithScoreReadResponse.java

-        List<CategoryScoreReadResponse> categories
-) {
-    public static ChecklistWithScoreReadResponse of(Checklist checklist, int checklistOptionCount, int checklistScore,
+public class ChecklistWithScoreReadResponse {


record에서 class로 바꾼 이유가 있으신가요?

객체 생성시 rank를 할당할 수가 없는데 Record를 사용할 경우 setter 사용이 불가했습니다.
rank 계산 후 값을 할당해주기 위해 class로 변경하였습니다!

shin-jisong · 2024-08-05T05:44:43Z

backend/bang-ggood/src/main/java/com/bang_ggood/checklist/repository/ChecklistRepository.java

@@ -25,7 +25,17 @@ default Checklist getById(long id) {

    List<Checklist> findByUser(User user);

-    List<Checklist> findByUserAndIdIn(User user, List<Long> checklistIds);
+    @Query("SELECT c FROM Checklist c "


쿼리를 따로 작성한 코드에 대해서는 테스트를 작성해 보아도 좋을 것 같습니다~

작성 완료!

shin-jisong · 2024-08-05T05:47:23Z

backend/bang-ggood/src/main/java/com/bang_ggood/checklist/service/ChecklistService.java

@@ -173,13 +174,13 @@ public SelectedChecklistResponse readChecklistById(long id) {

        List<SelectedOptionResponse> options = readOptionsByChecklistId(id);

-        List<SelectedCategoryQuestionsResponse> selectedCategoryQuestionsRespons =
+        List<SelectedCategoryQuestionsResponse> selectedCategoryQuestionsResponse =
                readCategoryQuestionsByChecklistId(id);

        int checklistScore = ChecklistScore.calculateTotalScore(checklist.getQuestions());


ChecklistScore를 호출하기 위해 체크리스트의 질문들을 꺼내서 넣는 게 어색하게 느껴져요!
checklist 내부에서 checklistScore를 갖고 있고 결과값을 넘기는 함수를 구현하는 게 어떨까요?
고려만 해 주세요 ㅎㅎ

해당 부분을 고민해봤는데 저희가 oneToMany를 제거하면 해당로직도 작동하지 않게 되어서,
아예 checklistQuestionRepository에서 질문들을 조회하는 방식으로 구현해야할 것 같습니다!

다만 방 체크리스트 조회 해당 pr이 열려있어서 여기에서 반영하도록 하겠습니다!

shin-jisong · 2024-08-05T05:51:08Z

backend/bang-ggood/src/main/java/com/bang_ggood/exception/ExceptionCode.java

@@ -6,6 +6,7 @@ public enum ExceptionCode {

    // 전체
    INVALID_PARAMETER(HttpStatus.BAD_REQUEST, "잘못된 인자입니다."),
+    SCORE_NOT_DESCENDING_SORTED(HttpStatus.BAD_REQUEST, "정렬되지 않은 점수입니다."),


에러코드 컨벤션에 맞게 수정해 보는 건 어떨까요?

Score 분류를 따로 만들어서 분류했습니다!

shin-jisong · 2024-08-05T05:51:36Z

backend/bang-ggood/src/test/java/com/bang_ggood/checklist/domain/ChecklistRankTest.java

+
+import static org.assertj.core.api.Assertions.assertThat;
+
+class ChecklistRankTest {


꼼꼼하게 테스트해 주셨네요 👍

…97-checklist-comparison-improvement # Conflicts: # backend/bang-ggood/src/main/java/com/bang_ggood/checklist/repository/ChecklistOptionRepository.java # backend/bang-ggood/src/main/java/com/bang_ggood/checklist/service/ChecklistService.java # backend/bang-ggood/src/test/java/com/bang_ggood/checklist/service/ChecklistServiceTest.java # backend/bang-ggood/src/test/java/com/bang_ggood/user/UserFixture.java

tkdgur0906 added 6 commits August 2, 2024 17:45

feat: 체크리스트 점수에 따라 등수 구하는 메서드 구현

f538567

feat: 방 정보 response 형식 변경

b36ff94

feat: 방 비교 결과 조회 기능 랭킹 구하도록 추가 구현

e2466d9

feat: Grade가 null일 경우 점수 계산 0점으로 처리하도록 구현

e581a50

feat: 방 비교 시 모든 카테고리에 대한 점수 반환하도록 기능 추가

1c7e10b

feat: 방 비교 시 필요한 db 조회 로직에 논리적 삭제 도입

51a8121

tkdgur0906 added 🔒 BE 😊 Feature labels Aug 3, 2024

tkdgur0906 added this to the 마일스톤 5주차 milestone Aug 3, 2024

tkdgur0906 requested review from shin-jisong, tsulocalize and JINU-CHANG August 3, 2024 09:52

tkdgur0906 self-assigned this Aug 3, 2024

style: 코드 포맷팅

0d15ffe

tsulocalize requested changes Aug 3, 2024

View reviewed changes

tkdgur0906 added 3 commits August 4, 2024 16:48

refactor: dto equals, hashcode, toString 삭제

9e3d1ea

style: 메서드 위치 getter를 맨 아래로 이동

3c4d63a

refactor: 방 비교하는 체크리스트가 유저의 것인지 검증하는 메서드 리팩토링

f247693

JINU-CHANG reviewed Aug 4, 2024

View reviewed changes

shin-jisong requested changes Aug 5, 2024

View reviewed changes

tkdgur0906 added 8 commits August 5, 2024 15:43

refactor: 방 비교 랭킹 구하는 메서드 리팩토링

0be2c6b

test: 체크리스트 비교 테스트에 값 테스트 추가

b18b3ae

feat: grade에 NONE 추가하여 null 처리 삭제

0ef6035

style: 에러 코드 컨벤션에 맞게 코드 정렬

7378345

refactor: 방 비교 메서드 검증 로직 기존 체크리스트 데이터로 수행하도록 변경

483c7cb

fix: 머지 충돌 해결

905a6cf

test: 직접 쿼리 작성한 findByUserAndIdIn db 메서드 테스트

7178c39

feat: 방 비교 결과 조회 서비스 로직 인자로 user 받도록 추가

8abc291

shin-jisong approved these changes Aug 5, 2024

View reviewed changes

tsulocalize approved these changes Aug 5, 2024

View reviewed changes

JINU-CHANG approved these changes Aug 5, 2024

View reviewed changes

JINU-CHANG merged commit 9bd7f5b into dev-be Aug 5, 2024
1 check passed

JINU-CHANG deleted the feat/197-checklist-comparison-improvement branch August 5, 2024 10:07

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[BE] 방 비교 결과 조회 API에 기능을 추가 구현한다. #215

[BE] 방 비교 결과 조회 API에 기능을 추가 구현한다. #215

tkdgur0906 commented Aug 3, 2024

JINU-CHANG left a comment

JINU-CHANG Aug 4, 2024

tkdgur0906 Aug 5, 2024

shin-jisong left a comment

shin-jisong Aug 5, 2024

tkdgur0906 Aug 5, 2024

shin-jisong Aug 5, 2024

tkdgur0906 Aug 5, 2024

shin-jisong Aug 5, 2024

tkdgur0906 Aug 5, 2024

shin-jisong Aug 5, 2024

tkdgur0906 Aug 5, 2024

shin-jisong Aug 5, 2024

tkdgur0906 Aug 5, 2024

shin-jisong Aug 5, 2024


		import static org.assertj.core.api.Assertions.assertThat;

		class ChecklistRankTest {

[BE] 방 비교 결과 조회 API에 기능을 추가 구현한다. #215

[BE] 방 비교 결과 조회 API에 기능을 추가 구현한다. #215

Conversation

tkdgur0906 commented Aug 3, 2024

❗ Issue

✨ 구현한 기능

📢 논의하고 싶은 내용

🎸 기타

JINU-CHANG left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

shin-jisong left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment