useState of the Tuple type is not safe
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'm reading TS and noticed a vulnerability like UseState, as described in the documentation, we get a tuple of two elements.
The tuple in TS has a vulnerability because you can use array methods like pop, push on them. Which isn't safe because TS doesn't compute the type after that, it stays.
I suggest adding a readonly modifier to avoid these situations
const state = useState();
state.pop();
const [value, setValue] = state;
// setValue is undefined, but ts shows that the value is of type React.Dispatch<React.SetStateAction<undefined>
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
L’issue ne nomme aucun fichier, test ni point d’entrée dans react.dev. Déterminez d’abord si le tuple readonly useState demandé relève de la documentation ou des définitions de types TypeScript de React, puis examinez les exemples useState existants et le comportement des types. Le travail est terminé lorsque le tuple ne peut pas être muté de manière non sûre et que le comportement TypeScript documenté est vérifié.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- react, typescript
- Domaine
- documentation
- Type d'issue
- Fonctionnalité
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Activité
- À l'abandon
- Clarté
- À clarifier
- Accessibilité débutants
- 25/100