2016-11-22 2 views
-2

Re: Виселица играIndexOf не возвращает правильный индекс

Цель: после того, как пользователь угадать букву, получить индекс буквы в слове.

Проблема: Число, которое возвращено, не является правильным индексом слова. Это третья в последней строке кода.

Я новичок, так что прощай меня, пожалуйста :) Это так сложно!

// JavaScript Document 


$(document).ready(function() {  // upon page load 

    var badGuesses; // reset bad guess counter 
    var alphabet = ["A", "B", "C", "D", "E", "F", "G", "H", "I", "J", "K", "L", "M", "N", "O", "P", "Q", "R", "S", "T", "U", "V", "W", "X", "Z"]; // array of letters to choose 
    $("#lettersRemaining").html(alphabet); // gets elements of the alphabet array and displays on UI 

// N E W G A M E B U T T O N C L I C K E D 

$("#newGame").click(function() { // when user clicks on Start New Game button... 

    $("#status").hide(); // upon game reset hide the status section stating game over 
    badGuesses = 0; // reset guess counter which is used later 
    var wordCollection = ["mansion", "statue", "gorilla", "notebook", "smartphone", "illustration", "photo", "elegant", "arborist", "keyboard", "calendar", "capital", "textbook", "horrible", "library"]; // array of words 
    var theWord = wordCollection[Math.floor(Math.random()*wordCollection.length)]; // randomly selects a word 
     console.log("theWord is ...."); 
     console.log(theWord); 

    var theWordLength = theWord.length;  // Get number of characters in randomly selected word 
     console.log("theWordLength is ...."); 
     console.log(theWordLength); 

// D I S P L A Y D A S H E S 

    var combineDashes = []; // creates an array to hold the number of dashes inside the for loop 
    for (var i = theWordLength; i > 0; i--) 
     { 
      combineDashes.push(" - "); // each loop through adds a dash to the array 
     } 
    combineDashes.join(" "); // joins cumulative dashes and converts to a string 
    $("#dashes").html(combineDashes); // displays dashes on UI 

}); 


// G U E S S L E T T E R 

$("#guessLetter").click(function(theWord) {  // when user clicks on the Guess Letter button pass in theWord value .... 
    var letter = $("#theLetter").val(); // gets the letter the user is guessing 
    console.log("letter is ..."); 
    console.log(letter); 

    // Is the letter a good or bad guess? 
    var letterContained = theWord.toString().indexOf(letter); // <-- NOT WORKING!! returns index from theWord for the letter guessed; -1 means bad guess 
    console.log("letterContained is..."); 
    console.log(letterContained); 


}); 

}); 
+0

Вы пробовали theWord.indexOf без toString() –

+0

Я думаю, что ваш алфавит - это верхний регистр, а ваши слова - в нижнем регистре, а indexOf чувствителен к регистру. –

+0

Если моя догадка правильная, установите значениеWord = wordCollection [ваш выбор здесь] .toUpperCase() должен помочь –

ответ

0

Это все приведенное выше замечание, но проблема заключалась в том, что theWord нужно быть объявлено на тот же уровень, как badGuesses (и назначено, но не объявлено в NewGame, так же, как badGuesses) и удалить в качестве параметра to guessLetter. Тогда guessLetter имел желаемое значение Word, и вызов indexOf работал.

Смежные вопросы