在 Javascript 中用 eval 替换重复出现的循环 - 好还是坏？

Question

我的代码中的各个函数中多次出现某个循环。 为了用一个例子来说明，它几乎符合以下内容：

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

其中 i 当然是循环计数器。为了减少我的代码大小并避免重复循环声明，我认为我应该将其替换为以下内容：

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]);");

或者我应该？我听说过很多关于 eval() 减慢代码执行速度的说法，我希望它即使在 Nintendo DSi 浏览器中也能像适当的循环一样快，但我也想减少代码。你有什么建议？

先感谢您！

Answer 1

为什么不直接将循环体放入函数中呢？

function doSomething(i, arr) {
    function1(function2(arr[i],i), $('div'+i));
    $('span'+i).value = function3(arr[i]);
}

并在循环中调用它：

function loop() {
    for (var i = 0; i <= 5; i++) { doSomething(i, arr); }
}

Answer 2

嘎！

这是一个很好的问题，但是不，永远不要这样做。一般来说，不建议使用 eval ，因为您不会在加载时看到解析错误，只会在运行时看到解析错误（更难调试），更难理解作用域中的内容（更难编写） ，并且您将失去所有工具链支持（语法突出显示、脚本调试）。

幸运的是，既然 Javascript 基本上是一种函数式语言，为什么不创建一个函数来封装你想要做的事情，然后调用它呢？

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

Answer 3

这是一个可怕的想法。

• 效率低下
• 调试起来比较困难

如果您担心带宽，请使用 缩小 和 HTTP压缩。

Answer 4

呃，不。 eval 应被视为接近最后的手段。 JavaScript 函数是第一类对象，所以我只需声明你需要的任何函数并传递它们作为参数之一。

Answer 5

为什么不：

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

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