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

refactor: 검색 기능을 최적화한다 #928

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from
Open

Conversation

feb-dain
Copy link
Collaborator

@feb-dain feb-dain commented Nov 14, 2023

📄 Summary

지금은 같은 키워드일 경우에도 요청을 보내는데, 같은 키워드일 경우에는 요청을 보내지 않도록 개선했습니다.

🕰️ Actual Time of Completion

20분

🙋🏻 More

close #927

🚀 Storybook

https://storybook.carffe.in/

- 다른 코드들 처럼 hook끼리 모아두기로 결정

[#927]
@feb-dain feb-dain added 🛠️ 리팩터링 개선사항입니다 FE 프론트엔드 관련 이슈입니다 labels Nov 14, 2023
@feb-dain feb-dain self-assigned this Nov 14, 2023
return;
}

setUserSearchWord(encodedSearchWord);
Copy link
Member

@kyw0716 kyw0716 Nov 26, 2023

Choose a reason for hiding this comment

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

이전 검색시 사용했던 검색어를 변화하는 현재 검색어와 비교해 같은 검색어는 새로운 요청을 발생시키지 않도록 하기 위해서 새로운 상태를 두어 해당 값을 캐시로 활용하고 계신거 같네요! 제가 이해한 내용이 맞다면 추가적인 상태를 늘려 훅의 복잡도가 높아질 여지가 있진 않을까 하는 생각이 듭니다.

현재 코드를 보면 검색 결과에 대한 쿼리를 등록하고 있으므로 해당 쿼리키에 해당하는 캐시 값이 있는지 여부를 통해 이전에 검색했던 값인지를 알아올 수도 있을 것 같아요! 아래 링크에 있는 메서드(queryClient.getQueryData)를 활용해도 괜찮을 것 같습니다 👍

쿼리키를 통해 캐시값을 읽어오는 메서드 참고 문서

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FE 프론트엔드 관련 이슈입니다 🛠️ 리팩터링 개선사항입니다
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

검색 기능을 최적화한다
3 participants