2016-05-29 6 views
-2

Dieser Code funktioniert nicht so, wie ich es gerne getan hätte, aber ich bin mir des Problems nicht sicher.Dieser JavaScript-Code funktioniert nicht. Habe ich die Syntax falsch oder ist es etwas anderes?


Code:

<body> 
    var randNumForQuote = Math.floor((Math.random() * 11)); 

    if (randNumForQuote == 0) { 
    document.getElementById("quoteDiv").innerHTML = "Hello"; 
    } else if (randNumForQuote == 1) { 
    document.getElementById("quoteDiv").innerHTML = "Hello1"; 
    } else if (randNumForQuote == 2) { 
    document.getElementById("quoteDiv").innerHTML = "Hello2"; 
    } else if (randNumForQuote == 3) { 
    document.getElementById("quoteDiv").innerHTML = "Hello3"; 
    } else if (randNumForQuote == 4) { 
    document.getElementById("quoteDiv").innerHTML = "Hello4"; 
    } else if (randNumForQuote == 5) { 
    document.getElementById("quoteDiv").innerHTML = "Hello5"; 
    } else if (randNumForQuote == 6) { 
    document.getElementById("quoteDiv").innerHTML = "Hello6"; 
    } else if (randNumForQuote == 7) { 
    document.getElementById("quoteDiv").innerHTML = "Hello7"; 
    } else if (randNumForQuote == 8) { 
    document.getElementById("quoteDiv").innerHTML = "Hello8"; 
    } else if (randNumForQuote == 9) { 
    document.getElementById("quoteDiv").innerHTML = "Hello9"; 
    } else if (randNumForQuote == 10) { 
    document.getElementById("quoteDiv").innerHTML = "Hello10"; 
    } 

    <div id="quoteDiv"></div> 
</body> 


Ich habe wenig Erfahrung in JavaScript, verstehe also nicht, das Problem zu gut.

ich die div erwarte einen der Ausgänge zu sagen (zB. Hello, Hello1, Hello2, etc.)

+0

Sie haben vergessen, uns zu sagen, was Sie erwarten .. Dieser Code funktioniert, wie es geschrieben wird .. – Rayon

+0

Dieser Code ist für mich gut funktioniert. Es ist besser, die Anweisung 'switch' zu verwenden, wenn Sie mehrere Bedingungen haben. – sabith

+0

Sorry, wird hinzugefügt –

Antwort

1

Sie müssen den JavaScript-Code innerhalb script Tag legen und den Code auszuführen, nachdem Seite so innerhalb window.onload Rückruf setzen geladen wird, welches am Ende des Ladevorgangs des Dokuments ausgelöst wird.

<body> 
 
    <script> 
 
    window.onload = function() { 
 
     var randNumForQuote = Math.floor((Math.random() * 11)); 
 

 
     if (randNumForQuote == 0) { 
 
     document.getElementById("quoteDiv").innerHTML = "Hello"; 
 
     } else if (randNumForQuote == 1) { 
 
     document.getElementById("quoteDiv").innerHTML = "Hello1"; 
 
     } else if (randNumForQuote == 2) { 
 
     document.getElementById("quoteDiv").innerHTML = "Hello2"; 
 
     } else if (randNumForQuote == 3) { 
 
     document.getElementById("quoteDiv").innerHTML = "Hello3"; 
 
     } else if (randNumForQuote == 4) { 
 
     document.getElementById("quoteDiv").innerHTML = "Hello4"; 
 
     } else if (randNumForQuote == 5) { 
 
     document.getElementById("quoteDiv").innerHTML = "Hello5"; 
 
     } else if (randNumForQuote == 6) { 
 
     document.getElementById("quoteDiv").innerHTML = "Hello6"; 
 
     } else if (randNumForQuote == 7) { 
 
     document.getElementById("quoteDiv").innerHTML = "Hello7"; 
 
     } else if (randNumForQuote == 8) { 
 
     document.getElementById("quoteDiv").innerHTML = "Hello8"; 
 
     } else if (randNumForQuote == 9) { 
 
     document.getElementById("quoteDiv").innerHTML = "Hello9"; 
 
     } else if (randNumForQuote == 10) { 
 
     document.getElementById("quoteDiv").innerHTML = "Hello10"; 
 
     } 
 
    } 
 
    </script> 
 

 
    <div id="quoteDiv"></div> 
 
