Clarify why useEffect() may need exhaustive deps
Personne n'a encore pris cette issue.
- Langage dominant
- JavaScript
- Étoiles
- 11.8k
- Forks
- 7.9k
- Merge moyen
- 1 j 11 h
- PR mergées (30 j)
- 11
Description
I may be having a... moment, but in Conditionally firing an effect, is this sentence accurate?
If you use this optimization, make sure the array includes all values from the component scope (such as props and state) that change over time and that are used by the effect. Otherwise, your code will reference stale values from previous renders.
Doesn't useEffect capture the latest props and state the moment any of its dependencies change? What can happen is the effect misses some updates when things it references change and they're not included as deps, but when it does run the props and state are "fresh" (pertaining to the latest render), are they not?
(Edit: removed unrelated point)
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez par la section liée « Conditionally firing an effect » et examinez la phrase concernant les valeurs du component scope au regard du comportement documenté des dépendances de useEffect. C’est terminé lorsque la formulation explique précisément ce qui se passe lorsque des valeurs référencées sont omises du tableau de dépendances.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- javascript, react
- Domaine
- documentation
- Type d'issue
- Documentation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 45/100