错误 - 不要在循环中创建函数 [关闭]

Question

我正在尝试替换电子邮件模板中的占位符，以下是我的代码：-

我收到了error : Don't make functions within a loop

var dataPlaceholders = [{
            "username":"John Johny",
            "website":"w3schools . com"
            }];

template_html = "<b>Hello <%= username %>,</b><br/><br/> You are successfuly Registered on xyz.<br/><br/>Thank you <%= website %>";

函数调用：-

function ReplacePlaceholders(dataPlaceholders, template_html){
    for (var i = 0; i < dataPlaceholders.length; i++) {
        var obj = dataPlaceholders[i];
        template_html += "" + template_html.replace("/<%=%>/g", function (match, property) {
            return obj[property];
        }) + "";

    }
    return template_html;
}

感谢您的帮助。

Answer 1

来自JSLint Error Explanations：

这里的基本问题是 JavaScript 解释器将在每次循环迭代时创建一个函数实例。它必须这样做，因为它不知道函数对象是否会在其他地方被修改。由于函数是标准的 JavaScript 对象，它们可以像任何其他对象一样具有可以在循环中更改的属性。因此，通过在循环上下文中创建函数，您会导致解释器创建多个函数实例，这可能会导致意外行为和性能问题。

Answer 2

这个错误是因为你在循环中创建了一个函数变量：function (match, property)

这意味着如果您的循环中有 100 个元素，它将创建 100 个匿名函数，这对性能不利。

要解决此问题，请将函数声明移到 for 循环之外。所以你的代码变成了这样

function ReplacePlaceholders(dataPlaceholders, template_html){
    // here we create a single instance of your match function
    var match = function (match, property) {
        return obj[property];
    }

    for (var i = 0; i < dataPlaceholders.length; i++) {
        var obj = dataPlaceholders[i];

        // now we use our single declared instance inside the for loop
        template_html += "" + template_html.replace("/<%=%>/g", match) + "";
    }
    return template_html;
}