Question

I have a certain loop occurring several times in various functions in my code.
To illustrate with an example, it’s pretty much along the lines of the following:

for (var i=0;i<= 5; i++) {
    function1(function2(arr[i],i),$('div'+i));
    $('span'+i).value = function3(arr[i]);
}

Where i is the loop counter of course. For the sake of reducing my code size and avoid repeating the loop declaration, I thought I should replace it with the following:

function loop(s) {
 for (var i=0;i<= 5; i++) { eval(s); }
}

[...]

loop("function1(function2(arr[i],i),$('div'+i));$('span'+i).value = function3(arr[i]);");

Or should I? I’ve heard a lot about eval() slowing code execution and I’d like it to work as fast as a proper loop even in the Nintendo DSi browser, but I’d also like to cut down on code. What would you suggest?

Thank you in advance!

Answer 1

Gah!

This is a good question, but no, don’t ever do that. Using eval in general is not recommended, as you won’t see parse errors at load time, only at run time (harder to debug), it’s harder to understand what’s in scope when (harder to write), and you lose all your toolchain support (syntax highlight, script debugging).

Fortunately, since Javascript is basically a functional language, why not create a function that encapsulates what you want to do, and just call that?

function doMyThingNTimes(n, arr) {
    for (var i=0;i <= n; i++) {
        function1(function2(arr[i],i),$('div'+i));
        $('span'+i).value = function3(arr[i]);
    }
}