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 がそれを行います。

実際の Web ページで実行する場合はonbodyload、コードを次のようにラップします。

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

または、html ファイルの下部にスクリプトを含めます。

可能な限りベスト プラクティスは、HTML でインライン JavaScript を使用しないことです。

于 2013-03-29T18:48:11.033 に答える