1

我正在尝试创建一些“插件”，例如 facebook like 按钮。但是我的循环只呈现第一个 div。而不是其他div。为什么？

JSFiddle

    <style>body{background: #ccc;}</style>

<div name="q" data-id="3"  data-width="200"></div>
<div name="q" data-id="1"  data-width="300"></div>
<div name="q" data-id="1"  data-width="400"></div>

<script>

    var s = document.getElementsByName("q");

    for (var i = 0; i < s.length; i++) {

        e = s[i];

        w = e.attributes['data-width'].value; 
        i = e.attributes['data-id'].value;

        var o={};

        o.iframe = document.createElement('iframe');
        o.iframe.setAttribute('src', 'http://resources2.news.com.au/images/2012/09/27/1226482/758034-tardar-sauce-the-cat.jpg');
        o.iframe.width = w;
        o.iframe.height = w + 100;
        o.iframe.border = 0;
        o.iframe.setAttribute('style', 'border: 0;overflow: visible;');

        e.appendChild(o.iframe);

        console.log('id:'+i+' width:'+w);
    }   

</script>
4

2 回答 2

2

您将变量i用作循环计数器和循环内部。

在第一次迭代中，您将值分配给"3"变量i。3在语句中进行比较时，它将转换为数字for，并且由于大于数组的长度，因此循环在第一次迭代后结束。

为在循环中保存 data-id 的变量使用不同的变量名。

编辑：

iframe 大小的问题在这里：

o.iframe.height = w + 100;

该变量w包含一个字符串，例如"200"，并且当运算符的任何操作数+是字符串时，它将进行字符串连接。所以结果不是300但是"200100"。您的 iframe 有几千像素高。

您需要在计算中使用数字：

o.iframe.height = parseInt(w) + 100;
于 2013-10-06T16:01:28.143 回答
1

i正在您的循环中进行修改。在循环中更改此声明的名称：

newNameForVar = e.attributes['data-id'].value;
于 2013-10-06T16:00:52.037 回答