-
Notifications
You must be signed in to change notification settings - Fork 46
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
[강수민] Sprint5 #161
The head ref may contain hidden characters: "React-\uAC15\uC218\uBBFC-sprint5"
[강수민] Sprint5 #161
Conversation
…ithub-actions [Fix] delete merged branch github action
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
첫 리액트 미션인데 너무 잘 구현해주셨습니다!
useEffect 의 의존성만 조금 개선해주시면 더욱 좋은 코드가 될 것 같네요 👍
* @param {*} totalCount 전체 Product 수 | ||
* @param {*} perPage 화면에 보여질 Product 수 (반응형에 사용) | ||
*/ | ||
const countTotalPage = useCallback( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이 함수는 이름만 보면 totalPage 수를 계산하는 역할만 할 것 같지만, 실제로는 현재 페이지를 setState 하는 로직까지 들어있네요.
그리고 위에 언급드린 바와 같이 totalPage 는 state 로 관리하지 않아도 될 것 같아요.
결국 이 함수에서 유효한 로직은 if (currentPage > totalPageCount) setCurrentPage(totalPageCount);
이 한줄인데 이건 Paginations.jsx 컴포넌트에서 관리해도 괜찮을 로직인 것 같습니다.
판다마켓
판다마켓
요구사항
기본
중고마켓
중고마켓 반응형
베스트 상품
전체 상품
심화
주요 변경사항
스크린샷
멘토에게