microsoft / microsoft/TypeScript
Implementation signatures that do not contain all overloaded signatures are allowed in class definitions
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Go
- Sterne
- 111k
- Forks
- 14.4k
- Ø Merge
- 1 T. 19 Std.
- Gemergte PRs (30 T.)
- 117
Beschreibung
🔎 Search Terms
implementation contain overloaded class member
🕗 Version & Regression Information
- This is the behavior in every version I tried, and I reviewed the FAQ for entries about overloading class member functions
⏯ Playground Link
💻 Code
class C
{
foo( a? : number ) : void;
foo( a : number, b : number ) : void;
foo( a : number, b? : number ) : void
{
console.log( typeof a ); // typeof a is number
}
}
new C().foo(); // prints "undefined"
🙁 Actual behavior
The compilation succeeded.
The type of argument "a" in function foo is number but prints "undefined" which is wrong.
Because the implementation signature of foo does not include all overload signatures, so compilation should fail.
🙂 Expected behavior
The compilation should fail.
Additional information about the issue
This bug is only reproduced with class member functions.
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne mit der verknüpften TypeScript Playground-Reproduktion und bestätige, dass die Überladungen von Klassenmembern kompiliert werden, obwohl die Implementierungssignatur den Fall ohne Argumente nicht enthält. Verfolge den Pfad zur Überprüfung von Überladungen für Klassenmember, füge einen Regressionstest für dieses Beispiel hinzu und überprüfe, dass die Kompilierung nun wie erwartet fehlschlägt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- typescript
- Bereich
- compilers
- Issue-Typ
- Bug
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Klar beschrieben
- Anfängerfreundlichkeit
- 35/100