JS图片轮播效果出不了，也不知道哪错，麻烦大神们看一下

<!doctype html>
<html>
<head>
<link rel="stylesheet" type="text/css" href="../CSS/index.css">
<script src="../JavaScript/index.js" type="text/javascript"></script>
<meta charset="utf-8">
<title>index</title>
</head>
<body>
<div class="banner">
<div class="banner_pic" id="banner_pic">
<div class="current"><img src="../Images/p4.jpg" alt="" /></div>
<div class="pic"><img src="../Images/p5.jpg" alt="" /></div>
<div class="pic"><img src="../Images/p6.jpg" alt="" /></div>
<div class="pic"><img src="../Images/p7.jpg" alt="" /></div>
</div>
<ol id="button">
<li class="current">1</li>
<li class="but">2</li>
<li class="but">3</li>
<li class="but">4</li>
</ol>
</div>
</body> 这是我截取源代码的部分var current_index=0;
var timer=window.setInterval(autoChange, 3000);
var button_li=document.getElementById("button").getElementsByTagName("li");
var pic_div=document.getElementById("banner_pic").getElementsByTagName("div");
for(var i=0;i<button_li.length;i++){
button_li[i].onmouseover=function(){
if(timer){
clearInterval(timer);
}
for(var j=0;j<pic_div.length;j++){
if(button_li[j]==this){
current_index=j;
button_li[j].className="current";
pic_div[j].className="current";
}
else{
pic_div[j].className="pic";
button_li[j].className="but";
}
}
}
button_li[i].onmouseout=function(){
timer=setInterval(autoChange, 3000);
}
}
function autoChange(){
++current_index;
if(current_index==button_li.length){
current_index=0;
}
for(var i=0;i<button_li.length;i++){
if(i==current_index){
button_li[i].className="current";
pic_div[i].className="current";
}
else{
button_li[i].className="but";
pic_div[i].className="pic";
}
}
}
有时候在Dreamweaver里的预览可以轮播，有时候不行，但用浏览器看的话，根本轮播不了，我也不知道为什么
还提示
window is not defined
document is not defined

解决方案 »

  1.   

    script标签放body底部
      

  2.   

    是〈body〉下一行吗
      

  3.   

    是〈body〉下一行吗
      

  4.   

    是〈body〉下一行吗
    </body>上一行
      

  5.   

    是〈body〉下一行吗
    </body>上一行
    卧槽，竟然好了，非常感谢~~~，为什么要放到最后
      

  6.   

    为什么要放到最后，你得知道html文档的加载顺序了，你这个放的也太往上了，最起码你不能放到meta上面去呀，最次也得放到</head>上一层，像引入的那种库(jquery等)，个人写的最好要放到</body>上一层，之所以放到这里，是因为要等html的DOM内容加载，渲染完毕，然后再执行你的js。另外看代码，楼主写的li的点击事件有问题，闭包带来的问题，你点击li并不会有理想的效果才对。