triviagmatico 30/09/26 a 13:30

revert Merge pull request 'Fix inicia partida todos los usuarios' (#21) from partida into main

Reviewed-on: #21

buscadorSala.php

@@ -71,7 +71,7 @@
                 </div>
                 <div class="modal-footer">
                     <button type="button" class="btn btn-secondary" data-bs-dismiss="modal">Cancelar</button>
-                    <button type="button" class="btn btn-kahoot-login" id="btnCrearSala">Crear sala</button>
+                    <button type="button" class="btn btn-kahoot-register" id="btnCrearSala">Crear sala</button>
                 </div>
             </div>
         </div>

controladores/controladorJuego.php

@@ -1,79 +1,118 @@
 <?php
 
 session_start();
-
 require_once '../clases/claseJuego.php';
-
 header('Content-Type: application/json; charset=utf-8');
 
-// Hay que estar logueado y decir de qué partida se trata
+// Comprobamos que haya un usuario conectado
+
 if (!isset($_SESSION['idUsuario'])) {
-    echo json_encode(['error' => 'Debes iniciar sesión para jugar.']);
+    echo json_encode([
+        'error' => 'Debes iniciar sesión para jugar.'
+    ]);
+
     exit;
 }
 
-$idUsuario = (int)$_SESSION['idUsuario'];
+$idUsuario = (int) $_SESSION['idUsuario'];
-$idPartida = intval($_POST['idPartida'] ?? 0);
+
+// Comprobamos que se haya indicado la partida
+
+if (!isset($_POST['idPartida'])) {
+    echo json_encode([
+        'error' => 'No se ha indicado la partida.'
+    ]);
+
+    exit;
+}
+
+$idPartida = intval($_POST['idPartida']);
 
 if ($idPartida <= 0) {
-    echo json_encode(['error' => 'No se ha indicado la partida.']);
+    echo json_encode([
+        'error' => 'La partida indicada no es válida.'
+    ]);
+
     exit;
 }
 
-// Solo pueden jugar los usuarios que están en uno de los equipos de la partida
+// Comprobamos que el usuario pertenezca a la partida
-$perteneceAPartida = Juego::usuarioEnPartida($idPartida, $idUsuario);
+
+$perteneceAPartida = Juego::usuarioEnPartida($idPartida,$idUsuario);
 
 if (is_array($perteneceAPartida)) {
     echo json_encode($perteneceAPartida);
+
     exit;
 }
 
+
 if (!$perteneceAPartida) {
-    echo json_encode(['error' => 'No formas parte de esta partida.']);
+    echo json_encode([
+        'error' => 'No formas parte de esta partida.'
+    ]);
+
     exit;
 }
 
+// Comprobamos la orden
 
-switch ($_POST['orden'] ?? '') {
+$orden = $_POST['orden'] ?? '';
+
+switch ($orden) {
+
+    // CARGAR LAS PREGUNTAS DE LA PARTIDA
 
-    // Devuelve las 10 preguntas con sus respuestas (sin marcar la correcta)
     case 'cargarPreguntas':
 
-        echo json_encode(Juego::cargarPreguntas($idPartida, $idUsuario));
+        $resultado = Juego::cargarPreguntas($idPartida, $idUsuario);
+        echo json_encode($resultado);
 
-        break;
+    break;
 
+    // REGISTRAR RESPUESTA
 
-    // Guarda la respuesta del usuario y dice si ha acertado
     case 'responder':
 
-        $idPartidaxPregunta = intval($_POST['idPartidaxPregunta'] ?? 0);
+        if (!isset($_POST['idPartidaxPregunta']) || !isset($_POST['idRespuesta'])){
-        $idRespuesta = intval($_POST['idRespuesta'] ?? 0);
+            echo json_encode([
-
+                'error' => 'Faltan datos para registrar la respuesta.'
-        if ($idPartidaxPregunta <= 0 || $idRespuesta <= 0) {
+            ]);
-            echo json_encode(['error' => 'Faltan datos para registrar la respuesta.']);
-            break;
-        }
-
-        echo json_encode(
-            Juego::registrarRespuesta(
-                $idPartida,
-                $idUsuario,
-                $idPartidaxPregunta,
-                $idRespuesta
-            )
-        );
 
         break;
+        }
 
+        $idPartidaxPregunta = intval($_POST['idPartidaxPregunta']);
+
+        $idRespuesta = intval($_POST['idRespuesta']);
+
+        if ($idPartidaxPregunta <= 0 || $idRespuesta <= 0){
+
+            echo json_encode([
+                'error' => 'Los datos de la respuesta no son válidos.'
+            ]);
+
+        break;
+        }
+
+        // Registramos la respuesta del usuario
+
+        $resultado = Juego::registrarRespuesta($idPartida, $idUsuario, $idPartidaxPregunta, $idRespuesta);
+
+        echo json_encode($resultado);
+
+    break;
+
+    // Finalizar la partida y registrar los aciertos del usuario
 
-    // Recibe los aciertos del jugador al terminar sus 10 preguntas,
-    // para que resultados.php los pueda usar al montar la hoja de resultados
     case 'finalizarPartida':
 
         $aciertos = intval($_POST['aciertos'] ?? 0);
 
+        // Guardamos los aciertos en la sesión
+
         if (!isset($_SESSION['aciertosPorPartida'])) {
+
             $_SESSION['aciertosPorPartida'] = [];
         }
 
@@ -81,16 +120,21 @@ switch ($_POST['orden'] ?? '') {
 
         echo json_encode([
             'exito' => 'Aciertos registrados correctamente.',
+
             'idPartida' => $idPartida,
+
             'aciertos' => $aciertos
         ]);
 
-        break;
+    break;
-
 
     default:
 
-        echo json_encode(['error' => 'La orden indicada no existe.']);
+        echo json_encode([
+            'error' => 'La orden indicada no existe.'
+        ]);
 
-        break;
+    break;
-}
+}
+
+?>

controladores/controladorPerfil.php

@@ -1,14 +0,0 @@
-<?php
-if (session_status() === PHP_SESSION_NONE) {
-    session_start();
-}
-
-// Redirección si el usuario no ha iniciado sesión
-if (!isset($_SESSION['idUsuario'])) {
-    header('Location: login.php');
-    exit();
-}
-
-$nombreUsuario = htmlspecialchars($_SESSION['nombreUsuario'] ?? 'Usuario');
-$avatarUsuario = htmlspecialchars($_SESSION['avatar'] ?? 'img/avatars/defaultProfile.jpeg');
-?>

css/buscadorSala.css

@@ -77,8 +77,9 @@
 }
 
 .btn-unirse:hover {
-    background-color: #0f56ab;
+    background-color: #1f6f09;
     color: white;
+    transform: translateY(-2px);
 }
 
 .modal-kahoot {

css/perfil.css

@@ -401,3 +401,18 @@
     }
 
 }
