Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
New: Add grouped-accessor-pairs rule (fixes #12277) (#12331)
* New: Add grouped-accessor-pairs rule (fixes #12277)

* Update docs/rules/grouped-accessor-pairs.md

Co-Authored-By: Jordan Harband <ljharb@gmail.com>

* Fix JSDoc

* Add related rules
  • Loading branch information
mdjermanovic authored and kaicataldo committed Nov 22, 2019
1 parent 5c68f5f commit 312a88f
Show file tree
Hide file tree
Showing 5 changed files with 992 additions and 0 deletions.
326 changes: 326 additions & 0 deletions docs/rules/grouped-accessor-pairs.md
@@ -0,0 +1,326 @@
# Require grouped accessor pairs in object literals and classes (grouped-accessor-pairs)

A getter and setter for the same property don't necessarily have to be defined adjacent to each other.

For example, the following statements would create the same object:

```js
var o = {
get a() {
return this.val;
},
set a(value) {
this.val = value;
},
b: 1
};

var o = {
get a() {
return this.val;
},
b: 1,
set a(value) {
this.val = value;
}
};
```

While it is allowed to define the pair for a getter or a setter anywhere in an object or class definition, it's considered a best practice to group accessor functions for the same property.

In other words, if a property has a getter and a setter, the setter should be defined right after the getter, or vice versa.

## Rule Details

This rule requires grouped definitions of accessor functions for the same property in object literals, class declarations and class expressions.

Optionally, this rule can also enforce consistent order (`getBeforeSet` or `setBeforeGet`).

This rule does not enforce the existence of the pair for a getter or a setter. See [accessor-pairs](accessor-pairs.md) if you also want to enforce getter/setter pairs.

Examples of **incorrect** code for this rule:

```js
/*eslint grouped-accessor-pairs: "error"*/

var foo = {
get a() {
return this.val;
},
b: 1,
set a(value) {
this.val = value;
}
};

var bar = {
set b(value) {
this.val = value;
},
a: 1,
get b() {
return this.val;
}
}

class Foo {
set a(value) {
this.val = value;
}
b(){}
get a() {
return this.val;
}
}

const Bar = class {
static get a() {
return this.val;
}
b(){}
static set a(value) {
this.val = value;
}
}
```

Examples of **correct** code for this rule:

```js
/*eslint grouped-accessor-pairs: "error"*/

var foo = {
get a() {
return this.val;
},
set a(value) {
this.val = value;
},
b: 1
};

var bar = {
set b(value) {
this.val = value;
},
get b() {
return this.val;
},
a: 1
}

class Foo {
set a(value) {
this.val = value;
}
get a() {
return this.val;
}
b(){}
}

const Bar = class {
static get a() {
return this.val;
}
static set a(value) {
this.val = value;
}
b(){}
}
```

## Options

This rule has a string option:

* `"anyOrder"` (default) does not enforce order.
* `"getBeforeSet"` if a property has both getter and setter, requires the getter to be defined before the setter.
* `"setBeforeGet"` if a property has both getter and setter, requires the setter to be defined before the getter.

### getBeforeSet

Examples of **incorrect** code for this rule with the `"getBeforeSet"` option:

```js
/*eslint grouped-accessor-pairs: ["error", "getBeforeSet"]*/

var foo = {
set a(value) {
this.val = value;
},
get a() {
return this.val;
}
};

class Foo {
set a(value) {
this.val = value;
}
get a() {
return this.val;
}
}

const Bar = class {
static set a(value) {
this.val = value;
}
static get a() {
return this.val;
}
}
```

Examples of **correct** code for this rule with the `"getBeforeSet"` option:

```js
/*eslint grouped-accessor-pairs: ["error", "getBeforeSet"]*/

var foo = {
get a() {
return this.val;
},
set a(value) {
this.val = value;
}
};

class Foo {
get a() {
return this.val;
}
set a(value) {
this.val = value;
}
}

const Bar = class {
static get a() {
return this.val;
}
static set a(value) {
this.val = value;
}
}
```

### setBeforeGet

Examples of **incorrect** code for this rule with the `"setBeforeGet"` option:

```js
/*eslint grouped-accessor-pairs: ["error", "setBeforeGet"]*/

var foo = {
get a() {
return this.val;
},
set a(value) {
this.val = value;
}
};

class Foo {
get a() {
return this.val;
}
set a(value) {
this.val = value;
}
}

const Bar = class {
static get a() {
return this.val;
}
static set a(value) {
this.val = value;
}
}
```

Examples of **correct** code for this rule with the `"setBeforeGet"` option:

```js
/*eslint grouped-accessor-pairs: ["error", "setBeforeGet"]*/

var foo = {
set a(value) {
this.val = value;
},
get a() {
return this.val;
}
};

class Foo {
set a(value) {
this.val = value;
}
get a() {
return this.val;
}
}

const Bar = class {
static set a(value) {
this.val = value;
}
static get a() {
return this.val;
}
}
```

## Known Limitations

Due to the limits of static analysis, this rule does not account for possible side effects and in certain cases
might require or miss to require grouping or order for getters/setters that have a computed key, like in the following example:

```js
/*eslint grouped-accessor-pairs: "error"*/

var a = 1;

// false warning (false positive)
var foo = {
get [a++]() {
return this.val;
},
b: 1,
set [a++](value) {
this.val = value;
}
};

// missed warning (false negative)
var bar = {
get [++a]() {
return this.val;
},
b: 1,
set [a](value) {
this.val = value;
}
};
```

Also, this rule does not report any warnings for properties that have duplicate getters or setters.

See [no-dupe-keys](no-dupe-keys.md) if you also want to disallow duplicate keys in object literals.

See [no-dupe-class-members](no-dupe-class-members.md) if you also want to disallow duplicate names in class definitions.

## Further Reading

* [Object Setters](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Functions/set)
* [Object Getters](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Functions/get)
* [Classes](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Classes)

## Related Rules

* [accessor-pairs](accessor-pairs.md)
* [no-dupe-keys](no-dupe-keys.md)
* [no-dupe-class-members](no-dupe-class-members.md)

0 comments on commit 312a88f

Please sign in to comment.