Problemas con MySQL

[nhygren]

Buenos días, estoy haciendo una página web en el hosting gratuito de Hostinger, y hasta ahora todo genial, pero necesito crearme un procedimiento.

El Procedimiento contiene un cursor ya que necesito hacer un Update sobre varias líneas y almacenando una serie de valores en variables...

El procedimiento me lo crea bien, pero cuando hago el call, se me queda CARGANDo, y no me hace nada. os dejo el código a continuación.

Código MySQL:

CREATE PROCEDURE getPuntosUsuario5()
BEGIN
DECLARE xvariable1 INT;
DECLARE xPiloto VARCHAR(50);
DECLARE xPiloto1 VARCHAR(50);
DECLARE xPiloto2 VARCHAR(50);
DECLARE xPiloto3 VARCHAR(50);
DECLARE xPiloto4 VARCHAR(50);
DECLARE xPiloto5 VARCHAR(50);
DECLARE xPiloto6 VARCHAR(50);
DECLARE xPiloto7 VARCHAR(50);
DECLARE xPiloto8 VARCHAR(50);
DECLARE xPiloto9 VARCHAR(50);
DECLARE xPiloto10 VARCHAR(50);
DECLARE xPilotoPole1 VARCHAR(50);
DECLARE xPilotopole2 VARCHAR(50);
DECLARE xPilotoPole3 VARCHAR(50);
DECLARE xPilotoUltimo VARCHAR(50);
DECLARE xposicion Int;
DECLARE xpole int;
DECLARE xultimo int;
DECLARE xpuntos int;
DECLARE xpos1 int;
DECLARE xpos2 int;
DECLARE xpos3 int;
DECLARE xpos4 int;
DECLARE xpos5 int;
DECLARE xpos6 int;
DECLARE xpos7 int;
DECLARE xpos8 int;
DECLARE xpos9 int;
DECLARE xpos10 int;
DECLARE xpole1 int;
DECLARE xpole2 int;
DECLARE xpole3 int;
DECLARE xpult int;
DECLARE xusuario VARCHAR(75);
DECLARE xliga VARCHAR(100);
DECLARE xfechamodificacion DATETIME;
DECLARE xID int;
DECLARE xaciertos int;
DECLARE done BOOLEAN DEFAULT 0;
 
DECLARE CursorF1 CURSOR FOR 
 
SELECT IdUser,Usuario,Liga,fechamodificacion FROM tbAustralia WHERE Temporada = 2013;
 
DECLARE CONTINUE HANDLER FOR SQLSTATE '02000' SET done=1;
 
OPEN CursorF1;
 
REPEAT
 
    FETCH CursorF1 INTO xID;
 
        SET xvariable1 = 0;
        SET xaciertos = 0;
 
        Select xpos1 = Primero, xpos2 = Segundo, xpos3 = Tercero, xpos4 = Cuarto, xpos5 = Quinto, xpos6 = Sexto, xpos7 = Septimo, xpos8 = Octavo, xpos9 = Noveno, xpos10 = Decimo, xpole1 = pole1, xpole2 = pole2, xpole3 = pole3, xpult = ultimo FROM tbAustralia
        WHERE IDUser = xID;
        
        Select xPiloto1 = Piloto FROM tbPosiciones WHERE  Australia = 1 and Temporada = 2013;
        Select xPiloto2 = Piloto FROM tbPosiciones WHERE  Australia = 2 and Temporada = 2013;
        Select xPiloto3 = Piloto FROM tbPosiciones WHERE  Australia = 3 and Temporada = 2013;
        Select xPiloto4 = Piloto FROM tbPosiciones WHERE  Australia = 4 and Temporada = 2013;
        Select xPiloto5 = Piloto FROM tbPosiciones WHERE  Australia = 5 and Temporada = 2013;
        Select xPiloto6 = Piloto FROM tbPosiciones WHERE  Australia = 6 and Temporada = 2013;
        Select xPiloto7 = Piloto FROM tbPosiciones WHERE  Australia = 7 and Temporada = 2013;
        Select xPiloto8 = Piloto FROM tbPosiciones WHERE  Australia = 8 and Temporada = 2013;
        Select xPiloto9 = Piloto FROM tbPosiciones WHERE  Australia = 9 and Temporada = 2013;
        Select xPiloto10 = Piloto FROM tbPosiciones WHERE  Australia = 10 and Temporada = 2013;
 
        Select xPilotoPole1 = Piloto FROM tbPoles WHERE  Australia = 1 and Temporada = 2013;
        Select xPilotoPole2 = Piloto FROM tbPoles WHERE  Australia = 2 and Temporada = 2013;
        Select xPilotoPole3 = Piloto FROM tbPoles WHERE  Australia = 3 and Temporada = 2013;
 
        Select xPilotoUltimo = Piloto FROM tbUltimos WHERE  Australia = 1 and Temporada = 2013;
 
        IF xPiloto1 = xPos1 THEN    
            SET xvariable1 = xvariable1 + 25;
            SET xaciertos = xaciertos + 1;
        END IF;
        IF xPiloto2 = xPos2 THEN    
            SET xvariable1 = xvariable1 + 18;
            SET xaciertos = xaciertos + 1;
        END IF;
        IF xPiloto3 = xPos3 THEN    
            SET xvariable1 = xvariable1 + 15;
            SET xaciertos = xaciertos + 1;
        END IF;
        IF xPiloto4 = xPos4 THEN    
            SET xvariable1 = xvariable1 + 12;
            SET xaciertos = xaciertos + 1;
        END IF;
        IF xPiloto5 = xPos5 THEN    
            SET xvariable1 = xvariable1 + 10;
            SET xaciertos = xaciertos + 1;
        END IF;
        IF xPiloto6 = xPos6 THEN    
            SET xvariable1 = xvariable1 + 8;
            SET xaciertos = xaciertos + 1;
        END IF;
        IF xPiloto7 = xPos7 THEN    
            SET xvariable1 = xvariable1 + 6;
            SET xaciertos = xaciertos + 1;
        END IF;
        IF xPiloto8 = xPos8 THEN    
            SET xvariable1 = xvariable1 + 4;
            SET xaciertos = xaciertos + 1;
        END IF;
        IF xPiloto9 = xPos9 THEN    
            SET xvariable1 = xvariable1 + 2;
            SET xaciertos = xaciertos + 1;
        END IF;
        IF xPiloto10 = xPos10 THEN  
            SET xvariable1 = xvariable1 + 1;
            SET xaciertos = xaciertos + 1;
        END IF;
 
        IF xPilotoPole1 = xPole1 THEN   
            SET xvariable1 = xvariable1 + 15;
            SET xaciertos = xaciertos + 1;
        END IF;
        IF xPilotoPole2 = xPole2 THEN   
            SET xvariable1 = xvariable1 + 10;
            SET xaciertos = xaciertos + 1;
        END IF;
        IF xPilotoPole3 = xPole3 THEN   
            SET xvariable1 = xvariable1 + 8;
            SET xaciertos = xaciertos + 1;
        END IF;
 
        IF xPilotoUltimo = xpult THEN   
            SET xvariable1 = xvariable1 + 5;
            SET xaciertos = xaciertos + 1;
        END IF;
 
        IF xaciertos > 2 and xaciertos < 4 THEN 
            SET xvariable1 = xvariable1 + 10;
        END IF;
 
 
 
            UPDATE tbAustralia SET Puntos = xvariable1
        WHERE IdUser = xID;
 
