microsoft / microsoft/TypeScript

Reflect.has fails to act as type guard (should act same as "in" operator)

Offen
#30,688 2 Kommentare 9 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Domain: lib.d.ts Experience Enhancement Suggestion
Vorherrschende Sprache
Go
Sterne
111k
Forks
14.3k
Ø Merge
2 T. 4 Std.
Gemergte PRs (30 T.)
132

Beschreibung

TypeScript Version: ^3.4.0-dev.20190330

Search Terms:
Reflect.has
in operator

Code

const test1 = (a: { field: number } | {}) => (("field" in a) ? a.field : 0);

const test2 = (a: { field: number } | {}) => Reflect.has(a, "field") ? a.field : 0;

Expected behavior:
Both compile successfully. "in" operator acts as a type guard per https://github.com/Microsoft/TypeScript/issues/10485. Reflect.has should act the same as "in" operator here.

We'd like to use Reflect.has a lot more and this case holds us back. Thank you!

Actual behavior:
"in" operator line compiles; Reflect.has line does not compile. Error:

allPasos.ts:445:31 - error TS2339: Property 'field' does not exist on type '{} | { field: number; }'.
Property 'field' does not exist on type '{}'.

445 Reflect.has(a, "field") ? a.field : 0;

Playground Link:

https://goo.gl/2R4dkS

Related Issues:

https://github.com/Microsoft/TypeScript/issues/10485

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne mit der verknüpften Playground-Reproduktion und vergleiche das bestehende Narrowing-Verhalten des in-Operators mit Reflect.has; lies zur Einordnung den zugehörigen Issue #10485. Als abgeschlossen gilt die Aufgabe, wenn der gezeigte Reflect.has-Ausdruck kompiliert und sich wie im Issue erwartet verhält und eine Abdeckung für die Regression vorhanden ist.

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
38/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.