microsoft / microsoft/TypeScript
Checking if Enum | falsely in if condition if value of Enum might be zero
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
Suggestion
🔍 Search Terms
checker, enum, undefined, dx
✅ Viability 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, new syntax sugar for JS, etc.)
- This feature would agree with the rest of TypeScript's Design Goals.
⭐ Suggestion
Check if a value has enum | falsely types, and one member of the enum is zero. And the value In top level of If/while/etc's conditions.
It's a common bug case. Even after condition, CFA could strip the zero value. But If we does not use the enum value, we will never find it out.
📃 Motivating Example
enum E {
Foo,
Bar
}
function foo (v: E | undefined) {
if (!v) { // Should be suggestion or error?
throw new Error("Hoops")
}
console.log("Acceptable") // But without E.foo
}
💻 Use Cases
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 motivierenden Beispiel E | undefined und überprüfe das Verhalten des Checkers bei der Wahrheitswert-Narrowing in if-Bedingungen. Bestimme, ob das Ergebnis ein Vorschlag oder ein Fehler sein sollte, und definiere anschließend Tests, die ein Enum-Mitglied mit dem Wert null und den eingegrenzten Typ abdecken; abgeschlossen ist die Arbeit, wenn das beabsichtigte Diagnoseverhalten spezifiziert und abgedeckt ist.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- typescript
- Bereich
- compilers
- Issue-Typ
- Feature
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 25/100