-
Notifications
You must be signed in to change notification settings - Fork 56
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
[1팀 권준우] [Chapter 2-2] 디자인 패턴과 함수형 프로그래밍 #22
Open
CreatyJohnKwon
wants to merge
21
commits into
hanghae-plus:main
Choose a base branch
from
CreatyJohnKwon:main
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- ./models to ./utils
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
과제 체크포인트
기본과제
React의 hook 이해하기
함수형 프로그래밍에 대한 이해
Component에서 비즈니스 로직을 분리하기
비즈니스 로직에서 특정 엔티티만 다루는 계산을 분리하기
Component에서 사용되는 Data가 아닌 로직들은 hook으로 옮겨졌나요?
주어진 hook의 책임에 맞도록 코드가 분리가 되었나요?
계산함수는 순수함수로 작성이 되었나요?
심화과제
뷰데이터와 엔티티데이터의 분리에 대한 이해
엔티티 -> 리파지토리 -> 유즈케이스 -> UI 계층에 대한 이해
Component에서 사용되는 Data가 아닌 로직들은 hook으로 옮겨졌나요?
주어진 hook의 책임에 맞도록 코드가 분리가 되었나요?
계산함수는 순수함수로 작성이 되었나요?
특정 Entitiy만 다루는 함수는 분리되어 있나요?
특정 Entitiy만 다루는 Component와 UI를 다루는 Component는 분리되어 있나요?
데이터 흐름에 맞는 계층구조를 이루고 의존성이 맞게 작성이 되었나요?
과제 셀프회고
과제를 시작하면서
발제로 알 수 있는 이번 과제의 목표는
여기서 React 의 주요 역할계층은 세 가지가 있다
각각 계층은 무슨 역할을 하는지 자세히 알아보자
합성 컴포넌트를 적절히 수행해 책임을 나눈다
최종 설계 단계
calculateDiscount
등은 특정 UI와 무관하게 독립적으로 동작하도록 설계한다위 설계로 얻는 리팩토링 결과의 장점은
과제에서 좋았던 부분
리팩토링 설계를 짜긴 했지만 사실 방향적으로만 그렇고, 구현은 중요 계층 3단계에 따라 코드를 분리 하려고 노력했다
가장 크게 느꼈던 부분은 계층 별 역할이 한 눈에 보이는 프로젝트로 완성 되었다
과제 막바지때는 모든 계층에서 각 장점들을 느낄 수 있었는데
리팩토링하며 느낀 경험으로 맞이할 프로젝트에 명확한 구조를 빚거나 효율적인 협업이 될 것 같다
과제를 풀며 겪은 문제점과 알게된 점
A 컴포넌트의 자식 컴포넌트인 B와 C에서 같은 커스텀 훅을 사용하려고 할 때,
useState
의setter
와getter
를 각각 참조하려는 상황이 있었다이 과정에서, A 컴포넌트에서 선언한 커스텀 훅을 B와 C에 Props로 전달해야 한다는 점이 번거롭고 비효율적이라고 느꼈다
또한
props drilling
이슈로 퍼질 수도 있는 문제이기도 하다문제의 해결책을 고민해보고 나눈 결과,
커스텀 훅을 전역으로 관리하면 Props 전달의 복잡성을 줄일 수 있다
Context API
나Zustand
와 같은 전역 상태 관리 도구를 사용하면,커스텀 훅의 상태를 어디서든 선언적으로 사용할 수 있다는 점을 알게 되었다
전역으로 선언된 커스텀 훅은 Props를 통해 하위 컴포넌트로 전달하지 않아도 접근 가능하므로, 코드가 더 간결해진다
지식을 공유해주신 12/13/15팀 감사해요!
리뷰 받고 싶은 내용 (번외의 질문)
많은 분들에게 들어보고 싶은 번외의 질문입니다..!!
바로 "건강!"인데요,
항해를 시작하면서 밤을 꼴딱 새는 일이 많은데 정말 다음날에 숙취처럼 지장이 많이 가더라구요 집중도 안되고!!
가만히 앉아서 코드를 타이핑하는 것에 많은 시간을 할애 하는 것에 대해 어떻게 생각하시나요?