Home/ Questions/Q 7810633
In Process

The Archive Base Latest Questions

Editorial Team
  • 0
Asked: 2026-06-02T03:51:18+00:00

Just wondering why my code isn’t working. Am I doing this completely wrong? window.onload

  • 0

Just wondering why my code isn’t working. Am I doing this completely wrong?

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';
    }
}
Share

Leave an answer

You must login to add an answer.

Need An Account,

1 Answer

  1. Editorial Team

    'randomColor' -> randomColor.
    Use a variable instead of a quoted string. Also, to ensure that each link gets a random color, generate the color in the loop:

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

    To minimize the delay between setting the color, and parsing the link, bind a DOMContentLoaded event, or simply paste the code at the end of the <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;
    }
})();
    • 0