Question

我正在尝试创建一个页面，允许观看者在每次用户点击时查看新图像。图像命名为0.jpg，1.jpg，2.jpg，3.jpg和4.jpg。

但是，图片网址在点击时不会改变。我还尝试在Chrome Developer工具中分析图像，控制台不会发出任何错误。那是为什么？

这是我的javascript。

var imageCount = 5;
var currentIndex = Math.floor(Math.random() * imageCount);

  function loadNewPhoto() {
    currentIndex = (currentIndex + (Math.random() * 3) + 1) % imageCount;
    alert(currentIndex);
    document.getElementById('generatedImage').src = 'images/' + currentIndex + '.jpg';
  }

  // Shuffle order of photos.
  document.onready = function() {
    loadNewPhoto();
    document.getElementById('getMeNewPhotoLink').onclick(function(e) {
      e.preventDefault();
      loadNewPhoto();
    });
  }

和HTML

<a href="#" id="getMeNewPhotoLink">
    <img id="generatedImage" src="images/0.jpg" alt="" />
</a>

http://jsfiddle.net/qLQZf/

Answer 1

在这里工作的是你的DEMO

新HTML：

<a href="javascript:shuffle()" id="getMeNewPhotoLink" >
    <img id="generatedImage" src="images/0.jpg" alt="" />
</a>

href现在是一个javascript函数，而不是"#"，这是一个更优雅的解决方案

新Javascript：

var imageCount = 5;
var currentIndex = Math.floor(Math.random() * imageCount);
alert(currentindex);
  function loadNewPhoto() {
    currentIndex = Math.floor((currentIndex + (Math.random() * 3) + 1) % imageCount);//Added Math.floor, else was generating decimal value
    alert(currentIndex);
    document.getElementById('generatedImage').src = 'images/' + currentIndex + '.jpg';
  }

  // Shuffle order of photos, no need of preventdefault, onclick and ready event handling via code.

  function shuffle() {
     loadNewPhoto();
  }

Answer 2

在原始代码中，您错误地分配了onclick处理程序。它应该是：

document.getElementById('getMeNewPhotoLink').onclick = function(e) {
  e.preventDefault();
  loadNewPhoto();
};

为什么我不能使用javascript显示新图像？

2 个答案: