microsoft / microsoft/TypeScript
String literal type inconsistency in overload resolution
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
As far as I see, the TS compiler still sees a string literal type of a single literal as an old style "string parameter" thing. I thought that the concept of string literal types should replace overloading on string parameters, but seems that it doesn't.
That complicates things, and results in a "strange" behaviour in overload resolution:
class A256 {
m(a): any;
m(a: "a" | "b"): string;
m(a: "c"): string;
m(a): any {}
}
var foo1 = new A256().m("a"); // 'any'
var foo2 = new A256().m("c"); // 'string'
What should be the correct logic with string literal types in overload resolution?
Shouldn't overloads with unions of literals precede the others as well, when resolving overloads?
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 compiler l’exemple A256 et comparez les types inférés de foo1 et foo2 avec le comportement attendu des surcharges. Suivez ensuite la gestion par le compilateur de la résolution des surcharges pour les unions de littéraux de chaîne ; le travail est terminé lorsque le comportement est défini et que l’exemple est résolu de manière cohérente avec cette règle, avec l’ajout d’une couverture de régression si l’emplacement de test pertinent est identifié.
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é
- 5/5
- Temps estimé
- Plus d'une semaine
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 25/100