base de datos con insert partida corregido

clases/ClasePartida.php

@@ -7,17 +7,27 @@ class Partida
     public $idPartida;
     public $nombrePartida;
     public $idUsuario;
+    public $fechaCreacion;
+    public $fechaInicio;
+    public $fechaFin;
 
     public function __construct(
         $idPartida = null,
         $nombrePartida = null,
-        $idUsuario = null
+        $idUsuario = null,
+        $fechaCreacion = null,
+        $fechaInicio = null,
+        $fechaFin = null
     ) {
         $this->idPartida = $idPartida;
         $this->nombrePartida = $nombrePartida;
         $this->idUsuario = $idUsuario;
+        $this->fechaCreacion = $fechaCreacion;
+        $this->fechaInicio = $fechaInicio;
+        $this->fechaFin = $fechaFin;
     }
 
+   
     public function insertar()
     {
         $conexion = conexionBD();
@@ -25,12 +35,7 @@ class Partida
             return $conexion;
         }
         try {
-            $consulta = $conexion->prepare("
-                INSERT INTO partida
-                (nombrePartida,idUsuario)
-                VALUES
-                (:nombrePartida,:idUsuario)
-            ");
+            $consulta = $conexion->prepare("INSERT INTO partida (nombrePartida,idUsuario) VALUES (:nombrePartida,:idUsuario)");
             $consulta->bindParam(":nombrePartida", $this->nombrePartida);
             $consulta->bindParam(":idUsuario", $this->idUsuario);
             $consulta->execute();
@@ -46,8 +51,7 @@ class Partida
         }
     }
 
