Skip to content

Conversation

@part3-5
Copy link
Collaborator

@part3-5 part3-5 commented Jan 4, 2025

🪐 작업 내용

  • ReviewInput.tsx

  • useDebounce 훅을 단순화하여 값 자체를 디바운스하도록 변경

  • 콜백 함수 대신 상태 값을 디바운스하는 방식으로 변경

  • useEffect를 사용하여 디바운스된 값에 대한 처리 구현

  • useCallback을 사용하여 handleContentChange 최적화

📝 간단하게 설명해주세요

후기 적기 전
image

200자 이상부터 maxLength prop으로 입력 제한을 처리
image

@part3-5 part3-5 changed the title 리뷰모달/후기상한선검사/파일명수정 Jan 4, 2025
Copy link
Member

@KimLuka KimLuka left a comment

Choose a reason for hiding this comment

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

기존에 onChange 로 감지하고 유효성 검사하던 것을 usedebounce 로 바꾼 게 맞을까요? 혹시 특별한 이유가 있으셨는지도 궁금합니다~

저는 onChange 가 더 적합하다고 보는데, 이유는 크게 아래 두가지입니다~

  1. 사용자 경험상 기다리지 않고 입력값에 대해 즉각적인 피드백(검사 결과)를 주는 것이 더 좋다고 생각
  2. usedebounce 는 과한 네트워크 요청 방지가 필요할 때 더 효과적일 거 같은데, 후기 글자 수 상한선은 api 요청과 무관

그리고 파일명 때문인지 모르겠지만 파일 두 개에서 충돌 일어나서, 확인 부탁드립니다~

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

리뷰모달/후기 상한선 유효성검사/하이드레이션에러수정

3 participants