2017-06-30 4 views
0

Ich schaute auf the documentation for ng-bootstrap und bemerkte, dass one of their examples ein Komma in einer Objekt-Literal-Definition auslässt. (Ich kann nicht direkt auf die Dateien in einem Plunker verknüpfen, aber es ist Linie 30 von src/app.ts.)Erlaubt TypeScript Objektliterale ohne Kommas zwischen Eigenschaften?

@NgModule({ 
    imports: [BrowserModule, FormsModule, ReactiveFormsModule, JsonpModule, NgbModule.forRoot()], 
    declarations: [App, NgbdDropdownManual] 
    bootstrap: [App] 
}) 
export class AppModule {} 

Dies führte mich mit dem Plunker zu spielen, um, und es stellt sich heraus, Sie alle Kommas entfernen in Die Objektdeklarationen und der Code kompiliert und läuft noch - versuchen Sie console.log({a:1 b:2}) hinzuzufügen, um zu sehen, was ich meine.

Wird dies formal in allen TypeScript unterstützt oder ist es ein Artefakt der Art und Weise, wie der Code in diesem speziellen Beispiel kompiliert und ausgeführt wird?

+0

Zeile 30 in src/app.ts für mich ist das 'Deklarationen: [App, NgbdDropdownManual]' Es ist nur ein Array. – toskv

+0

@toskv ja, aber es gibt kein Komma am Ende der Zeile. – Duncan

+0

ah, bemerkte es .. überprüfen Sie meine Antwort. :) – toskv

Antwort

3

Es ist wahrscheinlich, dass beim Einfügen von TSC fehlende Satzzeichen eingefügt werden.

zum Beispiel

let x = { 
    a: 2 b: 4 
} 

wird kompiliert:

var x = { 
    a: 2, b: 4 
}; 

Zur gleichen Zeit eine Kompilation Warnung ausgegeben Sie sagen, dass , erwartet wird.

Während dies zu funktionieren scheint, würde ich mich nicht darauf verlassen, da es nichts als unbeabsichtigte Konsequenz zu sein scheint.

Sie können dieses Beispiel auf dem Spielplatz here sehen.

+0

Ihr Beispiel-Beispiel-Link lädt die Standard-Playground-Seite für mich - vielleicht überprüfen Sie Ihre Link-Syntax? Wie auch immer, ich bin neu in TypeScript, aber solange ich Warnungen als Fehler in meinem Compiler behandeln kann, scheint das nicht allzu schlimm zu sein. Vielen Dank! – Coderer

+0

Ich habe den Link repariert – toskv

Verwandte Themen