microsoft / microsoft/TypeScript
Detect unreachable code in more scenarios
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
logical assignment binary expression dead never truthy falsy
Suggestion
Raise compiler errors when the left side of an expression using a logical operator (&& or ||) is unrechable.
Also detect unreachable branches of ternary expressions.
Use Cases
Code that is statically verifiable to be unreahable is almost always a programmer error so TypeScript should detect unreachable code in more scenarios.
Examples
declare const truthy: true
declare const falsy: false;
declare function doSomething(): number
const a = true ? 5 : doSomething(); // error: doSomething is unreachable
const b = truthy ? 5 : doSomething(); // error: doSomething is unreachable
const c = false ? doSomething() : 5; // error: doSomething is unreachable
const d = falsy ? doSomething() : 5; // error: doSomething is unreachable
if (!truthy) {
doSomething(); // error: doSomething is unreachable
}
if (falsy) {
doSomething(); // error: doSomething is unreachable
}
It would also be nice to expand unreachable code checks to other expressions that are known to be truthy or falsy eg. object/array literals, NaN eg.
declare function doSomething(): number
const a = [1,2,3] || doSomething() // error: doSomething is unreachable
const b = { count: 1 } && doSomething() // error: doSomething is unreachable
if ([1,2,3]) {
doSomething() // error: doSomething is unreachable
}
if ({ count: 1 }) {
doSomething() // error: doSomething is unreachable
}
Finally, detecting unreachable branches of a ternary expression
declare const truthy: true
declare const falsy: false;
declare function doSomething(): number
const a = true ? 5 : doSomething(); // error: doSomething is unreachable
const b = truthy ? 5 : doSomething(); // error: doSomething is unreachable
const c = false ? doSomething() : 5; // error: doSomething is unreachable
const d = falsy ? doSomething() : 5; // error: doSomething is unreachable
Checklist
My suggestion meets these guidelines:
- 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, etc.)
- This feature would agree with the rest of TypeScript's Design Goals.
This wouldn't be a breaking change in existing TypeScript/JavaScript code
There would be new type errors. The extra safety could be opted into by extending the allowUnreachableCode flag or creating a new flag.
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 gli esempi di operatori logici, operatore ternario e letterali nei link a TypeScript Playground indicati, quindi segui i controlli esistenti del compilatore sul codice irraggiungibile e l’opzione allowUnreachableCode. Il lavoro è completato quando i casi truthy e falsy concordati producono errori del compilatore 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
- 35/100