它似乎只适用于其他所有图片,而不是所有其他图片
这是破坏性迭代的经典标志。
考虑一下,如果我猜该函数
cvi_instant.add用
pic其他一个或多个其他元素替换命名的元素,会发生什么情况。
getElementsByName返回一个“活动的”
NodeList:每次您对DOM进行更改时,它都会保持最新状态。因此,如果它之前有五个元素,则在调用
cvi_instant.add它之后现在仅包含四个元素:第一个节点消失了,而节点1-4已向下移动到位置0-3。
现在,您再次循环。
i++,所以我们看的是元素1。但是元素1现在原来是元素2!我们跳过了原始元素1,然后将继续跳过所有其他元素,直到到达列表的末尾(现在是长度的一半)。
在迭代列表的同时更改列表会导致这种问题。如果迭代中的过程实际上 将 元素 添加 到列表中,您甚至可以得到无限循环!
快速解决方案是向后迭代循环。现在,您首先执行最后一个元素,将所有其他元素保留在其原始位置,并且不会跳过:
var e= document.getElementsByName("pic"); for (var i= e.length; i-->0;) { cvi_instant.add(e[i], { shadow: 75, shade: 10 }); }如果您 知道 总是要在每次调用的列表中删除元素,则另一个简单的解决方案是:
var e= document.getElementsByName("pic"); while (e.length>0) { cvi_instant.add(e[0], { shadow: 75, shade: 10 }); }当循环主体可以对列表执行 任何操作时
,例如插入
pic在文档开头命名的新元素或从中间删除其他元素,则需要最通用的解决方案。稍稍慢一些,但可以安全地创建列表的静态副本以进行操作:
function Array_fromList(l) { var a= []; for (var i= 0; i<l.length; i++) a.push(l[i]); return a; } var e= Array_fromList(document.getElementsByName("pic")); for (var i= 0; i<e.length; i++) { cvi_instant.add(e[i], { shadow: 75, shade: 10 }); }


