"undefined"を返すJavaScript関数

Question

"random_word（）;"を使用して、4つの中からランダムな名前を選択する簡単なコードを書きました。それは機能しません。それは言う：「未定義」。誰も私を説明して、それを修正するのを手助けすることはできますか？すべての助けに感謝します。

var randomWord; 

var word = new Array(3); 
word[0] = "Michael"; 
word[1] = "Simon"; 
word[2] = "Peter"; 
word[3] = "Mark"; 

function random_word(){ 
var randomWord = word[Math.floor(Math.random()*word.length)] 
} 

random_word(); 

document.write(randomWord); 

Answer 1

あなたはvar文を使用して、機能のプライベート変数にrandomWordを作ります。そうしなければ、結果にグローバル変数を使うことができます。

function random_word() { 
 
    randomWord = word[Math.floor(Math.random() * word.length)]; 
 
} 
 

 
var randomWord; 
 
    word = ["Michael", "Simon", "Peter", "Mark"]; 
 

 
random_word(); 
 
document.write(randomWord);

より良いスタイルとより簡潔には、選択した項目のリターンになります。他の配列にも関数を使うことができるので、配列としてパラメータを使用することもできます。

function getRandomItem(array) { 
 
    return array[Math.floor(Math.random() * array.length)]; 
 
} 
 

 
var words = ["Michael", "Simon", "Peter", "Mark"]; 
 

 
document.write(getRandomItem(words));

ボーナス、間隔でランダムな名前を追加します。

function getRandomItem(array) { 
 
    return array[Math.floor(Math.random() * array.length)]; 
 
} 
 

 
var words = ["Michael", "Simon", "Peter", "Mark"]; 
 

 
setInterval(function() { 
 
    document.getElementById('names').innerHTML += getRandomItem(words) + '<br>'; 
 
}, 2000);

<div id="names"></div>

Answer 2

ローカル変数を作成する関数では、varキーワードを使用しています。あなたがその機能から抜け出すと、グローバルなものは設定されません。

使用この代わりに：

function random_word(){ 
    randomWord = word[Math.floor(Math.random()*word.length)] 
} 

Answer 3

random_word関数コンテキスト内のローカル変数randomWordは外側の文脈の中で宣言randomWordとは何の関係もありません。外部コンテキストrandomWord変数の値をリセットする場合は、関数内にvarキーワードをドロップする必要があります。

function random_word(){ 
    randomWord = word[Math.floor(Math.random()*word.length)] 
} 

しかし、これはプログラミングの良い方法ではありません。理想的には、関数は値を返すべきです。

function random_word(){ 
    return word[Math.floor(Math.random()*word.length)]; 
} 

var randomWord = random_word(); 
var anotherRandomWord = random_word();