Disallow Functions in Loops (no-loop-func)

Writing functions within loops tends to result in errors due to the way the function creates a closure around the loop. For example:

for (var i = 0; i < 10; i++) {
    funcs[i] = function() {
        return i;
    };
}

In this case, you would expect each function created within the loop to return a different number. In reality, each function returns 10, because that was the last value of i in the scope.

Rule Details

This error is raised to highlight a piece of code that may not work as you expect it to and could also indicate a misunderstanding of how the language works. Your code may run without any problems if you do not fix this error, but in some situations it could behave unexpectedly.

The following patterns are considered warnings:

for (var i=10; i; i--) {
    (function() { ... })();
}

while(i) {
    var a = function() {};
    a();
}

do {
    function a() {};
    a();
} while (i);

The following patterns are considered okay and do not cause warnings:

var a = function() {};

for (var i=10; i; i--) {
    a();
}

Further Reading

Version

This rule was introduced in ESLint 0.0.9.

Resources