Unused local variables and functions should be removed

• UNUSED_DECL
• Code Quality
• Low
• No tags

This rule applies when local variables and functions are declared but not used.

For maintainability, it is recommended to remove unused code. Also, it might be a mistake that a programmer forgets to use declared variables and functions.

Note:

1. Not applied when the variable or function name starts with_ because we regard it as intentionally unused.
2. Not applied when the value of aconst variable holdsthis context because it is often used as boilerplate (e.g.const self = this;).
3. Not applied at test case code because unused declarations are usually harmless at tests and tend to occur more. Currently, BDD, TDD and QUnit style test cases are recognized.

Noncompliant Code Example

function foo() {  function unusedFunction() { // UNUSED_DECL alarm because local variable 'unusedFunction' is not used.  }  let bar = 1;  let unusedVar; // UNUSED_DECL alarm because local variable 'unusedVar' is not used.  doSomething(bar);}

Compliant Code Example

function foo() {  let bar = 1;  doSomething(bar);}

Version

This rule was introduced in DeepScan 1.0.0-alpha.

Was this documentation helpful?