microsoft / microsoft/TypeScript

Can not specify toString/valueOf methods in object literal

Offen
#18,537 3 Kommentare 2 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bug Domain: check: Excess Property Checking Needs Human Review
Vorherrschende Sprache
Go
Sterne
111k
Forks
14.4k
Ø Merge
1 T. 19 Std.
Gemergte PRs (30 T.)
117

Beschreibung

TypeScript Version: 2.5.2

Code

interface X {
	name: string
}

const x1: X = {
	get name() { return 'x1' },
	toString() { return this.name },
//	^^^^^^^^
//	Type '{ readonly name: string; toString(): string; }' is not assignable to type 'X'.
//	Object literal may only specify known properties, and 'toString' does not exist in type 'X'.
}
const x2: X = {name: 'x2'}
x2.toString = function () { return this.name } // But add `toString` later is allowed.

Expected behavior:

Should allow object literal specify toString, valueOf and all properties exists on Object.prototype, or at least improve the error report to give a workaround (const x: X & Object).

Actual behavior:

See comments in code.

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 TypeScript 2.5.2-Reproduktion im Issue und vergleiche die Excess-Property-Prüfung von Objekt-Literalen für toString/valueOf mit der Zuweisung derselben Methode nach der Initialisierung. Bestimme, ob Mitglieder von Object.prototype akzeptiert werden sollten oder ob die Diagnose den gezeigten X & Object-Workaround vorschlagen sollte. Als erledigt gilt die Aufgabe, wenn das gewählte Verhalten implementiert und durch einen Regressionstest abgedeckt 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
Aktiv
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
48/100

Neue Issues direkt in Ihr Postfach

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