+
+
+
+.modal-kahoot {
+    border: none;
+    border-radius: 20px;
+    overflow: hidden;
+    box-shadow: 0 15px 40px rgba(0, 0, 0, 0.25);
+}
+
+.modal-kahoot-header {
+    background-color: var(--kahoot-purple);
+    color: white;
+    border-bottom: none;
+}

js/buscadorSala.js

@@ -276,7 +276,7 @@ $(document).on("click", ".btn-abandonar-sala", function () {
     mensaje.text(
       "Si abandonas la sala como anfitrión, la partida terminará para todos los jugadores.",
     );
-    botonConfirmar.text("Sí, abandonar");
+    botonConfirmar.text("Sí, terminar partida");
   } else {
     titulo.text("¿Abandonar la sala?");
     mensaje.text(

js/cabecera.js

@@ -43,7 +43,7 @@ function comprobarSesion() {
             </button>
             <ul class="dropdown-menu dropdown-menu-end user-dropdown-menu" aria-labelledby="userDropdown">
               <li>
-                <a class="dropdown-item d-flex align-items-center gap-2" href="perfil.html">
+                <a class="dropdown-item d-flex align-items-center gap-2" href="perfil.php">
                   <i class="fa-solid fa-user text-primary"></i> Perfil
                 </a>
               </li>
@@ -202,4 +202,5 @@ $(document).on("click", "#botonCerrarSesion", function (event) {
       window.location.href = "index.php";
     }
   });
-});
+});
+

js/index.js

