Question

新闻图片切换的效果，JS有问题 急求

<DIV align="center">
<IMG src="images/news1.jpg" style="display:none;"id="ad1">
<IMG src="/images/news2.jpg" style="display:none;"id="ad2">
<IMG src="images/news3.jpg" style="display:none;"id="ad3">
<IMG src="images/news4.png" style="display:none;"id="ad4">
</DIV>
<SCRIPT language="javascript">
var NowFrame=1; //全局变量，轮换显示图片的第一张
var MaxFrame=4; //全局变量，轮换显示图片的最大张数
function adv(){
for(var i=1;i<=MaxFrame;i++){
if(i==NowFrame)
document.getElementById('ad'+NowFrame).style.display=''; //目前显示的图片
else
document.getElementById('ad'+i).style.display='none'; //隐藏其他图片
}
{ if(NowFrame==MaxFrame) //设置下一张显示的图片
NowFrame=1;
else
NowFrame=NowFrame+1; }
setTimeout('adv()',2000); //设置定时器，显示下一张图片
}
window.onLoad=adv(); //当页面载入时，调用 adv()函数
</SCRIPT>

Answer 1

window.onLoad=adv(); //当页面载入时，调用 adv()函数

要写成：

window.onLoad=adv;

Answer 2

setTimeout('adv()',2000);
上面的改成 setTimeout('adv',2000);

Answer 3

虽然不是一个很好的实现，但代码也没什么问题啊，能正常工作

追问

看不出错 就是没效果 所有图片从上往下排开

追答

你看页面上有没有报错，估计是其它地方的js影响了你的代码，同时你也可以通过添加div样式(style="height:xxxpx;overflow:hidden")来隐藏掉其它的图片,xxx为你图片的高度

Answer 4

我试了一下你的代码，没有问题。不知道你报的什么错误？

追问

所有的img从上到下排列 没有效果 不知道哪出错了

追答

我还是没有太懂你的意思。
你现在的效果是4个img切换的时候不是在一个地方切换，而是换行显示了吗？

追问

恩 效果没有出来 换行显示了

追答

不会吧，在我这运行就是在一个地方切换。
从你现在的代码看不可能换行显示。
那是不是有什么css影响到你的显示了？