You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We shouldn't have useMemo for one time initializations. While the current behaviour is the same as a useState, useMemo should only be used as an optimization. Future React releases might break this functionality.
You may rely on useMemo as a performance optimization, not as a semantic guarantee. In the future, React may choose to “forget” some previously memoized values and recalculate them on next render, e.g. to free memory for offscreen components. Write your code so that it still works without useMemo — and then add it to optimize performance.
Proposal
Replace useMemo with useState for one time initializations. There might be more places where we need to change this:
If the rule-of-thumb is "only use useMemo if the code would also work without it" - well, technically it does work, but not memoizing in these cases would trigger a dependency chain that could have a dramatic impact.
And this is true for all uses of useMemo I have reviewed so far (even including those you pointed out). The code will still work, but if react forgets about the memo it could incur some significant cost.
So how should we decide which pattern to use?
There are at least these options to choose from:
useMemo – should only be used if it would be ok for react to occasionally forget about it, but it also looks best and I personally think the name fits best and it does return the value and not an array to be destructured.
constthing=useMemo(()=>createThing());// with dependencyconstthing=useMemo(()=>dependency&&dependency.createThing(),[dependency]);
useState with lazy init / with useEffect for dependencies – this is also fine I guess, and everyone should be able to understand.
const[thing]=useState(()=>createThing());// with dependencyconst[thing,setThing]=useState();useEffect(()=>{dependency&&setThing(dependency.createThing());},[dependency]);
for the second case, we could even go one step further and introduce our own hook that would behave exactly like most people think useMemo does.
Target Use Case
We shouldn't have
useMemo
for one time initializations. While the current behaviour is the same as auseState
,useMemo
should only be used as an optimization. Future React releases might break this functionality.https://tkdodo.eu/blog/use-state-for-one-time-initializations
React Forget
Proposal
Replace
useMemo
withuseState
for one time initializations. There might be more places where we need to change this:react-google-maps/examples/deckgl-overlay/src/deckgl-overlay.ts
Line 16 in 8140b4a
react-google-maps/src/components/map-control.tsx
Line 50 in 8140b4a
react-google-maps/src/components/pin.tsx
Line 22 in 8140b4a
The text was updated successfully, but these errors were encountered: