microsoft / microsoft/TypeScript
Private class field helpers throw a confusing error when in unbound method with undefined `this`
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Go
- Sterne
- 111k
- Forks
- 14.4k
- Ø Merge
- 1 T. 19 Std.
- Gemergte PRs (30 T.)
- 117
Beschreibung
🔎 Search Terms
this undefined bind private identifier class __classPrivateFieldSet __classPrivateFieldGet bound unbound
🕗 Version & Regression Information
- This is the behavior in every version I tried
⏯ Playground Link
💻 Code
class Foo {
#value = 1234;
getValue() { return this.#value; }
}
const foo = new Foo();
console.log(`foo.getValue() = ${foo.getValue()}`);
// Oops, I didn't bind!
const getValue = foo.getValue;
// Throws: Cannot read private member from an object whose class did not declare it
console.log(`getValue() = ${getValue()}`)
🙁 Actual behavior
Throws Cannot read private member from an object whose class did not declare it.
But the actual problem is that this is undefined and I had no idea!
🙂 Expected behavior
A special error for this when undefined; IIRC there's no possible way this would normally pass, so a special error would be a lot more clear.
Additional information about the issue
No response
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 damit, den Zugriff auf ein ungebundenes privates Feld im verlinkten TypeScript Playground zu reproduzieren, und vergleiche ihn mit dem im Issue gezeigten gebundenen Aufruf. Verfolge, wie der vom Compiler generierte private-field-Helfer den Fehler meldet, ändere dann den undefined-this-Fall so, dass er den beschriebenen klareren Fehler erzeugt, und überprüfe beide Beispiele.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- javascript, typescript
- Bereich
- compilers
- Issue-Typ
- Bug
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 25/100