-    public function listar()
-    {
+    public function listar(){   
         $conexion = conexionBD();
         if (is_array($conexion)) {
             return $conexion;
@@ -64,18 +68,13 @@ class Partida
         }
     }
 
-    public function buscar()
-    {
+    public function buscar(){    
         $conexion = conexionBD();
         if (is_array($conexion)) {
             return $conexion;
         }
         try {
-            $consulta = $conexion->prepare("
-                SELECT *
-                FROM partida
-                WHERE idPartida=:idPartida
-            ");
+            $consulta = $conexion->prepare("SELECT * FROM partida WHERE idPartida=:idPartida");
             $consulta->bindParam(":idPartida", $this->idPartida);
             $consulta->execute();
             return $consulta->fetch(PDO::FETCH_ASSOC);
@@ -87,20 +86,14 @@ class Partida
         }
     }
 
-    public function editar()
-    {
+    
+    public function editar(){ 
         $conexion = conexionBD();
         if (is_array($conexion)) {
             return $conexion;
         }
         try {
-            $consulta = $conexion->prepare("
-                UPDATE partida
-                SET
-                    nombrePartida=:nombrePartida,
-                    idUsuario=:idUsuario
-                WHERE idPartida=:idPartida
-            ");
+            $consulta = $conexion->prepare("UPDATE partida SET nombrePartida=:nombrePartida, idUsuario=:idUsuario WHERE idPartida=:idPartida");
             $consulta->bindParam(":idPartida", $this->idPartida);
             $consulta->bindParam(":nombrePartida", $this->nombrePartida);
             $consulta->bindParam(":idUsuario", $this->idUsuario);
@@ -117,17 +110,13 @@ class Partida
         }
     }
 
-    public function eliminar()
-    {
+    public function eliminar(){    
         $conexion = conexionBD();
         if (is_array($conexion)) {
             return $conexion;
         }
         try {
-            $consulta = $conexion->prepare("
-                DELETE FROM partida
-                WHERE idPartida=:idPartida
-            ");
+            $consulta = $conexion->prepare("DELETE FROM partida WHERE idPartida=:idPartida");
             $consulta->bindParam(":idPartida", $this->idPartida);
             $consulta->execute();
             return [
@@ -142,48 +131,36 @@ class Partida
         }
     }
 
-    public function iniciar()
-    {
+   
+    public function iniciarPartida(){    
         $conexion = conexionBD();
-        if (is_array($conexion)) {
+        if (is_array($conexion)){
             return $conexion;
         }
         try {
-            $consulta = $conexion->prepare("
-                UPDATE partida
-                SET fechaInicio = NOW()
-                WHERE idPartida = :idPartida
-                AND fechaInicio IS NULL
-            ");
+            $consulta = $conexion->prepare("UPDATE partida SET fechaInicio=NOW() WHERE idPartida=:idPartida AND fechaInicio IS NULL");
             $consulta->bindParam(":idPartida", $this->idPartida);
             $consulta->execute();
             return [
-                "exito" => "La partida se inició correctamente.",
+                "exito" => "La partida ha comenzado.",
                 "exito_tecnico" => $this->idPartida
             ];
         } catch (PDOException $error) {
             return [
-                "error" => "No se pudo iniciar la partida.",
+                "error" => "Ocurrió un error al iniciar la partida.",
                 'error_tecnico' => $error->getMessage()
             ];
         }
     }
 
-
-    public function finalizar()
-    {
+    
+    public function finalizar(){  
         $conexion = conexionBD();
         if (is_array($conexion)) {
             return $conexion;
         }
         try {
-            $consulta = $conexion->prepare("
-                UPDATE partida
-                SET fechaFin = NOW()
-                WHERE idPartida = :idPartida
-                AND fechaInicio IS NOT NULL
-                AND fechaFin IS NULL
-            ");
+            $consulta = $conexion->prepare("UPDATE partida SET fechaFin=NOW() WHERE idPartida=:idPartida AND fechaFin IS NULL");
             $consulta->bindParam(":idPartida", $this->idPartida);
             $consulta->execute();
             return [
@@ -192,30 +169,83 @@ class Partida
             ];
         } catch (PDOException $error) {
             return [
-                "error" => "No se pudo finalizar la partida.",
+                "error" => "Ocurrió un error al finalizar la partida.",
                 'error_tecnico' => $error->getMessage()
             ];
         }
     }
 
+    
+      //true si la partida ya tiene fecha de fin (fechaFin no es NULL)
+      //Devuelve false si la partida no existe
+     
+    public function estaFinalizada() {   
+        $datos = $this->buscar();
+        if (is_array($datos) && isset($datos["error"])) {
+            return $datos;
+        }
+        if (!$datos) {
+            return false;
+        }
+        return $datos["fechaFin"] !== null;
+    }
 
-    public function getPreguntas()
-    {
+    
+     //Devuelve la partida más reciente del usuario que aún no ha
+     //terminado (fechaFin NULL), o false si no tiene ninguna.
+     //Incluye también las salas que todavía no han empezado.
+     
+    public function partidaEnCursoDeUsuario(){  
         $conexion = conexionBD();
         if (is_array($conexion)) {
             return $conexion;
         }
         try {
-            $consulta = $conexion->prepare("
-                SELECT
-                    pxp.idPartidaxPregunta,
-                    p.*
-                FROM partidaxpregunta pxp
-                INNER JOIN pregunta p
-                    ON pxp.idPregunta=p.idPregunta
-                WHERE pxp.idPartida=:idPartida
-                ORDER BY pxp.idPartidaxPregunta
-            ");
+            $consulta = $conexion->prepare("SELECT * FROM partida WHERE idUsuario=:idUsuario AND fechaFin IS NULL ORDER BY idPartida DESC LIMIT 1");
+            $consulta->bindParam(":idUsuario", $this->idUsuario);
+            $consulta->execute();
+            return $consulta->fetch(PDO::FETCH_ASSOC);
+        } catch (PDOException $error) {
+            return [
+                "error" => "Ocurrió un error al buscar la partida en curso.",
+                'error_tecnico' => $error->getMessage()
+            ];
+        }
+    }
+
+    
+    public function listarSalasAbiertas() {  
+        $conexion = conexionBD();
+        if (is_array($conexion)) {
+            return $conexion;
+        }
+        try {
+            $consulta = $conexion->prepare("SELECT p.idPartida, p.nombrePartida, e.idEquipo, e.nombreEquipo, u.idUsuario, u.nombreUsuario
+                FROM partida p INNER JOIN equipoxpartida exp ON exp.idPartida=p.idPartida INNER JOIN equipo e ON e.idEquipo=exp.idEquipo
+                LEFT JOIN equipoxusuario exu ON exu.idEquipo=e.idEquipo LEFT JOIN usuario u ON u.idUsuario=exu.idUsuario WHERE p.fechaInicio IS NULL
+                AND p.fechaFin IS NULL ORDER BY p.idPartida DESC, e.idEquipo ASC");
+            $consulta->execute();
+            return $consulta->fetchAll(PDO::FETCH_ASSOC);
+        } catch (PDOException $error) {
+            return [
+                "error" => "Ocurrió un error al recuperar las salas abiertas.",
+                'error_tecnico' => $error->getMessage()
+            ];
+        }
+    }
+
+    /**
+     * Devuelve las preguntas asignadas a la partida, junto con el
+     * idPartidaxPregunta (necesario para guardar respuestas después).
+     */
+    public function getPreguntas(){
+        $conexion = conexionBD();
+        if (is_array($conexion)) {
+            return $conexion;
+        }
+        try {
+            $consulta = $conexion->prepare("SELECT pxp.idPartidaxPregunta, p.* FROM partidaxpregunta pxp INNER JOIN pregunta p ON pxp.idPregunta=p.idPregunta
+                WHERE pxp.idPartida=:idPartida ORDER BY pxp.idPartidaxPregunta");
             $consulta->bindParam(":idPartida", $this->idPartida);
             $consulta->execute();
             return $consulta->fetchAll(PDO::FETCH_ASSOC);
@@ -227,20 +257,14 @@ class Partida
         }
     }
 
-   
-    public function asignarPregunta($idPregunta)
-    {
+
+    public function asignarPregunta($idPregunta) {  
         $conexion = conexionBD();
         if (is_array($conexion)) {
             return $conexion;
         }
         try {
-            $consulta = $conexion->prepare("
-                INSERT INTO partidaxpregunta
-                (idPartida,idPregunta)
-                VALUES
-                (:idPartida,:idPregunta)
-            ");
+            $consulta = $conexion->prepare("INSERT INTO partidaxpregunta (idPartida,idPregunta) VALUES (:idPartida,:idPregunta)");
             $consulta->bindParam(":idPartida", $this->idPartida);
             $consulta->bindParam(":idPregunta", $idPregunta);
             $consulta->execute();
@@ -256,18 +280,13 @@ class Partida
         }
     }
     //Cuenta cuántas preguntas tiene la partida en total. Es la primera mitad de la comprobación "¿ha terminado el usuario de jugar?".
-    public function contarPreguntasAsignadas()
-    {
+    public function contarPreguntasAsignadas() {  
         $conexion = conexionBD();
         if (is_array($conexion)) {
             return $conexion;
         }
         try {
-            $consulta = $conexion->prepare("
-                SELECT COUNT(*) AS total
-                FROM partidaxpregunta
-                WHERE idPartida=:idPartida
-            ");
+            $consulta = $conexion->prepare("SELECT COUNT(*) AS total FROM partidaxpregunta WHERE idPartida=:idPartida");
             $consulta->bindParam(":idPartida", $this->idPartida);
             $consulta->execute();
             $fila = $consulta->fetch(PDO::FETCH_ASSOC);
@@ -280,21 +299,14 @@ class Partida
         }
     }
 
-    public function contarRespuestasUsuario($idUsuario)
-    {
+    public function contarRespuestasUsuario($idUsuario){   
         $conexion = conexionBD();
         if (is_array($conexion)) {
             return $conexion;
         }
         try {
-            $consulta = $conexion->prepare("
-                SELECT COUNT(*) AS total
-                FROM respuestaalmacenada ra
-                INNER JOIN partidaxpregunta pxp
-                    ON ra.idPartidaxPregunta=pxp.idPartidaxPregunta
-                WHERE pxp.idPartida=:idPartida
-                AND ra.idUsuario=:idUsuario
-            ");
+            $consulta = $conexion->prepare("SELECT COUNT(*) AS total FROM respuestaalmacenada ra INNER JOIN partidaxpregunta pxp
+                    ON ra.idPartidaxPregunta=pxp.idPartidaxPregunta WHERE pxp.idPartida=:idPartida AND ra.idUsuario=:idUsuario");
             $consulta->bindParam(":idPartida", $this->idPartida);
             $consulta->bindParam(":idUsuario", $idUsuario);
             $consulta->execute();
@@ -309,11 +321,11 @@ class Partida
     }
 
     /**
-     * Sustituye al antiguo campo `terminada`: compara cuántas preguntas
-     * tiene la partida contra cuántas ha respondido este usuario.
+     * Indica si ESTE usuario ya ha respondido todas las preguntas:
+     * compara cuántas preguntas tiene la partida contra cuántas ha
+     * respondido él. (La partida completa se marca con finalizar().)
      */
-    public function estaFinalizadaParaUsuario($idUsuario)
-    {
+    public function estaFinalizadaParaUsuario($idUsuario){   
         $totalPreguntas = $this->contarPreguntasAsignadas();
         if (is_array($totalPreguntas)) {
             return $totalPreguntas;
@@ -328,19 +340,13 @@ class Partida
     /**
      * Partidas creadas por un usuario, de más reciente a más antigua.
      */
-    public function listarDelUsuario()
-    {
+    public function listarDelUsuario(){    
         $conexion = conexionBD();
         if (is_array($conexion)) {
             return $conexion;
         }
         try {
-            $consulta = $conexion->prepare("
-                SELECT *
-                FROM partida
-                WHERE idUsuario=:idUsuario
-                ORDER BY idPartida DESC
-            ");
+            $consulta = $conexion->prepare("SELECT * FROM partida WHERE idUsuario=:idUsuario ORDER BY idPartida DESC");
             $consulta->bindParam(":idUsuario", $this->idUsuario);
             $consulta->execute();
             return $consulta->fetchAll(PDO::FETCH_ASSOC);
@@ -352,22 +358,13 @@ class Partida
         }
     }
 
-    public function preguntaAsignada($idPregunta)
-    {
+    public function preguntaAsignada($idPregunta) { 
         $conexion = conexionBD();
-
         if (is_array($conexion)) {
             return $conexion;
         }
-
         try {
-            $consulta = $conexion->prepare("
-                SELECT COUNT(*) AS total
-                FROM partidaxpregunta
-                WHERE idPartida=:idPartida
-                AND idPregunta=:idPregunta
-            ");
-
+            $consulta = $conexion->prepare("SELECT COUNT(*) AS total FROM partidaxpregunta WHERE idPartida=:idPartida AND idPregunta=:idPregunta");
             $consulta->bindParam(":idPartida", $this->idPartida);
             $consulta->bindParam(":idPregunta", $idPregunta);
             $consulta->execute();
@@ -387,24 +384,13 @@ class Partida
      * Vincula un equipo a esta partida en la tabla intermedia 'equipoxpartida'.
      * Permite registrar qué grupos están participando en este torneo.
      */
-    public function registrarEquipoEnPartida($idEquipo)
-    {
-        $conexion = conexionBD(); // Obtenemos la conexión modular del proyecto
-
+    public function registrarEquipoEnPartida($idEquipo){ 
+        $conexion = conexionBD(); 
         if (is_array($conexion)) {
-            return $conexion; // Si hay error en la conexión, lo devuelve
+            return $conexion; 
         }
-
         try {
-            // Preparamos la consulta para insertar la relación en la tabla intermedia real
-            $consulta = $conexion->prepare("
-                INSERT INTO equipoxpartida 
-                (idPartida, idEquipo) 
-                VALUES 
-                (:idPartida, :idEquipo)
-            ");
-
-            // Vinculamos los parámetros con el objeto de la partida actual y el equipo recibido
+            $consulta = $conexion->prepare("INSERT INTO equipoxpartida (idPartida, idEquipo) VALUES (:idPartida, :idEquipo)");
             $consulta->bindParam(":idPartida", $this->idPartida);
             $consulta->bindParam(":idEquipo", $idEquipo);
             $consulta->execute();
@@ -414,7 +400,6 @@ class Partida
                 "exito_tecnico" => $conexion->lastInsertId()
             ];
         } catch (PDOException $error) {
-            // Capturamos el error extrayendo solo el mensaje de texto limpio
             return [
                 "error" => "Ocurrió un error al asignar el equipo a la partida.",
                 "error_tecnico" => $error->getMessage()
@@ -426,28 +411,14 @@ class Partida
      * Obtiene todos los equipos que están compitiendo en esta partida.
      * Es útil para mostrar la lista de rivales antes de empezar a responder.
      */
-    public function getEquiposParticipantes()
-    {
+    public function getEquiposParticipantes(){    
         $conexion = conexionBD();
-
         if (is_array($conexion)) {
             return $conexion;
         }
-
-        try {
-            // Hacemos un INNER JOIN para traer los nombres de los equipos desde la tabla intermedia
-            $consulta = $conexion->prepare("
-                SELECT 
-                    exp.idEquipoxPartida,
-                    e.idEquipo,
-                    e.nombreEquipo
-                FROM equipoxpartida exp
-                INNER JOIN equipo e 
-                    ON exp.idEquipo = e.idEquipo
-                WHERE exp.idPartida = :idPartida
-                ORDER BY e.nombreEquipo ASC
-            ");
-
+        try {           
+            $consulta = $conexion->prepare("SELECT  exp.idEquipoxPartida, e.idEquipo, e.nombreEquipo FROM equipoxpartida exp
+                INNER JOIN equipo e  ON exp.idEquipo = e.idEquipo WHERE exp.idPartida = :idPartida ORDER BY e.nombreEquipo ASC");
             $consulta->bindParam(":idPartida", $this->idPartida);
             $consulta->execute();
 
@@ -464,57 +435,29 @@ class Partida
      * Obtiene todos los jugadores reales (Usuarios) que están jugando en esta partida.
      * Utiliza la tabla 'equipoxusuario' en SQL para cruzar los datos.
      */
-    public function getJugadoresDeLaPartida()
-    {
+    public function getJugadoresDeLaPartida(){   
         $conexion = conexionBD();
-
         if (is_array($conexion)) {
             return $conexion;
         }
-
         try {
-
-            $consulta = $conexion->prepare("
-            SELECT DISTINCT
-                u.idUsuario,
-                u.nombreUsuario,
-                u.email,
-                e.idEquipo,
-                e.nombreEquipo
-            FROM equipoxpartida exp
-
-            INNER JOIN equipoxusuario exu
-                ON exp.idEquipo = exu.idEquipo
-
-            INNER JOIN usuario u
-                ON exu.idUsuario = u.idUsuario
-
-            INNER JOIN equipo e
-                ON exu.idEquipo = e.idEquipo
-
-            WHERE exp.idPartida = :idPartida
-
-            ORDER BY e.nombreEquipo ASC,
-                     u.nombreUsuario ASC
-        ");
-
+            $consulta = $conexion->prepare("SELECT DISTINCT u.idUsuario, u.nombreUsuario, u.email, e.idEquipo, e.nombreEquipo
+            FROM equipoxpartida exp INNER JOIN equipoxusuario exu ON exp.idEquipo = exu.idEquipo INNER JOIN usuario u
+                ON exu.idUsuario = u.idUsuario INNER JOIN equipo e ON exu.idEquipo = e.idEquipo WHERE exp.idPartida = :idPartida
+            ORDER BY e.nombreEquipo ASC, u.nombreUsuario ASC");
             $consulta->bindParam(
                 ":idPartida",
                 $this->idPartida
             );
-
             $consulta->execute();
-
             return $consulta->fetchAll(PDO::FETCH_ASSOC);
         } catch (PDOException $error) {
-
             return [
                 "error" =>
                 "Ocurrió un error al obtener la lista de jugadores.",
-
                 "error_tecnico" =>
                 $error->getMessage()
             ];
         }
     }
-}
+}

mysqulito/triviagmagicoDEFINITIVO.sql

@@ -166,6 +166,9 @@ CREATE TABLE `partida` (
   `idPartida` int(11) NOT NULL AUTO_INCREMENT,
   `nombrePartida` varchar(100) NOT NULL,
   `idUsuario` int(11) NOT NULL,
+  `fechaCreacion` DATETIME NOT NULL DEFAULT CURRENT_TIMESTAMP,
+  `fechaInicio` DATETIME NULL,
+  `fechaFin` DATETIME NULL,
   PRIMARY KEY (`idPartida`),
   KEY `FK idUsuario tabla usuario` (`idUsuario`),
   CONSTRAINT `FK idUsuario tabla usuario` FOREIGN KEY (`idUsuario`) REFERENCES `usuario` (`idUsuario`)
@@ -178,10 +181,10 @@ CREATE TABLE `partida` (
 
 LOCK TABLES `partida` WRITE;
 /*!40000 ALTER TABLE `partida` DISABLE KEYS */;
-INSERT INTO `partida` VALUES
-(1,'Partida de prueba 1',1),
-(2,'Partida de prueba 2',4),
-(3,'Partida de prueba 3',9);
+INSERT INTO `partida` (idPartida, nombrePartida, idUsuario, fechaInicio, fechaFin) VALUES
+(1,'Partida de prueba 1',1, NULL, NULL),
+(2,'Partida de prueba 2',4, NULL, NULL),
+(3,'Partida de prueba 3',9, NULL, NULL);
 /*!40000 ALTER TABLE `partida` ENABLE KEYS */;
 UNLOCK TABLES;