UNTIL done END REPEAT;          
CLOSE CursorF1;
END$$
 

[gnzsoloyo]

Bueno, hay varios defectos, algunos de lógica, otros de planteamiento, y uno que supongo que no conoces y es de performance.
- Los cursores en MySQL son una muy mala idea, que debe ser usada si y sólo si es estrictamente necesario, ya que tienen una pésima performance, son lentos y `pueden generar problemas de timeout.
- Por otro lado, una asignación de valores de un SELECT a un grupo de variables no se hace como muestras en las líneas 59 y 60 (te recomiendo usar lo highligts para que las cosas se puedan leer bien). Eso no es una asignación, no estás programando en un lenguaje, sino haceiendo una consulta. Eso que haces es una comparación lógica que devuelve TRUE o FALSE según el caso (debe estar devolviendo todo FALSE.
- No necesitas tresce SELECT para hacer lo que quieres. Con uno sólo alcanzaría.
- No tiene sentido hacer semejante cantidad de IFs, podrías encapsular el comportamiento en una stored function, y además en lugar de hacer semejante cantidad de IFs, sería más práctico usar un controlador de flujo como CASE, que es más eficiente para el caso.


Sin meterme en el uso del CURSOR (que ya dije que no me parece práctico), lo del SELECT sería:

Código MySQL:

Select Primero, Segundo, Tercero, Cuarto, Quinto, Sexto, 
     Septimo, Octavo, Noveno, Decimo, pole1, pole2, pole3, ultimo 
FROM tbAustralia
WHERE IDUser = xID
INTO xpos1, xpos2, xpos3, xpos4, xpos5, xpos6, xpos7, xpos8, 
      xpos9, xpos10, xpole1, xpole2, xpole3, xpult;
 

Respecto a los doce SELECT, yo lo plantearía:

Código MySQL:

Select 
    IF(Australia = 1, Piloto, '') P1,
    IF(Australia = 2, Piloto, '') P2,
    IF(Australia = 3, Piloto, '') P3,
    IF(Australia = 4, Piloto, '') P4,
    IF(Australia = 5, Piloto, '') P5,
    IF(Australia = 6, Piloto, '') P6,
    IF(Australia = 7, Piloto, '') P7,
    IF(Australia = 8, Piloto, '') P8,
    IF(Australia = 9, Piloto, '') P9,
    IF(Australia = 10, Piloto, '') P10
FROM tbPosiciones 
WHERE  Temporada = 2013
INTO xPiloto1, 
    xPiloto2 , 
    xPiloto3 , 
    xPiloto4 , 
    xPiloto5 , 
    xPiloto6 , 
    xPiloto7 , 
    xPiloto8 , 
    xPiloto9 , 
    xPiloto10;
 

Lo mismo para el otro:

Código MySQL:

Select 
    IF(Australia = 1, Piloto, '') Po1,
    IF(Australia = 2, Piloto, '') Po2,
    IF(Australia = 3, Piloto, '') Po3
FROM tbPoles 
WHERE  Temporada = 2013
INTO xPilotoPole1, 
xPilotoPole2, 
xPilotoPole3;
 

Código MySQL:

Select Piloto
FROM tbUltimos 
WHERE  Australia = 1 and Temporada = 2013
INTO xPilotoUltimo;
 

No me meto todaría con la SF posible, pero si te fijas en el manual de referencia, podrás ver cómo se usa.
de todos modos, por lo que puedo alcanzar a ver, a mi entender tu modelo de datos no está normalizado, tiene algunos defectos severos, y a futuro te traerá problemas. Pero eso es otro tema distinto.

A la noche me fijo mejor en el SP. Ahora estoy en el trabajo.