0

我有从提要加载数据的代码。您可以在此处查看 jsfiddle 。

<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).

对于每个标签，我希望它使用每种样式（normal and bold如 jsfiddle 中的示例）。但我不知道为什么它只从Label.

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

4

1 回答 1

6

这里有几个问题。

控制结构中的函数声明

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

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

您可以非常简单地区分函数声明和函数表达式：如果您立即使用构造的值，则它是一个表达式。

函数声明：

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在现代网页或 Web 应用程序中几乎没有位置。创建一个script节点并将其附加到 DOM。

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

您生成脚本标记的代码会将标签的内容附加到查询字符串中，但未对其进行正确编码。虽然这适用于您现在标签数组中的特定值，但如果您更改标签数组值，它将中断。而是使用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（小部件）。

那是因为您的 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);
}

我建议对DOM进行一些阅读，并且可能使用一个好的库来使这些东西变得更容易——jQuery、YUI、Closure或其他几个。

于 2013-10-16T13:06:45.550 回答