Question

我一直在用JavaScript制作幻灯片，但我似乎无法让它发挥作用。我认为问题的一部分可能是我对外部.js文件的引用...我不确定我是否正在使用href正确执行此操作。但是，我已经尝试将脚本放在我的HTML页面上并使用href:ecoquiet.html代替，但这也不起作用。

我的HTML：

<img class=lawnslide src="lawn.jpg" name="slideshow">
     <table>
          <tr>
               <td align=left><a href=ecoquiet.js onclick="return change_image(-1)">Previous</a></td>
               <td align=right><a href="ecoquiet:change_image(1)">Next</a></td>
               <td align=right><a href="ecoquiet:auto()">Auto</a></td>
           </tr>
     </table>

我的JavaScript：

function change_image (num) {

var image = new Array("lawn.jpg", "nicelawn.jpg", "eco.png");
var description = new Array("100% Organic", "Gasoline-Free", "75% Quieter");

var = Image_Number = 0;
var = Image_Length = Image.length - 1;

Image_Number = Image_Number + num;

if (Image_Number > Image_Length) {

    Image_Number = 0;        
}

if (Image_Number < 0) {

    Image_Number = Image_Length;
}

document.slideshow.src=Image[Image_Number];
document.getElementById("decription").innerHTML = description[Image_Number];

return false;

}

function auto () {

    setInterval("change_image(1)", 3000);

}

Answer 1

首先，您需要在文档的头部引用带有脚本标记的外部JS文件，如下所示：

<head>
    <script type="text/javascript" src="ecoquiet.js"></script>
</head>

其次，从该文件调用函数时，不需要引用文件名。

 <table>
      <tr>
           <td align="left"><a href="javascript:change_image(-1); return false;">Previous</a></td>
           <td align="right"><a href="javascript:change_image(1); return false;">Next</a></td>
           <td align="right"><a href="javascript:auto(); return false;">Auto</a></td>
       </tr>
 </table>

return false将阻止链接执行链接的正常行为（将您带到其他地方）。

Answer 2

如果要添加外部js文件，请将其添加到标题部分

    <script type="text/javascript" src="ecoquiet.js"></script>

你错过了几件事。更好的我们你的浏览器控制台来识别错误。你的html中没有一个元素叫做decription.It会出错。

document.getElementById("decription").innerHTML

你在var之后签了=，这会给出错误

var = Image_Number = 0;
var = Image_Length = Image.length - 1;

还有一个拼写错误。你把图像数组与Image

混淆了

var = Image_Length = Image.length - 1;// should be small case letter

应该是image.length。在全局范围外定义Image_Number变量，除非用封闭包装它。

调用setInterval时不要直接在函数上使用参数。用另一个函数包装它

function auto () {

    setInterval(function(){
       change_image(1);


    }, 3000);

}

不要在锚标记的href属性中添加任何外部链接。给它一个哈希字符串作为href，所以它不会将你重定向到另一个页面。

<td align=left><a href='#' onclick="change_image(-1);">Previous</a></td>
<td align=right><a href="#" onclick="change_image(1);">Next</a></td>
<td align=right><a href="#" onclick="auto();">Auto</a></td>

以下是我对您的代码所做的更改：

var  Image_Number = 0;
function change_image (num) {

var image = new Array("lawn.jpg", "nicelawn.jpg", "eco.png");
var description = new Array("100% Organic", "Gasoline-Free", "75% Quieter");


var  Image_Length = image.length - 1;
console.log(Image_Length)

Image_Number = Image_Number + num;
console.log(Image_Number);

if (Image_Number > Image_Length) {

    Image_Number = 0;        
}

if (Image_Number < 0) {

    Image_Number = Image_Length;
}

document.slideshow.src=image[Image_Number];
document.getElementById("description").innerHTML = description[Image_Number];

return false;

}

function auto () {

    setInterval(function(){
       change_image(1);


    }, 3000);

}

我的JavaScript幻灯片显示有什么错误？

2 个答案: