I have got little problem. I\'m learning javascript and I wanted to create random slogan generator for my site with using switch
.
So I created this html
Fixed here http://jsfiddle.net/NX3cz/13/
var qu;
var slogan;
function rndqu(min, max){
qu = Math.floor(Math.random() * (max - min + 1)) + min;
switch(qu){
case 1:
slogan = "Here is the 1";
break;
case 2:
slogan = "Here is the 2";
break;
case 3:
slogan = "Woah";
break;
default:
slogan = "Really?";
}
document.getElementById("quote").innerHTML = slogan;
}
rndqu(1, 3);
Your code was overly complicated also notice, in jsfiddle don't add a body onload="()"
function. Jsfiddle does that for you.
If you want to do it onbodyload
on a real webpage wrap your code in this:
window.onload = (function(){
//your code goes here
})
or include your script at the bottom of the html file.
Whenever possible best practice is to avoid inline javascript in your html.