microsoft / microsoft/TypeScript
Compilation option: `stricterPropertyInitialization`
Personne n'a encore pris cette issue.
- Langage dominant
- Go
- Étoiles
- 111k
- Forks
- 14.4k
- Merge moyen
- 1 j 19 h
- PR mergées (30 j)
- 117
Description
🔍 Search Terms
Keywords: "stricter property initiazliation"
✅ Viability Checklist
- This wouldn't be a breaking change in existing TypeScript/JavaScript code
- This wouldn't change the runtime behavior of existing JavaScript code
- This could be implemented without emitting different JS based on the types of the expressions
- This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, new syntax sugar for JS, etc.)
- This isn't a request to add a new utility type: https://github.com/microsoft/TypeScript/wiki/No-New-Utility-Types
- This feature would agree with the rest of our Design Goals: https://github.com/Microsoft/TypeScript/wiki/TypeScript-Design-Goals
⭐ Suggestion
So, TypeScript offers the strictPropertyInitialization compiler option, which is helpful, but it has this behavior:
class UserAccount {
name: string; // not an error, initialized in constructor
accountType = "user"; // initialized
email: string; // this is an error
address: string | undefined; // NOT an error but ALSO not technically initialized!
constructor(name: string) {
this.name = name;
}
}
This is great except for one thing: performance. Uninitialized properties in the class (specifically: address) will, at least in V8, potentially switch the class to dictionary mode vs struct mode when the property eventually gets assigned, leading to slower read/writes. This was confirmed by a V8 developer in this StackOverflow thread.
So the suggestion is that stricterPropertyInitialization sets strictPropertyInitialization to true AND throws an error if the author has added | undefined to the type to implicity define the type. In other words, "strict property initialization" would mean what it says: strict property initialization. If it is not initialized, it is an error. In the above example, from the tsconfig docs, the address is not initialized. It's too late to change that as default behavior, hence the additional flag.
Note: maybe stricterPropertyInitialization is too awkward? Something like disallowImplicitUndefinedClassFields? 🤷♂
📃 Motivating Example
Most devs probably won't need to worry about the performance implications of internal dictionary vs. struct. In my case, in the library I'm maintaining / working on, every millisecond / fraction of a millisecond counts, so I'm trying to determine the fastest path in every scenario. I expected strictPropertyInitialization to mean what it says to prevent properties from not being initialized and avoid any accidental performance pitfalls from non-initialization, but it doesn't. 🤷♂
Meaning, in the above example:
class UserAccount {
name: string; // not an error, initialized in constructor
accountType = "user"; // initialized
email: string; // this is an error
address: string | undefined; // NOT an error but ALSO not technically initialized!
constructor(name: string) {
this.name = name;
}
someMethod() {
// The class will now possible de-optimize its performance to Dictionary mode, since V8
// thinks that the object needs to have arbitrary properties added. It doesn't know about `address`
this.address = '123 fake street'
}
}
So, most simply, stricterPropertyInitialization would throw a compilation error if | undefined is added to a property and the property is not explicitly initialized with a field initializer or in the constructor.
If the code author wishes to actually initialize the field, to undefined, it must be something like:
class UserAccount {
// ...
address: string | undefined = undefined;
constructor(name: string) {
this.name = name;
}
}
💻 Use Cases
-
What do you want to use this for?
A high-performance TS/JS library -
What shortcomings exist with current approaches?
There are no workarounds as far as I know. You simply have to let all devs on your team know to not write this and hope people catch it. -
What workarounds are you using in the meantime?
Vigilance
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
L’issue ne nomme aucun fichier, test ni point d’entrée. Commencez par localiser l’implémentation et les tests de strictPropertyInitialization, puis déterminez comment une option distincte doit traiter les champs non initialisés typés avec | undefined par rapport aux champs explicitement initialisés. Le travail est terminé lorsque le comportement et le nom de la nouvelle option sont définis et que les exemples UserAccount sont couverts par des tests du compilateur.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- typescript
- Domaine
- compilers
- Type d'issue
- Fonctionnalité
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100