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
eunddodi
requested review from
Dongkyuuuu,
hyesungoh,
oyeon-kwon,
sensecodevalue,
Seojunhwan,
sjoleee and
WooWan
as code owners
hyesungoh
approved these changes
Dec 19, 2023
Comment on lines
+6
to
+9
| - 게으른 초기화 | ||
| - **'초깃값은 최초에만 사용된다.' 196p → 그럼 자바스크립트 연산도 한 번만 실행되는 거 아니야?** | ||
| - global.states에는 연산 결과가 저장될 거 아니야. | ||
| - initialState가 인수로 전달이 되어야 하니까 그때 자바스크립트 연산이 이뤄지나보다. |
There was a problem hiding this comment.
function 부모() {
const [a, setA] = useState(0)
return <자식 a={a} />
}
function 자식(a) {
const [b, setB] = useState(a)
return <>{b}</>
}요련 경우에서 부모가 a를 바꾸었을 때를 말씀하시는 것인지???
a를 바꾸었을 때 b가 바뀌는지??
Comment on lines
+38
to
+39
| - ### useRef | ||
| - 🤔 다들 useRef를 어떤 경우에 주로 사용하시나요 ? |
There was a problem hiding this comment.
dom에 접근해야하는 경우가 1순위인거 같고
책 내용처럼 값을 저장해야하는 경우 정도일 거 같아유
There was a problem hiding this comment.
저도 혜성님과 비슷합니다! setTimeout의 타이머 저장할때..?
Dongkyuuuu
approved these changes
Dec 19, 2023
| - state의 race condition을 야기할 수 있다. | ||
| - [What are good alternatives to data fetching in Effects?](https://react.dev/reference/react/useEffect#what-are-good-alternatives-to-data-fetching-in-effects) | ||
| - ### useMemo | ||
| - **렌더링이 발생할 때** 의존성 배열 값의 변화 여부를 검사 |
There was a problem hiding this comment.
"렌더링이 발생할 때"가 트리거였군요 놓쳤던 부분이였네요...
Comment on lines
+38
to
+39
| - ### useRef | ||
| - 🤔 다들 useRef를 어떤 경우에 주로 사용하시나요 ? |
There was a problem hiding this comment.
저도 혜성님과 비슷합니다! setTimeout의 타이머 저장할때..?
| - HOC의 이름은 with로 시작해야 한다 | ||
| - 부수효과를 최소화해야 한다 | ||
| - 대표적으로 인수로 받은 컴포넌트의 props를 임의로 수정, 추가, 삭제해서는 안된다. 수정할 경우 HOC를 이용하는 입장에서 props가 변경될지도 모른다는 사실을 계속 염두에 둬야 하는 부담감이 생긴다. | ||
| - [Higher Order Components는 여전히 유용하다](https://blog.hwahae.co.kr/all/tech/11631) |
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.
리뷰하기...메모