microsoft / microsoft/TypeScript
Incorporating exhaustiveness analysis into the control flow of if statements.
Nessuno ha ancora preso questa issue.
- Lingua principale
- Go
- Stelle
- 111k
- Fork
- 14.3k
- Merge medio
- 1g 19h
- PR unite (30g)
- 117
Descrizione
🔍 Search Terms
"initialization", "variable", "inference", "assignment", "assigned", "automatic detection"
✅ Viability Checklist
- This wouldn't be a breaking change in existing TypeScript/JavaScript code
- This wouldn't change the runtime behavior of existing JavaScript code
- This could be implemented without emitting different JS based on the types of the expressions
- This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, new syntax sugar for JS, etc.)
- This isn't a request to add a new utility type: https://github.com/microsoft/TypeScript/wiki/No-New-Utility-Types
- This feature would agree with the rest of our Design Goals: https://github.com/Microsoft/TypeScript/wiki/TypeScript-Design-Goals
⭐ Suggestion
One's code may implicitely involve that all cases have been processed in an if-else structure. TS does detect it with switch statements, however it does not detect it with if-else clauses.
Consider the following example:
type Dinosaur = "triceratops" | "brachiosaurus"
const myCollectionOfDinosaurs: Dinosaur[] = ["triceratops", "brachiosaurus"]
const randomKey = Math.floor(Math.random() * myCollectionOfDinosaurs.length)
const randomlyChosenDinosaur: Dinosaur = myCollectionOfDinosaurs[randomKey]
let dinosaurGreeting: string
if (randomlyChosenDinosaur === "triceratops") dinosaurGreeting = "Hello triceratops!"
else if (randomlyChosenDinosaur === "brachiosaurus") dinosaurGreeting = "Hello brachiosaurus!"
console.log(dinosaurGreeting)
TS complains on the last line because it says dinosaurGreeting is used before being assigned.
Replacing the if-else clause with a switch statement fixes the problem. However it would be great if TS did support this feature for if-else clauses too.
📃 Motivating Example
I'd use the dinosaurs example above, and assert that this feature would remove unnecessary constraints in the language.
💻 Use Cases
- What do you want to use this for?
Writing some meaningful code with less constraints, shaping it more precisely and remaining safe from type-related problems. - What shortcomings exist with current approaches?
Most current approaches involve working around the way you intend your code to work. For example I can use !: when typing the variable, or initialize the variable with a default variable, or define it anywhere it is initialized, but all these solutions would imply thinking the code differently, in a way that is less compatible with my need.
The best approach is using a switch statement instead of an if-else clause, which doesn't impact the logic of the code. - What workarounds are you using in the meantime?
Replacing the if-else clause with a switch statement.
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia riproducendo l'esempio del dinosauro dell'issue e confronta il suo comportamento di assegnazione definitiva con quello dell'istruzione switch equivalente. Traccia l'analisi del flusso di controllo e dell'esaustività del compilatore per determinare dove vengono gestiti i rami if-else; il lavoro è terminato quando i rami esaustivi non producono più l'errore di variabile usata prima dell'assegnazione, senza modificare il JavaScript emesso.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- typescript
- Ambito
- compilers
- Tipo di issue
- Funzionalità
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 25/100