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 a const variable holds this 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.
    }
    var bar = 1;
    var unusedVar; // UNUSED_DECL alarm because local variable 'unusedVar' is not used.
    doSomething(bar);
}

Compliant Code Example

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

Version

This rule was introduced in DeepScan 1.0.0-alpha.