Skip to content

Conversation

@yongaricode
Copy link
Contributor

🔥 Issues

#33

✅ What to do

  • 검색어 저장 API 호출 제거
  • 북스냅 검색 결과 헤더 UI 수정
  • 북스냅 스크롤 튕기는 오류 수정
  • 북스냅 최근 검색어 누르면 검색창에 반영

📄 Description

  • 검색어 저장 API가 swagger에 있어서 호출햇는데 알고보니 백엔드 확인용이었고, 검색만 하면 자동으로 검색어는 저장되는 것이었어서 호출을 제거
  • 북스냅에서 검색했을 때, 기존에는 검색어가 상단에 뜨지 않아 검색어가 뜨게 헤더를 수정
  • 북스냅에서 스크롤이 바닥에 닿으면 다음 페이지를 가져오는데 이때, 스크롤이 최상단으로 가는 오류 수정
  • 북스냅 검색에서 최근 검색어 누르면 검색창에 반영

🤔 Considerations

최근검색어 눌러도 헤더에 반영되지 않는 문제

  • 원인은 BookSearchHeader 내부에서 query prop을 받아 초기 searchWord 상태로만 설정하고, 이후에는 갱신하지 않아 상태가 고정되는 구조였기 때문
  • 이를 해결하기 위해 useEffect를 통해 query 값이 변경될 때마다 searchWord 상태가 동기화되도록 수정

스크롤 바닥 시 최상단으로 튀는 문제

  • 검색 결과를 무한 스크롤로 구현했는데, 스크롤이 바닥에 닿아 다음 페이지 데이터를 가져오는 순간 스크롤이 갑자기 최상단으로 튀는 현상이 발생
  • 초기에는 setReview([]) 등의 렌더 트리 변경이 원인이라 추정했으나, 실제 원인은 getReviews()가 중복 호출되며 불필요한 렌더링이 발생한 것
  • isLoading 상태를 활용해 중복 호출을 방지하고, 스크롤 이벤트가 안정적으로 동작하도록 수정

🛠️ Improvements to Make

개선할 사항

👀 References

image

@yongaricode yongaricode added ✨Feat 새로운 기능 개발 🛠️REFACTOR 코드 구조 리팩토링 ✔️Fix 에러 수정 labels May 14, 2025
@yongaricode yongaricode merged commit b9a5414 into develop May 14, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨Feat 새로운 기능 개발 ✔️Fix 에러 수정 🛠️REFACTOR 코드 구조 리팩토링

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant