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

검색 키워드 강조 로직 개선 작업 #318

Merged
merged 1 commit into from
Dec 14, 2022

Conversation

doputer
Copy link
Member

@doputer doputer commented Dec 13, 2022

개요

검색 키워드 강조 로직을 개선했습니다.

변경 사항

  • 검색 키워드 강조 함수 분리
  • 검색 키워드 강조 로직 개선

참고 사항

  • 검색 키워드 강조 로직을 개선하면서 고려한 사항은 다음과 같습니다.
  • 글 검색 결과와 책 검색 결과에 공통으로 사용되는 로직이기 때문에 중복을 줄이고자 유틸리티 함수로 분리했습니다.
  • 함수가 단일 책임을 갖도록 내부 로직을 분리했습니다.
  • Map 자료 구조를 활용해서 과도하게 사용되는 고차 함수를 줄였습니다.
  • 재귀적으로 동작하는 강조 함수가 Maximum call stack size exceeded 오류가 발생하지 않도록 최대 200자 내에서 처리하도록 개선했습니다.
  • 더 이상 강조할 수 없는 키워드(= 문자열에 더 이상 등장하지 않는 키워드, validKeywords 참고)는 연산에 사용되지 않도록 필터링했습니다.
  • lastIndexOf 등을 활용해서 성능이 좋지 않은 정규식 탐색을 제거했습니다.

@doputer doputer added FE Frontend IMPROVE Improve Code labels Dec 13, 2022
@doputer doputer self-assigned this Dec 13, 2022
@github-actions
Copy link

⚡️ Lighthouse report!

Category Score
🟢 Performance 91
🟢 Accessibility 97
🟢 Best Practices 92
🟢 SEO 100

Copy link
Collaborator

@dahyeon405 dahyeon405 left a comment

Choose a reason for hiding this comment

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

utils로 분리되니 더 깔끔하네요! 고생하셨습니다 👍

@doputer doputer merged commit 18e8321 into develop Dec 14, 2022
@doputer doputer deleted the feature/search_improve_kdh branch December 14, 2022 05:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FE Frontend IMPROVE Improve Code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants