diff --git a/index.js b/index.js index 4c2a94e..44f2362 100644 --- a/index.js +++ b/index.js @@ -10,6 +10,7 @@ const correctMessage = document.getElementById('correct'); let targetNumber; let attempts = 0; + const maxNumberOfAttempts = 5; // Returns a random number from min (inclusive) to max (exclusive) @@ -43,7 +44,8 @@ function checkGuess() { if (guess < targetNumber) { tooLowMessage.style.display = ''; } else { - tooLowMessage.style.display = ''; + /*Bug 4 - Changed variable from tooLowMessage to tooHighMessage*/ + tooHighMessage.style.display = ''; } const remainingAttempts = maxNumberOfAttempts - attempts; @@ -51,8 +53,8 @@ function checkGuess() { numberOfGuessesMessage.style.display = ''; numberOfGuessesMessage.innerHTML = `You guessed ${guess}.
${remainingAttempts} guesses remaining`; } - - if (attempts ==== maxNumberOfAttempts) { + /*Bug 2 - Equality operator is wrong instead of using === here it is====*/ + if (attempts === maxNumberOfAttempts) { submitButton.disabled = true; guessInput.disabled = true; } @@ -63,24 +65,27 @@ function checkGuess() { } function hideAllMessages() { - for (let elementIndex = 0; elementIndex <= messages.length; elementIndex++) { + for (let elementIndex = 0; elementIndex < messages.length; elementIndex++) { messages[elementIndex].style.display = 'none'; } } - -funtion setup() { +/*Bug-1 - Keyword function is mispelled as funtion*/ +function setup() { // Get random number targetNumber = getRandomNumber(1, 100); console.log(`target number: ${targetNumber}`); // Reset number of attempts - maxNumberOfAttempts = 0; + /*Bug 3 - changed maxNumberOfAttempts to attempts */ + attempts = 0; // Enable the input and submit button - submitButton.disabeld = false; + /*Bug 5 - disabled mispelled as disabeld*/ + submitButton.disabled = false; guessInput.disabled = false; hideAllMessages(); + /*Bug 6 - changed '' to 'none' this will hide the reset button in the start*/ resetButton.style.display = 'none'; }