From 20dea5887b6168e9cb49d880a1d69dc398188395 Mon Sep 17 00:00:00 2001
From: UO289432 <uo289432@uniovi.es>
Date: Mon, 29 Apr 2024 19:14:09 +0200
Subject: [PATCH] Corregidos avisos e2e

---
 webapp/src/components/Pages/Estadisticas.js | 13 +++++------
 webapp/src/components/Pages/Juego.js        | 26 ++++++++++-----------
 webapp/src/components/Temporizador.js       | 15 +++++++-----
 3 files changed, 28 insertions(+), 26 deletions(-)

diff --git a/webapp/src/components/Pages/Estadisticas.js b/webapp/src/components/Pages/Estadisticas.js
index cf46249..c153f49 100644
--- a/webapp/src/components/Pages/Estadisticas.js
+++ b/webapp/src/components/Pages/Estadisticas.js
@@ -14,13 +14,12 @@ const Estadisticas = ({isLogged, username}) => {
     const [completedGames, setCompletedGames] = useState(0);
     const [firstRender, setFirstRender] = useState(false);
 
-  useEffect(() => {
-    if (!firstRender) {
-      statsUser();
-      setFirstRender(true);
-    }
-  }, [firstRender])
-
+    useEffect(() => {
+      if (!firstRender) {
+        statsUser();
+        setFirstRender(true);
+      }
+    }, [firstRender, statsUser])  
     
     async function statsUser(){
       try {
diff --git a/webapp/src/components/Pages/Juego.js b/webapp/src/components/Pages/Juego.js
index a8c2818..8dc60d6 100644
--- a/webapp/src/components/Pages/Juego.js
+++ b/webapp/src/components/Pages/Juego.js
@@ -13,16 +13,16 @@ const Juego = ({isLogged, username, numPreguntas}) => {
   //Array de las cuatros respuestas
   const [resFalse, setResFalse] = useState([])
   //constante booleana para saber si se ha respondido ya o no (si se ha pulsado un boton se pone a true)
-  const [respondido, setRespodido] = useState(false)
+  //const [respondido, setRespodido] = useState(false)
   //constante para saber si ha ganado, booleana
-  const [victoria, setVictoria] = useState(false)
+  //const [victoria, setVictoria] = useState(false)
   //Para saber si el temporizador se ha parado al haber respondido una respuesta
   const [pausarTemporizador, setPausarTemporizador] = useState(false)
   const [restartTemporizador, setRestartTemporizador] = useState(false)
   const [firstRender, setFirstRender] = useState(false);
   const[ready, setReady] = useState(false)
   const [numPreguntaActual, setNumPreguntaActual] = useState(0)
-  const [arPreg, setArPreg] = useState([])
+  const [arPreg] = useState([]);
   const [finishGame, setFinishGame] = useState(false)
   const [numRespuestasCorrectas, setNumRespuestasCorrectas] = useState(0)
   const [numRespuestasIncorrectas, setNumRespuestasIncorrectas] = useState(0)
@@ -39,7 +39,7 @@ const Juego = ({isLogged, username, numPreguntas}) => {
       crearPreguntas(10);
       setFirstRender(true);
     }
-  }, [firstRender])
+  }, [firstRender, crearPreguntas]);  
 
   //Función que genera un numero de preguntas determinado
   async function crearPreguntas(numPreguntas){
@@ -93,16 +93,16 @@ const Juego = ({isLogged, username, numPreguntas}) => {
   const botonRespuesta = (respuesta) => { 
     //Comprueba si la respuesta es correcta o no y pone la variable victoria a true o false
     //por ahora esta variable no se utiliza para nada
-    setRespodido(true)
+    //setRespodido(true)
     setPausarTemporizador(true);
-    if(respuesta == resCorr){
+    if(respuesta === resCorr){
       //Aumenta en 1 en las estadisticas de juegos ganado
       setNumRespuestasCorrectas(numRespuestasCorrectas+1);
-      setVictoria(true)
+      //setVictoria(true)
     }
     else{
       setNumRespuestasIncorrectas(numRespuestasIncorrectas + 1);
-      setVictoria(false)
+      //setVictoria(false)
     }
     cambiarColorBotones(respuesta, true);
 
@@ -123,7 +123,7 @@ const Juego = ({isLogged, username, numPreguntas}) => {
         //Desactivamos TODOS los botones
         button.disabled=true; 
         //Ponemos el boton de la respuesta correcta en verde
-        if(button.textContent.trim() == resCorr) {
+        if(button.textContent.trim() === resCorr) {
           button.style.backgroundColor = "#05B92B";
           button.style.border = "6px solid #05B92B";
         }
@@ -140,8 +140,8 @@ const Juego = ({isLogged, username, numPreguntas}) => {
 
   //Función que cambia el color de un solo boton (acierto)
   function cambiarColorUno(respuesta, button){
-    if(button.textContent.trim()==respuesta.trim()){
-      if((button.textContent.trim() != resCorr)) {
+    if(button.textContent.trim() === respuesta.trim()){
+      if((button.textContent.trim() !== resCorr)) {
         button.style.backgroundColor = "#E14E4E";
         button.style.border = "6px solid #E14E4E";
       }
@@ -150,7 +150,7 @@ const Juego = ({isLogged, username, numPreguntas}) => {
 
   //Funcion que cambia el color de todos los botones (fallo)
   function cambiarColorTodos(button){
-    if(button.textContent.trim() == resCorr) {
+    if(button.textContent.trim() === resCorr) {
       button.style.backgroundColor = "#05B92B";
       button.style.border = "6px solid #05B92B";
     } else{
@@ -182,7 +182,7 @@ const Juego = ({isLogged, username, numPreguntas}) => {
  
   //Funcion que se llama al hacer click en el boton Siguiente
   const clickSiguiente = () => {
-    if(numPreguntaActual==numPreguntas){
+    if(numPreguntaActual === numPreguntas){
       setFinishGame(true)
       setReady(false)
       //finishGame()
diff --git a/webapp/src/components/Temporizador.js b/webapp/src/components/Temporizador.js
index e56fd34..1f0ce5c 100644
--- a/webapp/src/components/Temporizador.js
+++ b/webapp/src/components/Temporizador.js
@@ -4,25 +4,28 @@ const Temporizador =({restart, tiempoInicial, tiempoAcabado, pausa, handleRestar
 
     //Constante que va restando segundos
     const [tiempoSegundos, setTiempoSegundos] = useState(tiempoInicial);
-    
+    //Constante que indica si el temporizador esta en pausa
+    const [pausa, setPausa] = useState(false);
+
     useEffect(() => {
         let intervalID;
         if(restart){
             setTiempoSegundos(tiempoInicial);
-            pausa=false;
+            setPausa(false); 
             handleRestart();
         }
 
         if (tiempoSegundos > 0 && !pausa) {
-        intervalID = setInterval(() => {
-            setTiempoSegundos((prevTiempo) => prevTiempo - 1);
-        }, 1000);
+            intervalID = setInterval(() => {
+                setTiempoSegundos((prevTiempo) => prevTiempo - 1);
+            }, 1000);
         }
-        if(tiempoSegundos<=0)
+        if(tiempoSegundos <= 0)
             tiempoAcabado();
         return () => clearInterval(intervalID);
     }, [tiempoSegundos, pausa, restart]);
 
+
     return (
         <div className="temporizador"> <p> {tiempoSegundos} </p> </div>
     )