ea85a053c5
All checks were successful
Deploy / deploy (push) Successful in 7m30s
- Added new skills: find-skills, frontend-ui-engineering, and vercel-react-best-practices to skills-lock.json. - Updated use-pixel-heat.ts to import both animate and AnimationPlaybackControls from motion for improved animation handling.
46 lines
824 B
Markdown
46 lines
824 B
Markdown
---
|
|
title: Narrow Effect Dependencies
|
|
impact: LOW
|
|
impactDescription: minimizes effect re-runs
|
|
tags: rerender, useEffect, dependencies, optimization
|
|
---
|
|
|
|
## Narrow Effect Dependencies
|
|
|
|
Specify primitive dependencies instead of objects to minimize effect re-runs.
|
|
|
|
**Incorrect (re-runs on any user field change):**
|
|
|
|
```tsx
|
|
useEffect(() => {
|
|
console.log(user.id)
|
|
}, [user])
|
|
```
|
|
|
|
**Correct (re-runs only when id changes):**
|
|
|
|
```tsx
|
|
useEffect(() => {
|
|
console.log(user.id)
|
|
}, [user.id])
|
|
```
|
|
|
|
**For derived state, compute outside effect:**
|
|
|
|
```tsx
|
|
// Incorrect: runs on width=767, 766, 765...
|
|
useEffect(() => {
|
|
if (width < 768) {
|
|
enableMobileMode()
|
|
}
|
|
}, [width])
|
|
|
|
// Correct: runs only on boolean transition
|
|
const isMobile = width < 768
|
|
useEffect(() => {
|
|
if (isMobile) {
|
|
enableMobileMode()
|
|
}
|
|
}, [isMobile])
|
|
```
|