@putout/plugin-remove-useless-spread
v11.1.0
Published
🐊Putout plugin adds ability to remove useless spread
Downloads
64,815
Maintainers
Readme
@putout/plugin-remove-useless-spread
Spread syntax can be used when all elements from an object or array need to be included in a list of some kind.
(c) MDN
🐊Putout plugin adds ability to remove useless spread syntax.
Install
npm i @putout/plugin-remove-useless-spread
Rule
{
"rules": {
"remove-useless-spread/array": "on",
"remove-useless-spread/object": "on",
"remove-useless-spread/nested": "on"
}
}
array
The thing is [...b]
can be used for:
- copying an array;
- converting different value type like
string
to anarray
.
So better to be more concrete and use slice
for copying and Array()
/Array.from()
for converting to decrease cognitive load.
Also sometimes there is no need on any of this operations, and we can drop spread
.
❌ Example of incorrect code
for (const a of [...b]) {}
const places = [...getPlaces()];
✅ Example of correct code
for (const a of b) {}
const places = getPlaces();
// Array constructor creates sparse array
[...Array(5)].map(Number);
object
❌ Example of incorrect code
const a = {
...fn(),
};
✅ Example of correct code
const a = fn();
nested
Checkout in 🐊Putout Editor.
❌ Example of incorrect code
[
...[
...a,
...b,
],
...x,
];
✅ Example of correct code
[
...a,
...b,
...x,
];
License
MIT