[백지연] 커스텀 훅 실습 #1
Open
Conversation
This file contains hidden or 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
| <h2>useSomething 실습</h2> | ||
| <p>{name}</p> | ||
| <button className="modern-btn" onClick={changeName}> | ||
| 누구게 |
There was a problem hiding this comment.
효과가 많아서 재미있는 것 같아요
이름이 뜬 후에는 버튼 이름을 누구게 말고 다른 걸로 바꾸면 좋을 것 같아요
krlyeon
approved these changes
Aug 5, 2025
wonpark1
reviewed
Aug 5, 2025
| setName((prev) => (prev === "아기사자" ? "아기사자 백지연" : "아기사자")); | ||
| }; | ||
|
|
||
| useEffect(() => { |
There was a problem hiding this comment.
name 값이 바뀔 때마다 useEffect로 confetti가 실행되게 해둔 점이 좋네용!! 그리고 name변경과 confetti 실행 코드가 분리되어있어서 깔끔합니다
Soi-coding
approved these changes
Aug 27, 2025
There was a problem hiding this comment.
단순히 name만 바뀌는 게 아니라 fire2()라는 효과도 같이 줘서 좋은 것 같아요!
leeboyeon17
reviewed
Aug 28, 2025
| const { fire2 } = useConfetti(); | ||
|
|
||
| const changeName = () => { | ||
| setName((prev) => (prev === "아기사자" ? "아기사자 백지연" : "아기사자")); |
There was a problem hiding this comment.
상태 업데이트를 prev 기반으로 처리해서 최신 상태를 안전하게 반영시킨 점이 인상 깊습니다!
seoyeon435
reviewed
Aug 28, 2025
| const { fire2 } = useConfetti(); | ||
|
|
||
| const changeName = () => { | ||
| setName((prev) => (prev === "아기사자" ? "아기사자 백지연" : "아기사자")); |
There was a problem hiding this comment.
prev를 활용해 최신 상태 기준으로 안전하게 업데이트하는 점 배워갑니다아
Hyunuk-Lee
reviewed
Aug 28, 2025
Comment on lines
+8
to
+9
| const [name, setName] = useState("아기사자"); | ||
| const { fire2 } = useConfetti(); |
There was a problem hiding this comment.
이름 상태를 관리하면서 변경될 때마다 useConfetti의 fire2 효과를 실행하는 좋은 코드인 것 같습니다
HongYounKim
reviewed
Aug 28, 2025
| spread: 360, | ||
| origin: { | ||
| x: Math.random(), | ||
| y: Math.random() - 0.2, |
There was a problem hiding this comment.
Math.random()을 넣어서 confetti가 화면 곳곳에서 터지는 효과가 다채롭고 재밌는 것 같아요!
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.
커스텀 훅 세션
구현 내용
Refactor: useWindowSize로 리팩토링
Refactor: useCountdown, calculateTime으로 리팩토링
Feature: useMyName 구현