RYC0208
commented
1 month ago 튜터님!! 이번에도 피드백 감사합니다!!! 이벤트 버블링에 대한 참고 문서까지 주셔서 너무 감사드려요 피드백 달아주신 것을 토대로 리팩토링 해보겠습니다!! 감사합니다
Open CaesiumY opened 1 month ago
RYC0208
commented
1 month ago 튜터님!! 이번에도 피드백 감사합니다!!! 이벤트 버블링에 대한 참고 문서까지 주셔서 너무 감사드려요 피드백 달아주신 것을 토대로 리팩토링 해보겠습니다!! 감사합니다
개요
용철님 안녕하세요! 처음에 제공해주신 링크를 보고 놀랐어요. 구현이 하나도 안 되어있어서요ㅎㅎ 혹시나 해서 브랜치를 보니 브랜치에 잘 작성해두셨더라구요. 다음부터는 최종본을 main 브랜치에 머지해주세요!
고치면 좋을 점
루트 폴더의
package.jsonREADME.md와Pokemon-app폴더 말고도package.json과package-lock.json파일이 존재하고 있어요. 아마 잘못 생성하신 듯한데, 이러한 사용하지 않는 파일은 삭제해주세요!Styled-components로 통합하기
https://github.com/RYC0208/Pokemon-Dictionary/blob/bd6d46bd213a250cd3470e84972647049c09c299/Pokemon-app/src/App.jsx#L11
Link컴포넌트https://github.com/RYC0208/Pokemon-Dictionary/blob/bd6d46bd213a250cd3470e84972647049c09c299/Pokemon-app/src/pages/Home.jsx#L16
navigate를 사용했지만, 사실 이러한 부분에서는Link컴포넌트를 사용하는 게 더 좋아요!Link컴포넌트,useNavigate사이의 차이점 및 사용해야 하는 상황에 대해서는 강의 자료 링크를 남겨드릴게요!좋은 점
애니메이션 사용
https://github.com/RYC0208/Pokemon-Dictionary/blob/bd6d46bd213a250cd3470e84972647049c09c299/Pokemon-app/src/pages/Dex.jsx#L19-L33
포켓볼 채우기
https://github.com/RYC0208/Pokemon-Dictionary/blob/bd6d46bd213a250cd3470e84972647049c09c299/Pokemon-app/src/contexts/PokemonContext.jsx#L15-L18
selectedPokemon이 바뀌었을 때만 실행되도록useEffect를 사용하는 것도 추천 드립니다!이미지 대체 텍스트
https://github.com/RYC0208/Pokemon-Dictionary/blob/bd6d46bd213a250cd3470e84972647049c09c299/Pokemon-app/src/components/PokemonCard.jsx#L11
alt)를 제공할 수 있어요. 선택 옵션이긴 하지만 사실상 필수입니다.이벤트 버블링
https://github.com/RYC0208/Pokemon-Dictionary/blob/bd6d46bd213a250cd3470e84972647049c09c299/Pokemon-app/src/components/PokemonCard.jsx#L17-L23
e.stopPropagation()은 아마도PokemonContainer에 연결된 클릭 이벤트의 동작을 방지하려고 넣어주신 거 같아요.마무리
특별히 문제점이 없는 아주 깔끔하고 완성도 높은 코드였습니다! 덕분에 고치면 좋을 점을 찾기가 힘들었네요ㅎㅎ 오히려 칭찬할 점을 찾기가 쉬웠어요. 예를 들면
useContext도 커스텀 훅으로 만들어서 사용했다든지요! 이렇게 제 기대치를 올려두셨으면 각오를 해야겠죠?! 다음에도 완성도 높은 코드를 기대하겠습니다:) 이번 과제 진행하시느라 수고 많으셨어요!!