Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

nueva version #3

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

nueva version #3

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
70 changes: 49 additions & 21 deletions index.html
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@

<!DOCTYPE html>
<html>
<head>
Expand Down Expand Up @@ -36,63 +37,85 @@ <h1>Juego Adivina tu número</h1>
<p class="guesses"></p>
<p class="lastResult"></p>
<p class="lowOrHi"></p>
<p class="prueba"></p>
</div>

</body>

<script>
let randomNumber = Math.random() * 10;
let randomNumber = Math.floor(Math.random()*100)+1;

const ATTEMPS = 5;
const ATTEMPS = 10;
const guesses = document.querySelector('.guesses');
const lastResult = document.querySelector('.lastResult');
const lowOrHi = document.querySelector('lowOrHi');
const lowOrHi = document.querySelector('.lowOrHi');
const guessSubmit = document.querySelector('.guessSubmit');
const guessField = document.querySelector('.guessField');
const prueba = document.querySelector('.prueba');

let guessCount = 1;
let resetButton;


function checkGuess() {
let guessCount = 1;
let resetButton;

function checkGuess() {
let userGuess = guessField.value;
if(guessCount === 1) {

if(Number.isInteger(parseFloat(userGuess))){

if(guessCount ==1) {
guesses.textContent = 'Número aleatorio anterior: ';

}
guesses.textContent += userGuess + ' ';

if(userGuess === randomNumber) {
lastResult.textContent = '!!!Pérdistes!!!';
lastResult.style.backgroundColor = 'black';
if(userGuess == randomNumber) {
lastResult.textContent = '!!!FELICITACIONES! ADIVIANSTE EL NUMERO!!!' + " "+ randomNumber;
lastResult.style.backgroundColor = 'green';
lowOrHi.textContent = '';
setGameOver();
} else if(guessCount === ATTEMPS) {
lastResult.textContent = 'Felicitaciones! adivinaste el número!';
} else if(guessCount == ATTEMPS) {
lastResult.textContent = 'PERDISTE! LLEGASTE AL LIMITE DE INTENTOS :C';
lastResult.style.backgroundColor = 'red';
prueba.textContent ='EL NUMERO A ADIMINAR ERA :'+ ' '+ randomNumber;
prueba.style.backgroundColor = 'yellow';
setGameOver();

} else {
lastResult.textContent = 'Incorrecto! ';
lastResult.style.backgroundColor = 'green';
lastResult.style.backgroundColor = 'black';
if(userGuess < randomNumber) {
lowOrHi.textContent = 'El número es mayor!';
lowOrHi.style.color = 'white';
lowOrHi.style.backgroundColor = 'black';
} else if(userGuess > randomNumber) {
lowOrHi.textContent = 'El número es menor!';
lowOrHi.style.color = 'white';
lowOrHi.style.backgroundColor = 'black';
}
}

guessCount++;
guessField.value = '';
guessField.focus();
}
guessSubmit.addeventListener('click', checkGuess);

}else{
alert("ERROR EL NUMERO INGRESADO NO ES ENTERO O EL CAMPO ESTA VACIO");
guessField.value= "";
}

}




guessSubmit.addEventListener('click', checkGuess);

function setGameOver() {
guessField.disabled = true;
guessSubmit.disabled = true;
resetButton = document.createElement('button');
resetButton.textContent = 'Comienza un nuevo juego';
document.body.appendChild(resetButton);
resetButton.addeventListener('click', resetGame);
resetButton.addEventListener('click', resetGame);
}

function resetGame() {
Expand All @@ -111,7 +134,12 @@ <h1>Juego Adivina tu número</h1>

lastResult.style.backgroundColor = 'white';

randomNumber = Math.floor(Math.random()) + 1;
}
randomNumber = Math.floor(Math.random()*100) + 1;
}




</script>
</script>
</html>
</html>
26 changes: 26 additions & 0 deletions test-strategy.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,26 @@
Echo por Yostyn Cabrera

Errores que encontre

En algunas variables que se ivan a utilizar a algunas les hacia falta un "."
tambien hasta abajo no estaba cerrado el </scrip>
el limite de juego estaba en 5 en vez de 10
tambien el math.random no estaba bien establecido para que generara los numeros
algunos colores no eran los que debian
los mensajes salian de un color diferente a que deberian
no habia nada establecido para que no se generaran decimales

el error de que no se generaran numeros con punto decimal lo corregi viendo que hasta abajo ya habia establecido un math.floor
y arriba no tonces solo copie eso par arriba.

No se permitiera que el uruario agregara decimales, solo agregue un if que agarrara todo el codigo para que antes validadra que
enfectivamente eran numeros enteros y no decimales y ya depaso tambien no permite ingresar otro tipo de caracter que no sea numerico.

Revisando tambien encontre que algunos if tenian == de mas y eso me estaba soltando un error

tambien el posicionamiento de los {} tiene bastante que ver normalmente cuando solo se corrige un codigo ya echo es un poco
mas dificil ver los errores cuando un corchete cierra en donde no es que cuando vas escribiendo el codigo de 0

Tuve un error con lo que es la agregada de numeros que no paraba de pedirme numeros pero era por algunas variables mas que todo en los if que no tenia bien agregadas