0

なぜ私のコードが機能しないのか疑問に思っています。私はこれを完全に間違っていますか？

window.onload = function random() {
    var links = document.getElementsByTagName('a');
    var colours = new Array("green","red","blue");
    var randomColour = colours[Math.floor(colours.length * Math.random())];
    for (a = 0; a < links.length; a++) {
        links[a].style.color = 'randomColour';
    }
}
4

1 に答える 1

1

'randomColor'-> randomColor.
引用符で囲まれた文字列の代わりに変数を使用してください。また、各リンクがランダムな色になるようにするには、ループ内で色を生成します。

for (var a = 0; a < links.length; a++) {
    var randomColour = colours[Math.floor(colours.length * Math.random())];
    links[a].style.color = randomColour;
}

色の設定とリンクの解析の間の遅延を最小限に抑えるには、DOMContentLoadedイベントをバインドするか、単純にコードを の最後に貼り付けます<body>。

http://jsfiddle.net/V6Chb/1/

<a>Test</a>
  ... 
<a>Test</a>
<script>
(function() { // <-- Anonymous function to not leak variables to the global scope
    var links = document.getElementsByTagName('a');
    var colours = ["green", "red", "blue"];
    for (var a = 0; a < links.length; a++) {
        var randomColour = colours[Math.floor(colours.length * Math.random())];
        links[a].style.color = randomColour;
    }
})();
于 2012-04-15T16:22:33.773 に答える