0

我有小问题。我正在学习 javascript，我想使用switch.

所以我创建了这个html代码

<body onload="rndqu()">
    <div id="head"> <a href="index.html">Mira's place<a><br>
            <h2>&#8220;<span id="quote"></span>&#8221;</h2>
    </div>
</body>

并使用了这个 Javascript

var qu;
var slogan;
function rndqu(n){
    var random = function(min, max) {
    return Math.floor(Math.random() * (max - min + 1)) + min;
    };  
    qu = random(1, 3);
}
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;

我不明白为什么它不起作用。有人能帮我吗？谢谢！这是它的jsfiddle http://jsfiddle.net/NX3cz/

4

3 回答 3

4

我会为此使用数组而不是 switch 语句，以使其更灵活。例如：

var quotesList = ["I'm a great guy!", "Not even kidding.", "Just incredible."];

var randQuote = function(quotes) {
    var choice = Math.floor(Math.random() * quotes.length);
    return quotes[choice];
}

document.getElementById("quote").innerHTML = randQuote(quotesList);

这样，报价数组的大小可以自由更改，而无需更改任何代码。

演示：jsfiddle

于 2013-03-29T18:37:21.130 回答
2

您将部分代码留在了rndqu()函数之外。我在这里分叉并纠正了你的小提琴：http: //jsfiddle.net/BwJ7s/

这是更正后的 JS 代码：

var qu;
var slogan;
function rndqu(n)
{
    var random = function(min, max) {
    return Math.floor(Math.random() * (max - min + 1)) + min;
    };  
    qu = random(1, 3);

    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;
}
于 2013-03-29T18:38:00.543 回答
0

在这里修复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);

您的代码过于复杂也请注意，在 jsfiddle 中不要添加body onload="()"函数。Jsfiddle 为您做到这一点。

如果您想onbodyload在真实网页上执行此操作，请将您的代码包装在以下内容中：

window.onload = (function(){
    //your code goes here
})

或将您的脚本包含在 html 文件的底部。

最好的做法是尽可能避免在您的 html 中使用内联 javascript。

于 2013-03-29T18:48:11.033 回答