[9주차/지니] 워크북 제출합니다. #100
Open
[9주차/지니] 워크북 제출합니다. #100
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
added 2 commits
November 25, 2025 17:04
| import { create } from 'zustand'; | ||
| import type { CartItem } from '../constants/cartItems'; | ||
| import cartItems from '../constants/cartItems'; | ||
|
|
There was a problem hiding this comment.
cart, modal을 하나의 store로 묶어서 괸리에 더 용이해보이는것같아요 !
컴포넌트에서 가져다 쓰기도 훨씩 직관적인것같아 좋은코드인것같습니다
Comment on lines
+5
to
+16
| // 합계 계산 유틸리티 | ||
| const computeTotals = (items: CartItem[]) => { | ||
| return items.reduce( | ||
| (acc, item) => { | ||
| acc.amount += item.amount; | ||
| acc.total += item.amount * item.price; | ||
| return acc; | ||
| }, | ||
| { amount: 0, total: 0 } | ||
| ); | ||
| }; | ||
|
|
There was a problem hiding this comment.
cart항목이 변할때 amount와 total을 다시 계산해야하는 이 로직을 헬퍼함수로 분리하여서 재사용성과 가독성 면에서 좋아보입니다!~!
sunnyinha
reviewed
Nov 29, 2025
| const Footer = () => { | ||
| const amount = useStore((state) => state.amount); | ||
| const total = useStore((state) => state.total); | ||
| const openModal = useStore((state) => state.openModal); |
Collaborator
There was a problem hiding this comment.
연관된 값들을 한 번에 가져오고 싶을 때는 아래처럼 하나의 useStore 호출로 묶을 수 있어요.
다만 객체 셀렉터는 호출할 때마다 새로운 객체를 만들어서 반환하기 때문에,
shallow 없이 사용하면 값이 바뀌지 않아도 컴포넌트가 리렌더링될 수 있습니다.
그래서 아래 예시처럼 amount와 total을 묶어서 사용할 경우에는 shallow를 함께 적용하는 방식이 권장됩니다.
const { amount, total } = useStore(
(state) => ({ amount: state.amount, total: state.total }),
shallow
);
지금처럼 각각 따로 selector를 두면 레퍼런스가 안정적이라 불필요한 렌더링을 자연스럽게 피할 수 있어요!
이번 주 잘 해주셔서 따로 피드백 드릴 점이 없어 추가적인 TMI 적어드리고 갑니다!
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
✅ 워크북 체크리스트
✅ 컨벤션 체크리스트
📌 주안점