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.
36 lines
1018 B
Markdown
36 lines
1018 B
Markdown
---
|
|
title: Do not wrap a simple expression with a primitive result type in useMemo
|
|
impact: LOW-MEDIUM
|
|
impactDescription: wasted computation on every render
|
|
tags: rerender, useMemo, optimization
|
|
---
|
|
|
|
## Do not wrap a simple expression with a primitive result type in useMemo
|
|
|
|
When an expression is simple (few logical or arithmetical operators) and has a primitive result type (boolean, number, string), do not wrap it in `useMemo`.
|
|
Calling `useMemo` and comparing hook dependencies may consume more resources than the expression itself.
|
|
|
|
**Incorrect:**
|
|
|
|
```tsx
|
|
function Header({ user, notifications }: Props) {
|
|
const isLoading = useMemo(() => {
|
|
return user.isLoading || notifications.isLoading
|
|
}, [user.isLoading, notifications.isLoading])
|
|
|
|
if (isLoading) return <Skeleton />
|
|
// return some markup
|
|
}
|
|
```
|
|
|
|
**Correct:**
|
|
|
|
```tsx
|
|
function Header({ user, notifications }: Props) {
|
|
const isLoading = user.isLoading || notifications.isLoading
|
|
|
|
if (isLoading) return <Skeleton />
|
|
// return some markup
|
|
}
|
|
```
|