@@ -36,11 +36,11 @@ function renderizarTablaRanking(jugadores) {
                 <thead class="table-dark">
                     <tr>
                         <th scope="col" style="width: 10%;">#</th>
-                        <th scope="col" class="text-start" style="width: 35%;">Jugador</th>
+                        <th scope="col" class="text-start" style="width: 30%;"><i class="fa-solid fa-user"></i> <br> Jugador</th>
-                        <th scope="col" style="width: 15%;"><i class="fa-solid fa-trophy text-warning me-1"></i> Vic</th>
+                        <th scope="col" style="width: 15%;"><i class="fa-solid fa-trophy text-warning me-1"></i> <br> Victorias</th>
-                        <th scope="col" style="width: 15%;"><i class="fa-solid fa-xmark text-danger me-1"></i> Der</th>
+                        <th scope="col" style="width: 15%;"><i class="fa-solid fa-xmark text-danger me-1"></i> <br> Derrotas</th>
-                        <th scope="col" style="width: 15%;"><i class="fa-solid fa-bullseye text-success me-1"></i> Aciertos</th>
+                        <th scope="col" style="width: 15%;"><i class="fa-solid fa-bullseye text-success me-1"></i> <br> Aciertos</th>
-                        <th scope="col" style="width: 10%;"><i class="fa-solid fa-percent text-info me-1"></i> % Vic</th>
+                        <th scope="col" style="width: 15%;"><i class="fa-solid fa-medal text-info me-1"></i> <br> Efectividad</th>
                     </tr>
                 </thead>
                 <tbody>

js/juego.js

@@ -20,7 +20,18 @@ let bloqueado = false; // true mientras se envía una respuesta o se ve el feedb
 let numeroAciertos = 0; // número de aciertos del jugador
 
 $(document).ready(function () {
-  idPartida = parseInt($("body").data("id-partida"), 10);
+
+  // Obtenemos el idPartida de la URL
+  // Ejemplo: juego.php?idPartida=3
+
+  const parametros = new URLSearchParams(window.location.search);
+
+  idPartida = parseInt(parametros.get("idPartida"), 10);
+
+  if (!idPartida || idPartida < 1) {
+    mostrarError("La partida indicada no es válida.");
+    return;
+  }
 
   // Un solo listener para todos los botones de respuesta
   $("#respuestas").on("click", ".btn-answer", function () {
@@ -103,6 +114,7 @@ function responder($botonPulsado) {
   if (bloqueado) {
     return;
   }
+
   bloqueado = true;
 
   const pregunta = preguntas[indice];
@@ -216,4 +228,4 @@ function mostrarError(mensaje) {
   $("#feedback").html(
     '<a href="buscadorSala.php">Volver al buscador de salas</a>',
   );
-}
+}

js/perfil.js

@@ -1,26 +1,3 @@
-// COMPROBAR SESIÓN Y MOSTRAR NOMBRE/AVATAR
-// (antes en perfil.php)
-
-$.ajax({
-  method: "POST",
-  url: "controladores/controladorUsuario.php",
-  data: { orden: "comprobarSesion" },
-}).done(function (respuesta) {
-  if (typeof respuesta === "string") respuesta = JSON.parse(respuesta);
-
-  if (!respuesta.activa) {
-    // Sin sesión: fuera de aquí
-    window.location.href = "index.php";
-    return;
-  }
-
-  $("#nombre").text(respuesta.usuario.nombreUsuario || "Usuario");
-  $("#fotoPerfil").attr(
-    "src",
-    respuesta.usuario.avatar || "img/avatars/defaultProfile.jpeg"
-  );
-});
-
 // =========================
 // MOSTRAR ESTADISTICAS
 // =========================
@@ -73,8 +50,8 @@ function cambiarNombre() {
       $("#nuevoNombre").val("");
 
     
-      if (typeof comprobarSesion === "function") {
+      if (typeof comprobarEstadoSesion === "function") {
-        comprobarSesion();
+        comprobarEstadoSesion();
       }
     }
   });
@@ -133,8 +110,8 @@ $("#inputFoto").change(function () {
       );
 
       // Actualizamos también el avatar superior en la cabecera 
-      if (typeof comprobarSesion === "function") {
+      if (typeof comprobarEstadoSesion === "function") {
-        comprobarSesion();
+        comprobarEstadoSesion();
       }
     }
   });

juego.php

@@ -1,22 +1,6 @@
-<?php
-session_start();
-
-// Hay que estar logueado para jugar
-if (!isset($_SESSION['idUsuario'])) {
-    header('Location: index.php');
-    exit;
-}
-
-// La partida llega por la URL: juego.php?idPartida=3
-$idPartida = filter_input(INPUT_GET, 'idPartida', FILTER_VALIDATE_INT);
-
-if (!$idPartida || $idPartida < 1) {
-    header('Location: buscadorSala.php');
-    exit;
-}
-?>
 <!DOCTYPE html>
 <html lang="es">
+
 <head>
     <meta charset="UTF-8">
     <meta name="viewport" content="width=device-width, initial-scale=1.0">
@@ -24,28 +8,48 @@ if (!$idPartida || $idPartida < 1) {
     <title>Proyecto Trivial</title>
     <link rel="stylesheet" href="css/juego.css">
     <link rel="icon" type="image/svg+xml" href="img/favicon/favicon.svg">
+
 </head>
-<body data-id-partida="<?= (int)$idPartida ?>">
 
-    <!-- Progreso: "Pregunta 3 de 10" + barra -->
+<body>
+
+    <!-- CABECERA Y PROGRESO -->
+
     <div class="game-header">
-        <span id="contadorPregunta">Cargando…</span>
+
+        <span id="contadorPregunta">
+            Cargando…
+        </span>
+
     </div>
+
     <div class="progress-track">
+
         <div class="progress-fill" id="barraProgreso"></div>
+
     </div>
 
-    <!-- Contenedor de la Pregunta (lo rellena js/juego.js) -->
+    <!-- PREGUNTA -->
+
     <div class="question-container">
-        <div id="textoPregunta">Cargando preguntas…</div>
+
+        <div id="textoPregunta">
+            Cargando preguntas…
+        </div>
+
         <div id="feedback" class="feedback" aria-live="polite"></div>
+
     </div>
 
-    <!-- Contenedor de Respuestas (Estilo Kahoot, lo rellena js/juego.js) -->
+    <!-- RESPUESTAS -->
+
     <div class="answers-grid" id="respuestas"></div>
 
+    <!-- JAVASCRIPT -->
+
     <script src="js/jquery-4.0.0.min.js"></script>
     <script src="js/juego.js"></script>
 
 </body>
+
 </html>

perfil.php

@@ -1,3 +1,17 @@
+<?php
+if (session_status() === PHP_SESSION_NONE) {
+    session_start();
+}
+
+// Redirección si el usuario no ha iniciado sesión
+if (!isset($_SESSION['idUsuario'])) {
+    header('Location: login.php');
+    exit();
+}
+
+$nombreUsuario = htmlspecialchars($_SESSION['nombreUsuario'] ?? 'Usuario');
+$avatarUsuario = htmlspecialchars($_SESSION['avatar'] ?? 'img/avatars/defaultProfile.jpeg');
+?>
 <!DOCTYPE html>
 <html lang="es">
 
@@ -27,7 +41,7 @@
 
 <body>
 
-    <div id="cabecera"></div>
+    <?php include_once 'cabecera.php'; ?>
 
     <div class="perfil-container">
 
@@ -48,14 +62,14 @@
 
                         <!-- Nombre -->
                         <h1 id="nombre" class="nombre">
-                            
+                            <?= $nombreUsuario ?>
                         </h1>
 
                         <!-- FOTO DE PERFIL -->
                         <div class="foto-container">
 
                             <img id="fotoPerfil"
-                                 src="img/avatars/defaultProfile.jpeg"
+                                 src="<?= $avatarUsuario ?>"
                                  alt="Foto de perfil"
                                  class="avatar rounded-circle">
 
@@ -295,7 +309,6 @@
     <script src="https://cdn.jsdelivr.net/npm/bootstrap@5.3.3/dist/js/bootstrap.bundle.min.js"></script>
 
     <!-- Tu JavaScript -->
-    <script>$("#cabecera").load("cabecera.php");</script>
     <script src="js/perfil.js"></script>
 
 </body>

resultados.php

@@ -89,7 +89,7 @@ usort($equiposFinales, function($a, $b) { return $b['puntos'] <=> $a['puntos'];
     <meta name="viewport" content="width=device-width, initial-scale=1.0">
     <title>Resultados - Trivial</title>
     <link rel="stylesheet" href="css/resultados.css">
-    <script src="https://jquery.com"></script>
+    <script src="https://cdn.jsdelivr.net/npm/jquery@4.0.0/dist/jquery.min.js"></script>
 </head>
 <body data-id-partida="<?php echo $idPartida; ?>">