microsoft / microsoft/TypeScript
Type narrowing of `super` when using `this`
Personne n'a encore pris cette issue.
- Langage dominant
- Go
- Étoiles
- 111k
- Forks
- 14.4k
- Merge moyen
- 1 j 19 h
- PR mergées (30 j)
- 117
Description
Bug Report
🔎 Search Terms
super, this, class
🕗 Version & Regression Information
Nightly
⏯ Playground Link
💻 Code
class Animal<T> {
eat<T>(this: Animal<T>) {
throw new Error('This doesnt matter');
}
}
class Dog<T> extends Animal<T> {
eat<U>(this: Dog<U>) {
super.eat(); // The type is Animal<T>.eat<T>(this: Animal<T>): void
// The type should be Animal<U>.eat<U>(this: Animal<U>): void
}
}
🙁 Actual behavior
super.eat() is typed as Animal<T>.eat<T>(this: Animal<T>): void
🙂 Expected behavior
super.eat() should be typed as Animal<U>.eat<U>(this: Animal<U>): void since this is declared as Dog<U> which is a subclass of Animal<U>.
One can workaround this by writing super.eat<U>(), but the signature then becomes goofy (it becomes Animal<T>.eat<U>(this: Animal<U>): void)
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 le TypeScript Playground lié et reproduisez la différence entre les types actuels et attendus pour super.eat() dans les classes génériques Animal et Dog. Suivez le traitement des types this par le compilateur lors de l'accès aux propriétés de super, puis ajoutez un test de régression et confirmez que super.eat() préserve le type spécifique à U sans nécessiter d'argument de type explicite.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- typescript
- Domaine
- compilers
- Type d'issue
- Bug
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- À l'abandon
- Clarté
- Clairement spécifiée
- Accessibilité débutants
- 38/100