予期しない値を示すindexOf（）

Question

最初の要素のインデックス、つまり紫色が-1と表示されている間に、正しい結果を得るために予期せぬことが起こります.Hen私は期待どおりの結果を得ることができません。それは私がさらに問題に直面しているので、私はjavascriptに実際にコーディングすることができます。私はあなたができるだけ早く答えてくれることを願っています。

var colors = ["voilet", "indigo", "blue", "green", "yellow", "orange", "red"]; 
 
var target; 
 
var target_index; 
 
var guess_input; 
 
var finished = false; 
 

 
function do_game() { 
 
    var random_number = Math.random() * 7; 
 
    var random_number_integer = Math.floor(random_number); 
 
    var target_index = random_number_integer; 
 
    target = colors[target_index]; 
 
    alert(target); 
 
    while (!finished) { 
 

 
    var guess_input = prompt("I am thinking of a color " + 
 
     "violet,indigo,blue,green,yellow,orange,red" + 
 
     "What is the color?"); 
 
    alert(colors.indexOf(guess_input)); 
 
    finished = check_guess(); 
 
    } 
 
} 
 

 
function check_guess() { 
 

 
    if (colors.indexof(guess_input) < 0) { 
 
    alert('not present'); 
 
    return false; 
 
    } 
 
    if (guess_input > target) { 
 
    alert('you gave large'); 
 
    return false; 
 
    } 
 
    if (guess_input < target) { 
 
    alert('you gave small'); 
 
    return false; 
 
    } 
 
    return true; 
 
}

Answer 1

あなたguess_inputがvioletであり、それはvioletがあなたの配列があり

["voilet","indigo","blue","green","yellow","orange","red"]; 

に存在しないので、これは正常な動作です-1

を返すと言いましたタイプミスROR：あなたはvoiletを書いたの代わりに、それはグローバルではありませんので、violet

Answer 2

guess_inputはdo_game機能のvar内部で定義されています。

varを削除して、他の方法でアクセスできるようにするか、他の方法に渡すことができます。

/* fixed spelling here */ 
 
var colors = ["violet", "indigo", "blue", "green", "yellow", "orange", "red"]; 
 
var target; 
 
var target_index; 
 
var guess_input; 
 
var finished = false; 
 

 
function do_game() { 
 
    var random_number = Math.random() * 7; 
 
    var random_number_integer = Math.floor(random_number); 
 
    var target_index = random_number_integer; 
 
    target = colors[target_index]; 
 
    alert(target); 
 
    while (!finished) { 
 
    /* removed var here */ 
 
    guess_input = prompt("I am thinking of a color " + 
 
     "violet,indigo,blue,green,yellow,orange,red" + 
 
     "What is the color?"); 
 
    alert(colors.indexOf(guess_input)); 
 
    finished = check_guess(); 
 
    } 
 
} 
 

 
function check_guess() { 
 

 
    /* fixed typo here */ 
 
    if (colors.indexOf(guess_input) < 0) { 
 
    alert('not present'); 
 
    return false; 
 
    } 
 
    if (guess_input > target) { 
 
    alert('you gave large'); 
 
    return false; 
 
    } 
 
    if (guess_input < target) { 
 
    alert('you gave small'); 
 
    return false; 
 
    } 
 
    return true; 
 
} 
 

 
do_game()

Answer 3

あなたの配列の紫のスペルを確認してください。

と

if (colors.indexof(guess_input) < 0) { 
    alert('not present'); 
    return false; 
    } 

あなたがここにもタイプミスを持っています。

colors.indexOf(guess_input)