microsoft / microsoft/TypeScript
Function lacks ending return statement and return type does not include 'undefined'
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Go
- Sterne
- 111k
- Forks
- 14.3k
- Ø Merge
- 1 T. 19 Std.
- Gemergte PRs (30 T.)
- 117
Beschreibung
🔎 Search Terms
switch default
🕗 Version & Regression Information
The tsc complains that the the function return undefined if a add a default case, actually the default case will never be executed, just only for exhausive check.
⏯ Playground Link
💻 Code
type Size = 'large' | 'normal' | 'small' | 'mini'
const size = {} as Size
const block = <R>(f: () => R) => {
return f()
}
const css = block((): number => {
switch (size) {
case 'normal': {
return 1
}
case 'large': {
return 2
}
case 'mini': {
return -1
}
case 'small': {
return 0
}
default: {
const n: never = size
}
}
})
🙁 Actual behavior
The tsc think that the function of css block returns number | undefined.
🙂 Expected behavior
The tsc should know that the default case is never returned, and the function of css block should returns number
Additional information about the issue
No response
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 der switch-Anweisung im Beschreibungstext des Issues. Verfolge, wie der Type Checker den vollständigen default-Zweig und die Rückgabeanalyse der umgebenden Pfeilfunktion behandelt. Als abgeschlossen gilt die Änderung, wenn der inferierte Rückgabetyp des Beispiels number statt number | undefined ist und eine Regressionstestabdeckung für dieses Verhalten vorhanden ist.
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