Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[feat] 키워드에 따른 구역 추천 결과를 3개 반환하도록 수정한다. #81

Merged
merged 4 commits into from
Nov 23, 2024

Conversation

juuuunny
Copy link
Contributor

✅ PR 유형

어떤 변경 사항이 있었나요?

  • 새로운 기능 추가
  • 버그 수정
  • 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 문서 수정
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제

📝 작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요(이미지 첨부 가능)

  • 키워드에 따른 구역 추천 결과 반환 시 무조건 3개를 반환할 수 있도록 변경한다.
  • 각자 구장의 enum의 1번째, 2번째, 3번째가 가장 무난한 순서인 구장enum이므로
    키워드에 따른 필터링된 결과가 3개 미만일 경우에는 현재 속해 있지 않은 구역으로 3개를 채워 반환한다.
List<T> priorityDummyZones = Arrays.stream(zones)
                .filter(zone -> {
                    int ordinal = zone.ordinal();
                    return ordinal == 0 || ordinal == 1 || ordinal == 2;
                })
                .toList();

        for (T priorityDummyZone : priorityDummyZones) {
            if (filteredZones.size() >= 3) {
                break;
            }
            if (!filteredZones.contains(priorityDummyZone)) {
                filteredZones.add(priorityDummyZone);
            }
        }

        return filteredZones;

-> zone.ordinal을 통하여 enum의 index를 통하여 1번째, 2번째, 3번째를 찾을 수 있다!

✏️ 관련 이슈

본인이 작업한 내용이 어떤 Issue Number와 관련이 있는지만 작성해주세요


🎸 기타 사항 or 추가 코멘트

@juuuunny juuuunny self-assigned this Nov 23, 2024
@juuuunny juuuunny added 🚀 feat 새로운 기능 추가 / 일부 코드 추가 / 일부 코드 수정 (리팩토링과 구분) / 디자인 요소 수정 😆 JUNHYEONG 준형 Issue or PR labels Nov 23, 2024
Copy link
Member

@bbbang105 bbbang105 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생하셨습니다!! LGTM 👍🏻

@juuuunny juuuunny merged commit c949bc6 into develop Nov 23, 2024
1 check passed
@juuuunny juuuunny deleted the feature/#80/modify-recommend-zone-logic branch November 23, 2024 08:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🚀 feat 새로운 기능 추가 / 일부 코드 추가 / 일부 코드 수정 (리팩토링과 구분) / 디자인 요소 수정 😆 JUNHYEONG 준형 Issue or PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[feat] 구역추천 결과 로직을 수정한다.
2 participants