</body>


Obwohl Sie ca Code

<body> 
 
    <script> 
 
    window.onload = function() { 
 
     var randNumForQuote = Math.floor((Math.random() * 11)); 
 
     document.getElementById("quoteDiv").innerHTML = "Hello" + (randNumForQuote ? " " + randNumForQuote : ''); 
 
    } 
 
    </script> 
 

 
    <div id="quoteDiv"></div> 
 
</body>

+0

Vielen Dank @Pranav C Balan, das hat einwandfrei funktioniert –

+0

@TheEngineer: froh zu helfen –

0

Javascript muss reduzieren in einem Script-Tag sein, wenn es in html ist, und Sie müssen die haben div kommen vor dem Skript-Tag.

<body> 
 
    <div id="quoteDiv"></div> 
 
<script> 
 
    var randNumForQuote = Math.floor((Math.random() * 11)); 
 

 
    if (randNumForQuote == 0) { 
 
    document.getElementById("quoteDiv").innerHTML = "Hello"; 
 
    } else if (randNumForQuote == 1) { 
 
    document.getElementById("quoteDiv").innerHTML = "Hello1"; 
 
    } else if (randNumForQuote == 2) { 
 
    document.getElementById("quoteDiv").innerHTML = "Hello2"; 
 
    } else if (randNumForQuote == 3) { 
 
    document.getElementById("quoteDiv").innerHTML = "Hello3"; 
 
    } else if (randNumForQuote == 4) { 
 
    document.getElementById("quoteDiv").innerHTML = "Hello4"; 
 
    } else if (randNumForQuote == 5) { 
 
    document.getElementById("quoteDiv").innerHTML = "Hello5"; 
 
    } else if (randNumForQuote == 6) { 
 
    document.getElementById("quoteDiv").innerHTML = "Hello6"; 
 
    } else if (randNumForQuote == 7) { 
 
    document.getElementById("quoteDiv").innerHTML = "Hello7"; 
 
    } else if (randNumForQuote == 8) { 
 
    document.getElementById("quoteDiv").innerHTML = "Hello8"; 
 
    } else if (randNumForQuote == 9) { 
 
    document.getElementById("quoteDiv").innerHTML = "Hello9"; 
 
    } else if (randNumForQuote == 10) { 
 
    document.getElementById("quoteDiv").innerHTML = "Hello10"; 
 
    } 
 
</script> 
 
</body>

0
<html> 
<body> 
    <div id="quoteDiv"></div> 



</body> 
<script> 
// self executing function here 

    var randNumForQuote = Math.floor((Math.random() * 11)); 

    if (randNumForQuote == 0) { 
    document.getElementById("quoteDiv").innerHTML = "Hello"; 
    } else if (randNumForQuote == 1) { 
    document.getElementById("quoteDiv").innerHTML = "Hello1"; 
    } else if (randNumForQuote == 2) { 
    document.getElementById("quoteDiv").innerHTML = "Hello2"; 
    } else if (randNumForQuote == 3) { 
    document.getElementById("quoteDiv").innerHTML = "Hello3"; 
    } else if (randNumForQuote == 4) { 
    document.getElementById("quoteDiv").innerHTML = "Hello4"; 
    } else if (randNumForQuote == 5) { 
    document.getElementById("quoteDiv").innerHTML = "Hello5"; 
    } else if (randNumForQuote == 6) { 
    document.getElementById("quoteDiv").innerHTML = "Hello6"; 
    } else if (randNumForQuote == 7) { 
    document.getElementById("quoteDiv").innerHTML = "Hello7"; 
    } else if (randNumForQuote == 8) { 
    document.getElementById("quoteDiv").innerHTML = "Hello8"; 
    } else if (randNumForQuote == 9) { 
    document.getElementById("quoteDiv").innerHTML = "Hello9"; 
    } else if (randNumForQuote == 10) { 
    document.getElementById("quoteDiv").innerHTML = "Hello10"; 
    } 
</script> 

</html> 
+0

Könnten Sie in eine Beschreibung (muss nicht lang sein) von dem, was der Code tut, um den Benutzer zu beheben Problem? :) –

Verwandte Themen