JavaScriptコードのgetOptionコマンド

Question

は、i≥-1 コンマで区切った単語「」、および整数の文字列をパラメータmを取り、Mのi番目 ワードを返す関数getOptionコマンド（M、i）を書きます。 i = -1の場合、i番目の単語 が返されます。エラーチェックは必要ありません。

たとえば、getoption（ "hello、hi、why、ok"、1）はhelloを返す必要があります。

getoption("hello,hi,why,ok",4)はokを返す必要があります。

getoption("hello,hi,why,ok",-1) may hello、hi、who OR okです。

ヒント：文字列mを文字 "、"で分割する必要があります。これは という単語の配列になります。最初の単語がインデックス 0であることを忘れないでください！今、iが-1より大きい場合、配列の開始インデックス が0であるため、要素i + 1を返します。i = -1の場合、0からarray.length-1までの数字 をランダムに生成してから、対応する をこの数値に対応させます。

こんにちは、私は以下のコードを書いており、6行目にタイプエラーがあり、修正方法がわかりません。助けてもらえますか？

Answer 1

function getoption(m, i) { 
 
    var num = m + Math.floor((m + 1) * Math.random()); 
 
    var j = i.indexOf(num); 
 

 
    while (j != -1) { 
 
    num = m + Math.floor((m + 1) * Math.random()); 
 
    j = i.indexOf(num); 
 
    } 
 

 
    return (num); 
 
} 
 

 
document.write(getoption("hello,hi,why,ok", -1))

が望む結果を得るために、私は String#split()で単語列に文字列を分割し、適切な変数に格納することをお勧めします。

次に、インデックスが-1と等しいかどうかを確認し、配列の長さとMath.random()の新しいインデックスを取得することをお勧めします。

単語配列のインデックスを持つ要素を返します。

function getoption(words, index) { 
 
    var wordArray = words.split(','); 
 
    if (index === -1) { 
 
     index = Math.floor(Math.random() * wordArray.length); 
 
    } 
 
    return wordArray[index]; 
 
} 
 

 
document.write(getoption("hello,hi,why,ok", -1))

Answer 2

あなたはもっと簡単な方法をあなたの関数を減らすことができます。

そのような何か試してみてください：D