2017-02-19 5 views
-1

私はループしないので、私はそれをローカル変数、今はループの仕事に問題は、それを回してvarを追加するので、私はランダム変数のproblemeを持っています私は実用的な方法で学びたい飛ばすので、いくつかの基本的なproblemeアム自己学習者であるものを結果()関数は、初心者の質問には申し訳ありませんローカル変数でNANエラー

function randomvariable() { 
var myarray = new Array; 

myarray = [0,1,2,3,4,5,6,7,8,9,10,11,12,13,14,15,16,17,18,19,20] 
var randomvariable = Math.floor(Math.random() * 2); 
result = (myarray[randomvariable]); 
document.getElementById("demo").innerHTML = randomvariable; 

} 

function launchfunction() { 


myFunction(); 
randomvariable(); 
theresult(); 


} 

    function theresult() { 

    f = document.getElementById("dkhoul").value; 
    r=randomvariable; 
    var x = f*1-r; 
    var s = Math.abs(x); 
    document.getElementById("result").innerHTML = s; 



} 
+0

あなたは変数と同じ名前の関数を持っています。 'randomvariable'以外の記述的な関数名を使うことができます。 'getRandomValue'のような名前を使うのが良いでしょう。 –

+1

配列をインスタンス化した行にセミコロンがありません。 – JordanH

答えて

0

は、このコードでそんなに間違っている[OK]を、私はあなたが行くお勧め基本に戻る...あなたのここの目的も全く不思議です。

しかし、それが助けてくれると思っていますが、私が意図したと考えている機能的でクリーンアップされたスニペットがあります。

グッドluck-

function randomVariable() { 
 
    var myArray = []; 
 
    for(i=0;i<21;i++) { 
 
    myArray.push(i); 
 
    } 
 
    var randomVar = Math.floor(Math.random() * myArray.length); 
 
    result = myArray[randomVar]; 
 
    document.getElementById("demo").innerHTML = randomVar; 
 
} 
 

 
function getResult() { 
 
    f = document.getElementById("demo").innerHTML; 
 
    r = Math.random()*2; 
 
    var x = f*(1-r); 
 
    var s = Math.abs(x); 
 
    document.getElementById("result").innerHTML = s; 
 
} 
 

 
function launchFunction() { 
 
    randomVariable(); 
 
    getResult(); 
 
} 
 

 
launchFunction();
<span id="demo"></span><br> 
 
<span id="result"></span>