eslint-plugin-no-array-reduce

eslint-plugin-no-array-reduce

ESLint rule to disallow Array.reduce() method.

Method reduce() in most cases can be written as map(), filter() or one of the for() loops which benefits in code readability and makes it easier to maintain for future developers.
There are many debates, discussions and other resources related to it:

• Jake Archibald
• Is reduce() bad
• Write more readable JavaScript

Install

npm install --save-dev eslint-plugin-no-array-reduce

Then extend eslint config:

{
  "extends": [
    // ...
    "plugin:no-array-reduce/recommended"
  ]
}

Fail

const groceries = [
  { name: 'milk', type: 'dairy' },
  { name: 'cheese', type: 'dairy' },
  { name: 'beef', type: 'meat' },
];

// Filter dairy products
const dairy = groceries.reduce((acc, grocery) => (grocery.type === 'dairy' ? acc.concat(grocery) : acc), []);

Pass

// Filter dairy products
const dairy = groceries.filter((grocery) => grocery.type === 'dairy');

Subjectively there are cases where you still might want to use reduce() with eslint-disable.

Contributing

All contributions are welcome!