javascript横幅冻结在第一条横幅上

我最近在我的网站上添加了两个横幅，我想每5秒更改一次， 不幸的是，它只显示第一个并冻结

这是我的foreach数据的全部代码

var-links=[”http://site", "http://site"];
var images=[“/uploads/ad1.png”、“/uploads/ad2.png”]；
var i=0；
var renew=setInterval（函数（）{
如果（links.length==i）{
i=0；
}否则{
document.getElementById（“bannerImage”）.src=images[i]；
document.getElementById（“bannerLink”）.href=links[i]；
i++；
}
}, 500);
此处的代码将每5秒更改一次图像和链接


var-links=[”http://site","http://site"];
变量图像=[”https://images.pexels.com/photos/414612/pexels-photo-414612.jpeg?auto=compress&cs=tinysrgb&dpr=1&w=500","https://cdn.pixabay.com/photo/2015/04/23/22/00/tree-736885__340.jpg"];
var i=0；
var renew=setInterval（函数（）{
document.getElementById（“bannerImage”）.src=images[i]；
document.getElementById（“bannerLink”）.href=links[i]；
i++；
if（i==links.length）{
i=0；
}
}, 5000);

div
元素没有加载事件<代码>窗口执行。图像可以。如果是的话。其他媒体元素最有可能做到这一点<代码>div
s不允许。如果您将console.log放在您试图加载运行的方法中，您将看到它没有执行。感谢@Taplar，我成功地完成了这项工作，但我在php foreach中有html代码，并且标题在第一个foreach数据中更改，而在第二个数据中没有更改
php
在服务器上运行，而不是在客户端上运行。因此，我不确定服务器端的任何循环将如何影响您试图在客户端实现的循环。除非您说php循环生成重复的html元素，否则只有一个可以使用。如果是这样的话，请您更新您的问题以显示重复的标记，并提供您对原始代码所做的任何更改。@Taplar我更新了我使用的代码，它正在工作，但仅在foreach数据的第一行谢谢，但它是相同的，它仅在foreach数据的第一行更改！“foreach数据的第一行”是什么意思？它应该使用提供的所有数据。你能给出一个例子或一种复制问题的方法吗？