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

[6팀 안기환][Chapter 1-3] React, Beyond the Basics #41

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

ahngh0113
Copy link

@ahngh0113 ahngh0113 commented Oct 10, 2024

과제 체크포인트

기본과제

  • shallowEquals 구현 완료
  • deepEquals 구현 완료
  • memo 구현 완료
  • deepMemo 구현 완료
  • useRef 구현 완료
  • useMemo 구현 완료
  • useDeepMemo 구현 완료
  • useCallback 구현 완료

심화 과제

  • 기본과제에서 작성한 hook을 이용하여 렌더링 최적화를 진행하였다.
  • Context 코드를 개선하여 렌더링을 최소화하였다.

리뷰 받고 싶은 내용

  • Provider의 구성 및 구조의 적절성
  • 컴포넌트 분리(component, layout)의 적절성

궁금한 점

(이전에 vue를 주로 사용하고, react는 찍먹만 해봤습니다.)

  • Provider 파일들

    • context를 많이 안써봐서 잘 모르는데 XXXProvider에서 context랑 provider를 한 파일에 구성했는데 괜찮을까요?
    • 또한 모든 Provider를 모은 providers/index.tsx가 있는데 이렇게 사용해도 될까요?(이런 구성이나, 이런 네이밍...)
  • 리액트에서는 얕은 비교만 하는거 같은데 그 이유가 성능 문제 때문일까요?

    • 깊어지면 그만큼 재귀로 비교를 해야하니 성능에 악영향이 미칠거 같다는 생각이 들었습니다.

과제를 수행하면서 느낀점

과제 시작 전 생각

하...할 수 있을까?ㅠ
그래도 재...밌겠다..!

과제 제출 후 생각

지피티를 잘 활용하자..!
아직 갈길이 멀다.. 더 화이팅 하자!

기타

과제 난이도

3/5
(사실 이것도 어려웠는데 이전 과제와 비교하여 측정했습니다.)

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.

1 participant