microsoft / microsoft/TypeScript
Discriminated unions is not work when field key is number
Offen
Dieses Issue hat noch niemand übernommen.
Awaiting More Feedback
Suggestion
- Vorherrschende Sprache
- Go
- Sterne
- 111k
- Forks
- 14.4k
- Ø Merge
- 1 T. 19 Std.
- Gemergte PRs (30 T.)
- 117
Beschreibung
🔎 Search Terms
- discriminated unions, "discriminated-unions"
- "numbers as keys in object", "number key"
- "type guard"
🕗 Version & Regression Information
- This is a crash
⏯ Playground Link
💻 Code
type A = number[] | string[]
type B = { 0: number, a: 1 } | { 0: string, a: '1' }
declare const a: A
declare const b: B
if (typeof a[0] === 'number') {
console.log(a)
// ^? A
}
if (typeof b['a'] === 'number') {
console.log(b)
// ^? { 0: number, a: 1 }
}
if (typeof b['0'] === 'number') {
console.log(b)
// ^? B
}
if (typeof b[0] === 'number') {
console.log(b)
// ^? B
}
🙁 Actual behavior
Unable to type guard b as an option of the union type B.
🙂 Expected behavior
Type b as { 0: number, a: 1 } when type guarding it with typeof b['0'] === 'number'.
Additional information about the issue
- microsoft/TypeScript#55632
- microsoft/TypeScript#55766
- microsoft/TypeScript#55961
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 dem verknüpften TypeScript Playground-Beispiel und vergleiche das bestehende Verhalten für String- und numerische Schlüssel. Lies die zugehörigen Issues microsoft/TypeScript#55632, #55766 und #55961, um den Kontext der Type Guards zu verstehen. Erledigt ist die Aufgabe, wenn typeof b['0'] === 'number' B auf { 0: number, a: 1 } eingrenzt, ohne die anderen Beispiele zu beeinträchtigen.
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
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100