Rule proposal: Require declaring class members upfront
Declaring the members upfront as class fields makes code more readable and can prevent typos. We would also allow initializing in the constructor, but I think we should auto-fix to class field as it's neater.
Inspired by eslint/eslint#11540.
Any suggestions on the rule name?
Fail
class X {
func setName(name) {
this.name = name;
}
func say() {
console.log(this.name);
}
}Pass
class X {
name;
func setName(name) {
this.name = name;
}
func say() {
console.log(this.name);
}
}How it would prevent typos:
class X {
name;
func setName(name) {
this.name = name;
}
func say() {
// This would have resulted in `undefined` being printed without this rule.
console.log(this.myName); // ESLint error
}
}
sindresorhus / eslint-plugin-unicorn
More than 100 powerful ESLint rules
4.2k
How does funding with Polar work?
1
Pay now to fund the work behind this issue.
2
Get updates on progress being made.
3
Maintainer is rewarded once the issue is completed.
FAQ
Backer
You're funding impactful open source efforts
Contributor
You want to contribute to this effort
Maintainer
You want to get funding like this too