From 29135a4576714942566960fe01b1bbe303bba156 Mon Sep 17 00:00:00 2001 From: Gerardo Date: Thu, 5 Mar 2026 12:28:56 -0600 Subject: [PATCH] 5 bugs found and fixed --- index.js | 61 +++++++++++++++++++++++++++++--------------------------- 1 file changed, 32 insertions(+), 29 deletions(-) diff --git a/index.js b/index.js index 4c2a94e..8557fb7 100644 --- a/index.js +++ b/index.js @@ -1,14 +1,14 @@ -const guessInput = document.getElementById('guess'); -const submitButton = document.getElementById('submit'); -const resetButton = document.getElementById('reset'); -const messages = document.getElementsByClassName('message'); -const tooHighMessage = document.getElementById('too-high'); -const tooLowMessage = document.getElementById('too-low'); -const maxGuessesMessage = document.getElementById('max-guesses'); -const numberOfGuessesMessage = document.getElementById('number-of-guesses'); -const correctMessage = document.getElementById('correct'); - -let targetNumber; +const guessInput = document.getElementById("guess"); +const submitButton = document.getElementById("submit"); +const resetButton = document.getElementById("reset"); +const messages = document.getElementsByClassName("message"); +const tooHighMessage = document.getElementById("too-high"); +const tooLowMessage = document.getElementById("too-low"); +const maxGuessesMessage = document.getElementById("max-guesses"); +const numberOfGuessesMessage = document.getElementById("number-of-guesses"); +const correctMessage = document.getElementById("correct"); + +let targetNumber = 0; let attempts = 0; const maxNumberOfAttempts = 5; @@ -23,6 +23,7 @@ function getRandomNumber(min, max) { } function checkGuess() { + //*1 // Get value from guess input element const guess = parseInt(guessInput.value, 10); attempts = attempts + 1; @@ -30,10 +31,11 @@ function checkGuess() { hideAllMessages(); if (guess === targetNumber) { - numberOfGuessesMessage.style.display = ''; + //*2 + numberOfGuessesMessage.style.display = ""; numberOfGuessesMessage.innerHTML = `You made ${attempts} guesses`; - correctMessage.style.display = ''; + correctMessage.style.display = ""; submitButton.disabled = true; guessInput.disabled = true; @@ -41,50 +43,51 @@ function checkGuess() { if (guess !== targetNumber) { if (guess < targetNumber) { - tooLowMessage.style.display = ''; + tooLowMessage.style.display = ""; } else { - tooLowMessage.style.display = ''; + tooHighMessage.style.display = ""; //*3 } const remainingAttempts = maxNumberOfAttempts - attempts; - numberOfGuessesMessage.style.display = ''; + numberOfGuessesMessage.style.display = ""; numberOfGuessesMessage.innerHTML = `You guessed ${guess}.
${remainingAttempts} guesses remaining`; } - if (attempts ==== maxNumberOfAttempts) { + if (attempts === maxNumberOfAttempts) { submitButton.disabled = true; guessInput.disabled = true; } - guessInput.value = ''; + guessInput.value = ""; - resetButton.style.display = ''; + resetButton.style.display = ""; } function hideAllMessages() { - for (let elementIndex = 0; elementIndex <= messages.length; elementIndex++) { - messages[elementIndex].style.display = 'none'; + for (let elementIndex = 0; elementIndex < messages.length; elementIndex++) { + // *4 + messages[elementIndex].style.display = "none"; } } -funtion setup() { - // Get random number - targetNumber = getRandomNumber(1, 100); +function setup() { + targetNumber = getRandomNumber(1, 100); // Get random number + console.log(`target number: ${targetNumber}`); // Reset number of attempts - maxNumberOfAttempts = 0; + attempts = 0; // Enable the input and submit button - submitButton.disabeld = false; + submitButton.disabled = false; //*5 guessInput.disabled = false; hideAllMessages(); - resetButton.style.display = 'none'; + resetButton.style.display = "none"; } -submitButton.addEventListener('click', checkGuess); -resetButton.addEventListener('click', setup); +submitButton.addEventListener("click", checkGuess); +resetButton.addEventListener("click", setup); setup();