在For循环问题中的功能

Question

我有从代码加载数据的代码。您可以查看jsfiddle here。

<div class="agenda"></div>
<div class="agenda"></div>

<script>
label = ["Tutorials", "Widgets"];
for (m = 0; m < label.length; m++) {
    function listEvents(json) {
        var feed = json.feed;
        for (var i = 0; i < 10; i++) {
            var entry = json.feed.entry[i];
            var posttitle = entry.title.$t;

            //Style
            if (i == 0) {
                x = "<span style='color:red'>Style 1 + Label 1</span><p>" + posttitle + "</p>";
                p = "<span style='color:red'>Style 2 + Label 2</span><h4>" + posttitle + "</h4><p>";
            }
            if (i == 1) {
                y = "<p>" + posttitle + "</p>";
                g = "<h4>" + posttitle + "</h4></p>";
            }
        }

        style = [x + y, p + g];
        for (m = 0; m < style.length; m++) {
            document.getElementsByClassName("agenda")[m].innerHTML = style[m];
        }

    }

    document.write("<script src=\"http://www.allbloggertricks.com/feeds/posts/default/-/" + label[m] + "?orderby=published&alt=json-in-script&callback=listEvents\"><\/script>");
}
</script>

我使用For循环从2 Labels (Tutorials, Widgets)加载数据。

对于每个Label，我希望它使用每种样式（normal and bold作为jsfiddle中的示例）。但我不知道为什么它只从Label加载数据。

我需要你的帮助。谢谢你的所有建议。

Answer 1

这里有几个问题。

控制结构中的函数声明

函数声明（你的listEvents函数有什么用）在任何控制结构中都是无效的。它们必须出现在全局范围或功能范围的顶部级别，而不是在循环，条件等中。某些引擎在这些结构中容忍它们，猜测是什么你的意思是，但其他人则更严格。

另一方面，函数表达式可以出现在任何表达式出现的任何地方。 （通常最好不要把它们放在循环中。）

您可以非常简单地分辨函数声明和函数表达式之间的区别：如果您正在使用构造的值，那么它就是一个表达式。< / p>

功能声明：

function foo() { /* ... */ }

函数表达式：

var f = function() { /* ... */ };
var nf = function foo() { /* ... */ }; // Avoid this on old versions of IE
doSomething(function() { /* ... */ });

重用相同的功能名称

您的代码正在尝试将loadEvents设置为JSONP回调。您不需要拥有loadEvents的多个副本，只需要一个 - 并且由于JSONP的工作方式，必须是一个全局函数。

确保loadEvents内的逻辑处理不止一次，因为每个JSONP回调都会被触发。

覆盖m

您正在使用该函数覆盖m变量的值。它拥有它的方式是无害的，但它仍然是一个坏主意。

使用document.write

document.write在现代网页或网络应用程序中几乎没有任何地位。创建一个script节点，然后将其附加到DOM。

在URL查询字符串中使用原始字符串值

生成脚本标记的代码会将标记的内容附加到查询字符串，而不对其进行正确编码。虽然这适用于标签数组 now 中的特定值，但如果更改标签数组值，它将会中断。使用encodeURIComponent对其进行编码。

---

这是一个最小的更新：

var m, label, parent; // Declare variables!
label = ["Tutorials", "Widgets"];
parent = document.getElementsByTagName('script')[0].parentNode;
for (m = 0; m < label.length; m++) {

    var script = document.createElement('script');
    script.src = "http://www.allbloggertricks.com/feeds/posts/default/-/"
                 + encodeURIComponent(label[m])
                 + "?orderby=published&alt=json-in-script&callback=listEvents";
    parent.appendChild(script);
}

function listEvents(json) {
    var m;
    var feed = json.feed;
    for (var i = 0; i < 10; i++) {
        var entry = json.feed.entry[i];
        var posttitle = entry.title.$t;

        //Style
        if (i == 0) {
            x = "<span style='color:red'>Style 1 + Label 1</span><p>" + posttitle + "</p>";
            p = "<span style='color:red'>Style 2 + Label 2</span><h4>" + posttitle + "</h4><p>";
        }
        if (i == 1) {
            y = "<p>" + posttitle + "</p>";
            g = "<h4>" + posttitle + "</h4></p>";
        }
    }

    style = [x + y, p + g];
    for (m = 0; m < style.length; m++) {
        document.getElementsByClassName("agenda")[m].innerHTML = style[m];
    }

}

---

重新评论以下内容：

  

但它仍然只加载来自一个“标签”的数据。我想要每种风格，它加载一个标签。 Style1（普通文本）加载Label1（教程），style2（粗体文本）加载Label2（Widgets）。

那是因为你的loadEvents会覆盖议程元素的内容。您可能最好将动态添加到某个容器中，而不是在标记中包含议程元素。例如：

<div id="agendas"></div>

然后改变这个：

for (m = 0; m < style.length; m++) {
    document.getElementsByClassName("agenda")[m].innerHTML = style[m];
}

到此：

var agendas = document.getElementById("agendas");
for (m = 0; m < style.length; m++) {
    var agenda = document.createElement('div');
    agenda.className = "agenda"; // If you still need this for anything
    agenda.innerHTML = style[m];
    agendas.appendChild(agenda);
}

我建议您在the DOM上阅读，并可能使用一个好的库来简化这些工作 - jQuery，YUI，Closure或{{3} }。