microsoft / microsoft/TypeScript
Suggestion: should non-null assert propagate?
Personne n'a encore pris cette issue.
- Langage dominant
- Go
- Étoiles
- 111k
- Forks
- 14.3k
- Merge moyen
- 2 j 4 h
- PR mergées (30 j)
- 132
Description
I was wondering whether ! should be taken into account by the dataflow analysis, and I don't see why it shouldn't.
It is like a cast that says "this value is not null", so from this point I guess the compiler could remove undefined/null from future uses.
For example, I use ! to work around #9631, so I have code that looks like (simplified):
protected dataArrayChanged(changes: ChangeRecord[]) {
// #9631: TS incorrectly infers `change: ChangeRecord | undefined`
for (const change of changes) {
for (let i = 0; i < change!.removed.length; i++)
this.dt.row(i).remove();
if (change!.addedCount > 0)
this.dt.rows.add(this.data.slice(change!.index, change!.index + change!.addedCount));
}
this.dt.draw();
}
Observe how I added 5 ! to make change not null.
The first one could have been enough. After all, once I say "change is not undefined", there is no reason to assume it could be until I modify the variable again.
// Let's say I know x is not undefined
const x: number | undefined;
// Here x: number | undefined, so x! is required
x!.toString();
// Here x: number because of x! above
x.toString(); // ok
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 examiner le comportement de l’analyse de flux de données décrit dans cet issue ainsi que le contexte associé dans #9631. Reproduisez les exemples impliquant des non-null assertions répétées et déterminez la sémantique de propagation attendue. La tâche est terminée lorsque le comportement est résolu de manière cohérente et couvert par des tests appropriés du compilateur.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- typescript
- Domaine
- compilers
- Type d'issue
- Fonctionnalité
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 30/100