@stylistic/js/

max-statements-per-line

A line of code containing too many statements can be difficult to read. Code is generally read from the top down, especially when scanning, so limiting the number of statements allowed on a single line can be very beneficial for readability and maintainability.

function foo () { var bar; if (condition) { bar = 1; } else { bar = 2; } return true; } // too many statements

Rule Details

This rule enforces a maximum number of statements allowed per line.

Options

max

The "max" object property is optional (default: 1).

Examples of incorrect code for this rule with the default { "max": 1 } option:

/*eslint @stylistic/js/max-statements-per-line: ["error", { "max": 1 }]*/

var bar; 
var baz;
if (condition) { 
bar = 1;
 }
for (var i = 0; i < length; ++i) { 
bar = 1;
 }
switch (discriminant) { default: 
break;
 }
function foo() { 
bar = 1;
 }
var qux = function qux() { 
bar = 1;
 };
(function foo() { 
bar = 1;
 })();

incorrect

Examples of correct code for this rule with the default { "max": 1 } option:

/*eslint @stylistic/js/max-statements-per-line: ["error", { "max": 1 }]*/

var bar, baz;
if (condition) bar = 1;
for (var i = 0; i < length; ++i);
switch (discriminant) { default: }
function foo() { }
var qux = function qux() { };
(function foo() { })();

correct

Examples of incorrect code for this rule with the { "max": 2 } option:

/*eslint @stylistic/js/max-statements-per-line: ["error", { "max": 2 }]*/

var bar; var baz; 
var qux;
if (condition) { bar = 1; } else { 
baz = 2;
 }
for (var i = 0; i < length; ++i) { bar = 1; 
baz = 2;
 }
switch (discriminant) { case 'test': break; default: 
break;
 }
function foo() { bar = 1; 
baz = 2;
 }
var qux = function qux() { bar = 1; 
baz = 2;
 };
(function foo() { bar = 1; 
baz = 2;
 })();

incorrect

Examples of correct code for this rule with the { "max": 2 } option:

/*eslint @stylistic/js/max-statements-per-line: ["error", { "max": 2 }]*/

var bar; var baz;
if (condition) bar = 1; if (condition) baz = 2;
for (var i = 0; i < length; ++i) { bar = 1; }
switch (discriminant) { default: break; }
function foo() { bar = 1; }
var qux = function qux() { bar = 1; };
(function foo() { var bar = 1; })();

correct

ignoredNodes

The "ignoredNodes" object property is optional (default: []).

Examples of correct code for this rule with the "ignoredNodes" option:

/*eslint @stylistic/js/max-statements-per-line: ["error", { "max": 1, "ignoredNodes": ['BreakStatement'] }]*/

switch (lorem) {
  case ipsum: dolor(); break;
  case sit: amet(); break;
}

correct

When Not To Use It

You can turn this rule off if you are not concerned with the number of statements on each line.