microsoft / microsoft/TypeScript
Detect unreachable code in more scenarios
Personne n'a encore pris cette issue.
- Langage dominant
- Go
- Étoiles
- 111k
- Forks
- 14.3k
- Merge moyen
- 2 j 4 h
- PR mergées (30 j)
- 132
Description
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.
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez par reproduire les exemples d’opérateurs logiques, d’opérateur ternaire et de littéraux dans les liens TypeScript Playground fournis, puis suivez les vérifications existantes du compilateur concernant le code inatteignable ainsi que l’option allowUnreachableCode. Le travail est terminé lorsque les cas truthy et falsy convenus produisent des erreurs du compilateur sans modifier le JavaScript généré.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- typescript
- Domaine
- compilers
- Type d'issue
- Fonctionnalité
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100