microsoft / microsoft/TypeScript

strictPropertyInitialization should consider `return`

Offen
#27,555 3 Kommentare 3 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

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.2.0-dev.20181004

Code

class C {
    x: number;

    constructor(b: boolean) {
        if (b) {
            this.x = 3;
        } else {
            return new C(true);
        }
    }
}

Expected behavior:

No error.

Actual behavior:

src/a.ts:2:5 - error TS2564: Property 'x' has no initializer and is not definitely assigned in the constructor.

2     x: number;

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 dem TypeScript-Beispiel und der strictPropertyInitialization-Diagnose für die Eigenschaft x. Verfolge dann die Kontrollflussanalyse des Konstruktors, die return-Anweisungen behandelt, und überprüfe anschließend, dass das Beispiel keinen TS2564-Fehler erzeugt, wenn der return-Pfad ein neues C erstellt.

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

Neue Issues direkt in Ihr Postfach

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