microsoft / microsoft/TypeScript
Typescript does not recognize function Boolean() as a check for undefined
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Go
- Sterne
- 111k
- Forks
- 14.3k
- Ø Merge
- 2 T. 4 Std.
- Gemergte PRs (30 T.)
- 132
Beschreibung
I followed the submission guidelines.
I prefer to check for a defined object using Boolean(object) rather than !!object, which I consider bad style. Typescript linter recognizes !! as a check for undefined but does not recognize Boolean() as a check for undefined.
I could not find a tsconfig or tslint configuration setting related to this behavior.
2.9.0-dev.20180511 (typescript@next)
"Boolean" "Boolean(" "function Boolean"
Code
This is not production code but a simple demonstration only to satisfy the expression tree.
function booleanTest() {
let value: Array<object> | undefined = undefined;
if (Date.now() > 1) {
value = [{}];
}
if (Boolean(value)) {
console.log(value.length);
}
}
Expected behavior:
No Typescript linter errors
Actual behavior:
Typescript linter in VS Code displays an error "[ts] Object is possibly 'undefined'." with a red squiggly on the final reference to variable "value".
Playground Link:
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 bereitgestellten TypeScript-Beispiel und untersuche den Type-Checking-Pfad für das Control-Flow-Narrowing durch Boolean(value), wobei du ihn mit der erkannten Form !!value vergleichst. Als abgeschlossen gilt die Aufgabe, wenn das Beispiel nicht mehr meldet, dass value möglicherweise undefined ist, und dabei das korrekte Verhalten für andere Boolean-Aufrufe erhalten bleibt.
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