fix: 필터링 로직 contains->Pattern으로 로직 전면 수정 #258
Conversation
Summary of ChangesHello @JjungminLee, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! 이 PR은 Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. 정규식의 힘, 메뉴 필터 정교하게, 리뷰는 깔끔히. Footnotes
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
| boolean allExcluded = true; | ||
| for (String item : items) { | ||
| String trimmed = item.trim(); | ||
| if (trimmed.isBlank()) continue; | ||
|
|
||
| if (!isExcludedSingle(trimmed)) { | ||
| allExcluded = false; | ||
| break; | ||
| } | ||
| } | ||
|
|
||
| if (menuName.contains("밥") | ||
| && !menuName.contains("자장밥") | ||
| && !menuName.contains("볶음밥") | ||
| && !menuName.contains("비빔밥")) { | ||
| return true; | ||
| } | ||
| return allExcluded; |
There was a problem hiding this comment.
이 로직은 Java Stream API를 사용하면 더 간결하고 선언적으로 표현할 수 있습니다. for 루프 대신 allMatch를 사용하면 모든 항목이 제외 대상인지 확인하는 의도를 더 명확하게 드러낼 수 있습니다. 코드 상단에 import java.util.Arrays;를 추가해야 합니다.
return Arrays.stream(items)
.map(String::trim)
.filter(s -> !s.isBlank())
.allMatch(MenuFilterUtil::isExcludedSingle);| for (String keyword : EXCLUDED_KEYWORDS) { | ||
| if (name.equals(keyword)) return true; | ||
| } |
There was a problem hiding this comment.
EXCLUDED_KEYWORDS는 Set 자료구조이므로, for 루프를 통해 모든 원소를 순회하는 것보다 contains() 메소드를 사용하는 것이 더 효율적입니다. Set.contains()의 시간 복잡도는 평균적으로 O(1)인 반면, 루프는 O(n)입니다.
if (EXCLUDED_KEYWORDS.contains(name)) return true;| if (SINGLE_KIMCHI_PATTERN.matcher(name).find()) return true; | ||
| if (SINGLE_BAP_PATTERN.matcher(name).find()) return true; |
There was a problem hiding this comment.
isExcludedFromReview 메소드에서 menuName을 split하고 trim한 결과가 name으로 전달되므로, name은 앞뒤 공백이 없는 단일 단어입니다. 따라서 (^|\s)김치($|\s)와 같은 정규식 패턴은 name.equals("김치")와 동일하게 동작합니다. 불필요한 정규식 사용은 성능 저하를 유발할 수 있으므로, 간단한 equals 비교로 변경하는 것이 좋습니다. 이 변경과 함께 SINGLE_KIMCHI_PATTERN과 SINGLE_BAP_PATTERN 필드는 제거할 수 있습니다.
if (name.equals("김치")) return true;
if (name.equals("밥")) return true;
#️⃣ Issue Number
📝 요약(Summary)
💬 공유사항 to 리뷰어
✅ PR Checklist
PR이 다음 요구 사항을 충족하는지 확인하세요.