1

私はJavascriptにかなり慣れていないので、単純なフォトビューアー、スライドショーアプリケーションを構築しようとしています. おそらく、まだ知らないコードのエラー/間違ったプラクティスがあります。スライドショー ボタンのイベントが発生し、コンソールで出力を確認できますが、ランダム スライド ショー ボタンのイベントは発生しません。

HTML5 スニペット

<form>
   <div id="controls">
       <input type="button" id="slideshow" value="Slide Show" />
       <input type="button" id="randomSlideshow" value="Random Slide Show" />
   </div>
</form>
<script src="js/PhotoViewer.js"></script>
</body>

JS スニペット

var photosArrayGlobal = new Array();
var photoIndexGlobal = 0;
var displayGlobal;

window.onload = main;

function main() {
  "use strict";
  document.getElementById("slideshow").onclick = getArrayPhotosNames;

  document.getElementById("randomSlideshow").onclick = randomize(photosArrayGlobal);

  displayGlobal = document.getElementById("myImage");

  document.getElementById("nextSlide").onclick = function () {
    displayGlobal.setAttribute("src", photosArrayGlobal[1]); //Test value, image 2
  };
}

function getArrayPhotosNames() {
  var folderName = document.getElementById("photoFolder").value;
  var commonName = document.getElementById("commonName").value;
  var startNum = document.getElementById("startNum").value;
  var endNum = document.getElementById("endNum").value;
  var j = 0;

  if (startNum > endNum) {
    alert("Invalid Numbers");
  }

  var nameArray = new Array();

  for (var i = startNum; i <= endNum; i++) {
    nameArray[j] = folderName + commonName + i + ".jpg";
    j++;
  }

  photosArrayGlobal = nameArray.slice();
   console.log(photosArrayGlobal);

  return nameArray;
}

function randomize(dataArray) {
  var i = dataArray.length;
  var j, tempi, tempj;
  if (i === 0) {
    return false;
  }

  while (--i) {
    j = Math.floor(Math.random() * (i + 1));
    tempi = dataArray[i];
    tempj = dataArray[j];
    dataArray[i] = tempj;
    dataArray[j] = tempi;
  }
 console.log(dataArray);

}
4

1 に答える 1

1

ハンドラーは関数を期待していますが、関数から返された値 (たまたま)onclickを渡しています。次のように変更します。randomize()undefined

document.getElementById("randomSlideshow").onclick = function() {
    randomize(photosArrayGlobal);
};
于 2013-01-09T22:18:53.247 に答える